ActionListener Source vergleichen

[Tarnpflanze]

Hallo

Ich möchte in meinem Taschenrechner abfragen, welcher JButton gedrückt wird. Ich habe die Methode "getSource()" verwendet und es sollte, wenn ich den Button "nr1" drücke dem String "zeichen" die Zahl 1 reinschreiben. Jedoch kommt es immer beim "if-Code" zu else.

Wieso das?

import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import javax.swing.JButton;
import javax.swing.JLabel;


public class DerListener implements ActionListener{
    
    String zeichen;
    JLabel eingabe;
    JButton nr1, nr2, nr3, nr4, nr5, nr6, nr7, nr8, nr9, nr0, punkt, negation,
            clear, klammerAuf, klammerZu;
    
    DerListener(String zeichen){
        this.zeichen = zeichen;
    }
    
    @Override
    public void actionPerformed(ActionEvent e) {
        
        Object source = e.getSource();
        
        if(source == nr1){
            zeichen += "1";
        }else{
            zeichen += "";
        }
    }
}

Ich habe in einer anderen Klasse, bei der ich die Buttons erstellt habe, den ActionListener dem Button hinzugefügt mit der Zeile:

nr1.addActionListener(new DerListener(zeichen));

 

[senior.weber]

Ich habe in einer anderen Klasse, bei der ich die Buttons erstellt habe

Ist der Button also eine andere Instanz ? Damit das funktioniert, müsstest du wohl sowas schreiben:

DerListener listener = new DerListener(zeichen);
listener.nr1 = nr1;
nr1.addActionListener(listener);

 

[Harry Kane]

Alle deine JButton Instanzen sind nicht initialisiert und somit null (es sei denn, sie werden von außerhalb der Klasse initialisiert, was, da sie nicht private sind, durchaus möglich wäre). Der Vergleich auf Identität mit irgendeinem anderen Objekt liefert deshalb immer false.

 

[Tarnpflanze]

(es sei denn, sie werden von außerhalb der Klasse initialisiert, was, da sie nicht private sind, durchaus möglich wäre)

Wie kann ich meine Buttons denn initialisieren? Ich habe bemerkt, über den Konstruktor "DerListener" kann ich nicht mehrere Datentypen übergeben.

(Wahrscheinlich ist es komplett einfach. Aber dies ist mein 1. Projekt, welches ich programmiere)

 

[qdoi123]

