Dynamische JButtons mit ActionListener

[mororu]

Hallo zusammen, ich habe folgendes Problem. Ich habe ein GUI mit einem Panel drauf. Der User kann nun in einem JTextField eine Anzahl angeben und erstellen drücken. Danach soll im Panel, die angegebene Anzahl Buttons erstellt werden. Soweit so gut. Das Problem, das ich nun habe ist, dass ich den ActionListener nicht zuweisen kann?!?
Habe mal zu Testzwecken folgendes programmiert:

public void createButtons(int count) {
for (int i = 0; i < x; i++) {
            JButton btn = new JButton("Hallo " + i);
            btn.addActionListener(this);
            pnlChooser.add(btn);
        }
}

public void actionPerformed(ActionEvent e) {
        System.out.println("Hallo");
        this.setButtonColor(0);
    }

In der Methode ActionPerformed möchte ich anschliessend prüfen welcher Button gedrückt ist. Je nach Button der gedrückt wurde, sollte dann die Methode setButtonColor(int) aufgerufen werden.

Jetzt gibt mir NetBeans aber immer eine Fehlermeldung an. Bei der Zeile btn.addActionListener(this), sagt er mir "es werde ein ActionListener erwartet und er habe ein Button gefunden". Das ist mir ja noch klar, aber bei allen Beispielen im Internet wird das so gemacht? Was ist denn hier das Problem?

 

[AmunRa]

Hast du in deiner Klasse das Interface ActionListener implementiert?

 

[petterson]

Hallo!

Wenn ich dich richtig verstanden habe (und das Problem identifizieren kann), so musst du herausfinden, WELCHER Button den Event ausgelöst hat, oder?

Ich würde das so lösen (vorsicht, PSEUDOCODE, für etwas Anständiges reicht mein KnowHow noch nicht):
Die Knöpfe würde ich in ein Array oder in eine ArrayList packen.

public void actionPerformed(ActionEvent event){
Object dieQuelle = event.getSource();
[...]
if (dieQuelle == knopfListe[eintrag]){
    knopfListe[eintrag].setButtonColor(0);
}

 

[mororu]

Habe nun nochmals etwas versucht. Bin mir noch nicht sicher ob das so funktioniert. Vorallem ist die Prüfung, welcher Knopf gedrückt wurde, nicht sehr elegant :noe:.

final JButton btn = new JButton(""+(i+1));
            btn.addActionListener(new ActionListener() {
                public void actionPerformed(ActionEvent arg0) {
                    int index = Integer.valueOf(btn.getText());
                    switch(index) {
                        case 1:
                            setButtonColor(0);
                            break;
                        case 2:
                            setButtonColor(1);
                            break;
                        case 3:
                            setButtonColor(2);
                            break;
                        case 4:
                            setButtonColor(3);
                            break;
                        case 5:
                            setButtonColor(4);
                            break;
                        case 6:
                            setButtonColor(5);
                            break;                            
                    }


                } 
            });

 

[SlaterB]

wie wärs mit

int index = Integer.valueOf(btn.getText());
setButtonColor(index-1);

 

[mororu]

Oh mann :lol:, da hatte ich aber nicht nur ein Brett, sondern einen ganzen Baumstamm vor der Stirn. Danke!

 

[Marco13]

Und wenn auf dem Button mal etwas stehen soll, wo man NICHT den Index rausparsen kann, kannst du dir mal http://www.java-forum.org/java-faq-beitraege/49255-componenten-und-listener.html ansehen.

 

[KrokoDiehl]

Ich behaupte mal für mich, dass ich es eleganter finde, wenn man statt den Button-Text sein Action-Command ausliest:

for (int i = 0; i < x; i++)
{
    JButton btn = new JButton("Hallo "+i);
    btn.addActionListener(this);
    btn.setActionCommant(""+i);
    panel.add(btn);
}

...
public void actionPerformed(ActionEvent event)
{
    String cmd   = event.getActionCommand();
    int    index = Integer.valueOf(cmd);
    //usw.
}

...aber das ist im Grunde Jacke wie Hose