ActionListener will nicht wie ich will

[Igäl]

Hallo Leute

Ich weiss dass ich ab und an zu schnell poste. Werd versuchen meine Ungeduld zu zügeln...

Dieses Problem hab ich jetzt aber eine Weile herumgeschleppt, bis ich mich dazu entschlossen habe zu fragen. Meiner Meinung nach müsste es eigentlich funktionieren was ich will (da es das nicht tut, gehe ich davon aus, dass meine Meinung falsch ist und ich sie revidieren lassen muss). Nun brauch ich eure Hilfe:

Stimmt es, dass folgendes Konstrukt (natürlich nicht so wie es da steht, aber sinngemäss), eigentlich die Ausgabe "Klick Overview" in die Konsole schreiben müsste?

public class ContentControl
    implements ActionListener    {
    JButton overView;

    public JPanel getTopPan()    {
        [...] //Panel initialisieren - Das Panel wird am Schluss per return an die Hauptklasse übergeben, die es darstellt
        
        overView = new JButton("OverView");
        myPanel.add(overView);
        overView.addActionListener(this);


        [...] //Mehrer Aktionen und der return an die Hauptklasse
    }

    [...] //Diverses Zeugs (paint() und so)

    public voic actionPerformed(ActionEvent ae)    {
        if(ae.getSource() == "overView")    {
            System.out.println("Klick: Overview");
        }else{
            System.out.prinln("Where are you clickin'?");
        }
        repaint();
    }
}

Da jedesmal "Where are you clickin'?" ausgegeben wird, gehe ich davon aus, dass zumindest der ActionListener funktioniert. Doch trotz Klick auf den Button overView erkennt getSource() nicht dass der User da geklickt hat. Wo steh ich aufm Schlauch?

Gruss Igäl

 

[der JoJo]

du vergleichst den wert aus getSource() mit ==, das macht man nicht.
vergleiche statt dessen mal mit .equals("overView")

#edit
neh das is auch mist,

frage mit instanceof, ob das SourceObject vom type JButton ist,
Wenn ja caste es und hol dir dann dort den Titel

 

[SlaterB]

oder
if(ae.getSource() == this.overView)

 

[@x.l]

weise deinem Button eine ActionCommand zu.

overView.setActionCommand("button1");

dann kannst beim ActionEvent darauf prüfen

if(ae.getActionCommand().equals("button1"))
    // tue was

 

[Igäl]

Muss mich wohl noch daran gewöhnen, dass Java keine automatische Typenkonvertierung kennt...

Aber auch mit
if(ae.getSource().equals("overView"))
komme ich auf das selbe Ergebnis.

 

[Guest]

Hallo Leute

Ich weiss dass ich ab und an zu schnell poste. Werd versuchen meine Ungeduld zu zügeln...

Dieses Problem hab ich jetzt aber eine Weile herumgeschleppt, bis ich mich dazu entschlossen habe zu fragen. Meiner Meinung nach müsste es eigentlich funktionieren was ich will (da es das nicht tut, gehe ich davon aus, dass meine Meinung falsch ist und ich sie revidieren lassen muss). Nun brauch ich eure Hilfe:

Stimmt es, dass folgendes Konstrukt (natürlich nicht so wie es da steht, aber sinngemäss), eigentlich die Ausgabe "Klick Overview" in die Konsole schreiben müsste?

public class ContentControl
    implements ActionListener    {
    JButton overView;

    public JPanel getTopPan()    {
        [...] //Panel initialisieren - Das Panel wird am Schluss per return an die Hauptklasse übergeben, die es darstellt
        
        overView = new JButton("OverView");
        myPanel.add(overView);
        overView.addActionListener(this);


        [...] //Mehrer Aktionen und der return an die Hauptklasse
    }

    [...] //Diverses Zeugs (paint() und so)

    public voic actionPerformed(ActionEvent ae)    {
        if(ae.getSource() == "overView")    {
            System.out.println("Klick: Overview");
        }else{
            System.out.prinln("Where are you clickin'?");
        }
        repaint();
    }
}

Da jedesmal "Where are you clickin'?" ausgegeben wird, gehe ich davon aus, dass zumindest der ActionListener funktioniert. Doch trotz Klick auf den Button overView erkennt getSource() nicht dass der User da geklickt hat. Wo steh ich aufm Schlauch?

Gruss Igäl

Dein if passt nicht. getSource gibt soz. deinen Button zurück.

Wenn overView in der gesamten Klasse bekannt ist kannst du das so

if(ae.getSource() == overView)    { // Ohne Anführungszeichen

machen. Ist der Button nicht in der gesamten Klasse bekannt müsstest du

if (((JButton) ae.getSource()).getActionCommand().equals("OverView")) {

machen.

 

[Igäl]

Herzlichen Dank. Die Lösung vom Gast hat mir geholfen.

@x.l. - Danke... Du hast mir gleich die Lösung geliefert, wie ich die Buttons dynamisch in einer Methode erzeugen und dennoch weiter darauf zugreifen kann.