JButton nr1 = new JButton("nr1);

Dann noch ActionListener adden und fertig

 

[Harry Kane]

Wie kann ich meine Buttons denn initialisieren? Ich habe bemerkt, über den Konstruktor "DerListener" kann ich nicht mehrere Datentypen übergeben.

Da die Klasse "DerListener" von dir ist, kannst du den mit Parametern versehen, wie du willst. Ich würde aber davon abraten, alle JButtons in deiner Listener-Klasse zu initialisieren (so wie von Senior.Weber beschrieben).
Ich versuche Listenern immer so zu programmieren, daß sie alle Objekte möglichst genau kennen, mit denen sie arbeiten sollen, aber auch nur die. Das ganze sieht dann z. B. so aus:

public class ParametrizedActionListener {

    public static void main(String[] args) {
        JFrame main = new JFrame("Demo");
        JTextField tf = new JTextField();
        main.add(tf, BorderLayout.NORTH);
        JPanel grid = new JPanel();
        grid.setLayout(new GridLayout(3, 3));
        HashMap<JButton, String> map = new HashMap<JButton, String>();
        TextAppender ta = new TextAppender(tf, map);
        for (int i = 1; i < 10; i++) {
            String s = String.valueOf(i);
            JButton jb = new JButton(s);
            map.put(jb, s);
            jb.addActionListener(ta);
            grid.add(jb);
        }
        main.add(grid);
        main.pack();
        main.setVisible(true);
        main.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    }

    static class TextAppender implements ActionListener {

        private JTextField target;
        private Map<JButton, String> map;

        TextAppender(JTextField target, Map<JButton, String> map) {
            this.target = target;
            this.map = map;
        }

        public void actionPerformed(ActionEvent ae) {
            if (!(ae.getSource() instanceof JButton)) {
                return;
            }
            JButton jb = (JButton) ae.getSource();
            String text = map.get(jb);
            if (text != null) {
                target.setText(target.getText() + text);

            }
        }
    }

}

 

[senior.weber]

Hallo,

Lieber "Henry Kane", warum würdest du

davon abraten, alle JButtons in deiner Listener-Klasse zu initialisieren (so wie von Senior.Weber beschrieben).

- das ist doch gar nicht, was mein Code macht. Er setzt nur den Variablenpointer nr1 auf einen offensichtlich bereits initialisierten Button "nr1". Was genau ist daran problematisch?

- Einen Listener zu bauen, dem man eine Map mit JButtons übergibt, die dann die Information "Text auf dem JButton" redundanterweise beinhaltet - ist imo Blödsinn^2 - diese Info bekomm man ja auch über getText()



Hier mal ein sehr gutes Tutorial:

ActionEvent und ActionListener


Das hat den ActionListener in der Klasse definiert, in der auch die Buttons übergeben werden - das hat den Vorteil, dass der Listener alle Komponenten "kennt", aber den Nachteil, dass der Listener nicht woanders verwendet werden kann. Aber für deinen Anwendungsfall wäre es imo völlig in Ordnung.

Als anderer Ansatz könntest du auch JButton#setName(String name) in Swing verwenden, um Aktionen bei bestimmten Buttons (unabhängig vom angezeigten Text auf dem Button) auszuführen:

import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import javax.swing.JButton;
import javax.swing.JLabel;
 
 
public class DerListener implements ActionListener{
 
    String zeichen;

    /*dieser Konstruktor macht keinen Sinn, da Strings immutable sind
DerListener(String zeichen){
        this.zeichen = zeichen;
    }*/

  public String getActualString(){
     return zeichen;
  }
 
    @Override
    public void actionPerformed(ActionEvent e) {
 
        JButton source = (JButton) e.getSource();
 
        if(source.getName().equals("nr1")){
            zeichen += "1";
        }else{
            zeichen += "";
        }

        //oder zb
        zeichen+=source.getName();//name ist auf "1" gesetzt
    }
}

 

[Harry Kane]

- das ist doch gar nicht, was mein Code macht. Er setzt nur den Variablenpointer nr1 auf einen offensichtlich bereits initialisierten Button "nr1". Was genau ist daran problematisch?

- Einen Listener zu bauen, dem man eine Map mit JButtons übergibt, die dann die Information "Text auf dem JButton" redundanterweise beinhaltet - ist imo Blödsinn^2 - diese Info bekomm man ja auch über getText()

Zu 1: Beschäftige dich mal mit dem Unterschied zwischen "Deklaration" und "Initialisierung". Und ich halte es für sinnlos, in einer Klasse 9 JButtons nr1 bis nr9 zu deklarieren und initialisieren und in einer 2. Klasse dasselbe zu tun. Im günstigsten Fall ist das überflüssige Tipparbeit. In einem ungünstigeren Fall erzeugt man dadurch ein zu enge Kopplung zwischen den beteiligten Klassen. Dann kann man gleich alles in eine Klasse klatschen.
Zu 2: Prinzipiell ein korrekter Einwand. Ich finds nur grundätzlich etwas heikel, basierend auf dem Text von JButtons eine Logik aufzubauen. Wenn später die Zahlen durch Icons dargestellt werden sollen, oder wenn bei Drücken einer Taste nicht mehr Zeichen in ein JTextField eingetragen werden sollen, sondern z. B. Funktionen auf einen Stack (immerhin gings hier um einen Taschenrechner), funktioniert das nicht mehr. Wenn man eine Map verwendet, müsste man die Deklaration der Map natürlich anpassen und die Logik im Listener, aber das "Grundgerüst" bliebe bestehen.

Hier mal ein sehr gutes Tutorial:

ActionEvent und ActionListener

Das "sehr gute" Tutorial hat absolutes Einsteigerniveau. Wenn ich schon lese "class XYZ extends JFrame implements ActionListener" - kein Kommentar. Die gezeigten Beispiele sind übrigens für die eigentliche Problematik (das es nämlich mehrere JButtons gibt und je nach JButton ähnliche Aktionen ausgelöst werden sollen) eher irreführend. Es kann doch nicht dein Ernst sein, für 9 JButtons eine 9-stufige If-else-Abfrage zu empfehlen.

import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import javax.swing.JButton;
import javax.swing.JLabel;
 
 
public class DerListener implements ActionListener{
 
    String zeichen;

    /*dieser Konstruktor macht keinen Sinn, da Strings immutable sind
DerListener(String zeichen){
        this.zeichen = zeichen;
    }*/

  public String getActualString(){
     return zeichen;
  }
 
    @Override
    public void actionPerformed(ActionEvent e) {
 
        JButton source = (JButton) e.getSource();
 
        if(source.getName().equals("nr1")){
            zeichen += "1";
        }else{
            zeichen += "";
        }

        //oder zb
        zeichen+=source.getName();//name ist auf "1" gesetzt
    }
}

Dein Kommentar zu dem Konstruktor von DerListener macht für mich keinen Sinn. Wenn man den Konstruktor so wie du empfiehlst, auskommentiert, fliegt eine NPE, sobald auf zeichen zugegriffen wird. S. o.: Schau dir die Basics zu Deklaration und Initialisierung an.

 

[senior.weber]

Lieber Harry Kane,

die Frage wurde von einem "absoluten Einsteiger", Tarnpflanze gestellt. Deshalb ist ein Tutorial "auf Einsteigerniveau" genau das Richtige.

Mein beispielhafter Code gibt eine NPE - hier der Fix:

String zeichen="";

Einsteiger tendieren dazu, zu denken, dass der übergebene String im weiteren Verlauf "geändert wird" - was nicht der Fall ist. Um dem zu entgehen, habe ich den Konstruktor auskommentiert. Falls seine Intention war "'zeichen' ist der String bevor ich weitere Zeichen anhänge" - OK, kann man so machen - jedoch sieht man in seinem Code nicht, wo er den am Ende erstellten String (+= 1) weiter verwendet.

Beste Grüße