ActionPerformed Fehler...

[sicLotus]

Hallo, ich erzeuge in einer Schleife mehrere Buttons.
Da das ganze in einer Schleife geschieht, werden ja "zwangsweise" immer wieder die selben Namen für den Button verwendet und mit dem nächsten Durchlauf überschrieben.
Ungefähr so:

for (int i=0; i<=5;i++) {
   JButton btnGericht = new JButton(""+i);
}

Mein Problem ist jetzt, jeder dieser Buttons braucht noch ein ActionEvent!!!
Ich hab probiert das ganze mit, ich glaube das nennt sich innere Klassen, zu lösen:

for (int i=0; i<=5;i++) {
   JButton btnGericht = new JButton(""+i);
   btnGericht.addActionListener(new ActionListener() {
		      public void actionPerformed( ActionEvent e ) {
                      //event!!!
                      });
}

Wenn ich das jedoch Aufrufe kommt folgender Fehler:
Exception in thread "AWT-EventQueue-0" java.lang.NullPointerException
und dann noch eine ganze Liste mit TraceStack (unknown source)

Wisst ihr wie man das Problem lösen kann? Oder gibts dafür vllt gar keine Lösung?

 

[diggaa1984]

die exception als solche könnte viellecht aufschluss geben + der code der in etwa um die erwähnte zeile herum liegt

 

[icarus2]

Also wenn du mehrere Buttons gleichzeitig haben möchtest, so würde ich entweder von der Klasse JButton einen Array erstellen oder noch besser eine List<JButton>.

Ich verstehe nur leider nicht so ganz was du da überhaupt machen möchtest... vielleicht sowas??
[Java]
package severalbuttons;

import java.awt.Container;
import java.awt.EventQueue;
import java.awt.FlowLayout;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.util.ArrayList;
import java.util.List;

import javax.swing.JButton;
import javax.swing.JFrame;

public class Buttons extends JFrame {

Container c;
List<JButton> buttons; // list in which all JButtons are stored

public Buttons(){

c = getContentPane();
c.setLayout( new FlowLayout() );

buttons = new ArrayList<JButton>();

for( int i = 0; i <= 5; i++ ){
buttons.add(new JButton("" + i)); // new JButton added to list
c.add( buttons.get( i ) ); // JButton is added
// ActionListener is added
buttons.get( i ).addActionListener(new ActionListener(){
public void actionPerformed( ActionEvent e ){
System.out.println( "Hallo" );
}
});
}

setSize( 600, 300 );
setVisible( true );
setDefaultCloseOperation( JFrame.EXIT_ON_CLOSE );

}

public static void main(String[] args) {

EventQueue.invokeLater(new Runnable(){
@Override
public void run(){
new Buttons();
}
});

}

}
[/Java]

Bei mir gibts damit keine NPE.

 

[diggaa1984]

vielleicht kommt die NPE ja auch aus dem Event-teil .. das sehen wir ja eben nich ^^

 

[U2nt]

Hmm alle Buttons in ein array zu packen wäre nicht gerade die idealste Lösung (weiß nich nochnich solange). Man schreibt lieber eine Klasse die vom JButton erbt, und dann die jeweilie Optionen darauf direkt anwendet...
Außerdem verstehe ich nicht was du mit dem

JButton btnGericht = new JButton("" + i)

machen willst... Du überschreibst jedes mal den Button btnGericht...

 

[sicLotus]

Okay, ich denke ich werd nochmal genauer beschreiben was ich da plane

Ich bastel zur Zeit mein aller erstes GUI. Ich habe eine RadioButtonGroup mit 3 Auswahlmöglichkeiten. Je nachdem welche ich auswähle, blätter ich durch meine JPanels (CardLayout). Nun habe ich vorher schon eine Collection(Liste) mit Elementen gegeben und für jedes Element davon möchte ich einen Button erzeugen. (Wiederum soll der Button halt nur einmal erzeugt werden, darum auch das CardLayout zum durchblättern)

Somit sieht meine eigentlicher Code eher in der Form aus:

Gericht g;
for ( Iterator<Gericht> item = karte.getGerichte().iterator(); item.hasNext(); ){
			g = item.next();
			   JButton btnGericht = new JButton(g.getGerichtName());
			   btnGericht.addActionListener(new ActionListener() {
		      public void actionPerformed( ActionEvent e ) {
					tmpBestellung.addBestellung(g);
       }
		      });

Zur kurzen Erklärung, das is ne Liste mit Gerichten, wenn man auf den Button klickt soll man genau dieses Gericht quasi bestellen. Zur Zeit ist das Event nur eine Syso-Ausgabe auf der Console, von daher kann der Fehler "eigentlich" nicht am Event liegen.

Hier mal die genaue Fehlermeldung:

Exception in thread "AWT-EventQueue-0" java.lang.NullPointerException
	at Gui_CardLayout$1.actionPerformed(Gui_CardLayout.java:102)
	at javax.swing.AbstractButton.fireActionPerformed(Unknown Source)
	at javax.swing.AbstractButton$Handler.actionPerformed(Unknown Source)
	at javax.swing.DefaultButtonModel.fireActionPerformed(Unknown Source)
	at javax.swing.DefaultButtonModel.setPressed(Unknown Source)
	at javax.swing.plaf.basic.BasicButtonListener.mouseReleased(Unknown Source)
	at java.awt.Component.processMouseEvent(Unknown Source)
	at javax.swing.JComponent.processMouseEvent(Unknown Source)
	at java.awt.Component.processEvent(Unknown Source)
	at java.awt.Container.processEvent(Unknown Source)
	at java.awt.Component.dispatchEventImpl(Unknown Source)
	at java.awt.Container.dispatchEventImpl(Unknown Source)
	at java.awt.Component.dispatchEvent(Unknown Source)
	at java.awt.LightweightDispatcher.retargetMouseEvent(Unknown Source)
	at java.awt.LightweightDispatcher.processMouseEvent(Unknown Source)
	at java.awt.LightweightDispatcher.dispatchEvent(Unknown Source)
	at java.awt.Container.dispatchEventImpl(Unknown Source)
	at java.awt.Window.dispatchEventImpl(Unknown Source)
	at java.awt.Component.dispatchEvent(Unknown Source)
	at java.awt.EventQueue.dispatchEvent(Unknown Source)
	at java.awt.EventDispatchThread.pumpOneEventForFilters(Unknown Source)
	at java.awt.EventDispatchThread.pumpEventsForFilter(Unknown Source)
	at java.awt.EventDispatchThread.pumpEventsForHierarchy(Unknown Source)
	at java.awt.EventDispatchThread.pumpEvents(Unknown Source)
	at java.awt.EventDispatchThread.pumpEvents(Unknown Source)
	at java.awt.EventDispatchThread.run(Unknown Source)

 

[diggaa1984]

at Gui_CardLayout$1.actionPerformed(Gui_CardLayout.java:102)

kann es sein das tmpBestellung irgendwie nicht initialisiert ist!?

 

[sicLotus]

Omg ja du hast recht, jetzt kann ich die Buttons anklicken und es kommt kein Fehler mehr.. nur bei jedem ButtonClick kommt jetzt das letzte hinzugeschriebene Ereignis. Es wurde also immer wieder überschrieben.. geht das jetzt nur mit ner extra Liste das der sich das merkt oder weiß jmd noch ne andere Möglichkeit?


edit:

habs probiert mit der list auf die buttons und

buttons.get( i ).addActionListener(new ActionListener(){
                public void actionPerformed( ActionEvent e ){
                    System.out.println( "Hallo" );
                }
            });

hat aber leider auch nicht geklappt.. es wurde trotzdem nur das zuletzt hinzugefügte event ausgegeben

 

[Gast2]

Hmm alle Buttons in ein array zu packen wäre nicht gerade die idealste Lösung (weiß nich nochnich solange). Man schreibt lieber eine Klasse die vom JButton erbt, und dann die jeweilie Optionen darauf direkt anwendet...
Außerdem verstehe ich nicht was du mit dem

JButton btnGericht = new JButton("" + i)

machen willst... Du überschreibst jedes mal den Button btnGericht...

he was?

 

[sicLotus]

Das war nur nen Test.. mehr oder weniger ist der Konstruktor vom JButton doch nur der Text der angezeigt wird auf dem Button oder nicht?

edit:

Habs.. hier die Lösung:

for (Gericht item : karte.getGerichtec())
		{
			final Gericht g = item;
			   JButton btnGericht = new JButton(g.getGerichtName());
			   btnGericht.setName(""+g.getGerichtNummer());
			   btnGericht.addActionListener(this);
			   btnGericht.addActionListener(new ActionListener() {
		      public void actionPerformed( ActionEvent e ) {
				bestellen(g);

		        }
		      });