einzelner ActionListener vs anonyme ActionListener

[Blender3D]

Bezug nehmend auf mein einfaches Test Programm habe ich es nun hinbekommen den leeren Graph im Panel anzuzeigen 🙂

Hier der Code:

[..., siehe #19]
3) Du erzeugst 2 ActionListener die den Listener dann an private Funktionen weiterleiten. Besser GraphPanel implementiert den ActionListener und du reagierst dort auf die Ereignisse. --> Viel übersichtlicher im Code.

    @Override
    public void actionPerformed(ActionEvent e) {
        Object source = e.getSource();
        if (source == jBStart) {
            System.out.println("start");
        }

        if (source == jBExit) {
            System.out.println("exit");
        }
    }

 

[mrBrown]

3) Du erzeugst 2 ActionListener die den Listener dann an private Funktionen weiterleiten. Besser GraphPanel implementiert den ActionListener und du reagierst dort auf die Ereignisse. --> Viel übersichtlicher im Code.

Dieser Tipp ist aber Unsinn. Eine Methode pro Aktion ist deutlich besser, als eine Methode für alle Aktionen - völlig unabhängig, ob nun ActionListener oder sonstwas.

 

[Blender3D]

Dieser Tipp ist aber Unsinn

Dann schau Dir an welcher Code besser zu lesen ist.

 

[mrBrown]

Habs mal abgespalten, da's mit dem anderem Thema nicht viel zu tun hat, aber ein ganz interessantes Thema ist...

 

[mrBrown]

Dann schau Dir an welcher Code besser zu lesen ist.

    @Override
    public void actionPerformed(ActionEvent e) {
        Object source = e.getSource();
        if (source == jBStart) {
            System.out.println("start");
        }

        if (source == jBExit) {
            System.out.println("exit");
        }
    }

vs

private void jBStartActionPerformed(ActionEvent evt) {
     System.out.println("start");//Inhalt angepasst, damit das gleiche passiert
}

private void jBExitActionPerformed(ActionEvent evt) {
    System.out.println("exit");
}

Naja, sieht für mich ziemlich eindeutig aus...

* keine Verzweigung nötig (= Übersichtlicher)
* eine Methode pro Aktion (SRP...), bessere Benennung wäre aber durchaus möglich
* Buttons müssen keine Felder sein (= kleinerer Scope = weniger Fehlermöglichkeiten)
* kürzerer Code (wenn man das zu Methodenreferenzen kürzt, was jede gute IDE da macht) (für mich kein Argument, für viele aber schon)

Ich wüsste nicht, was für das implements ActionListener... spricht (was ja nebenbei auch noch den Typ der Klasse erweitert)?

 

[Blender3D]

Naja, sieht für mich ziemlich eindeutig aus...

    // add Listeners
        jBStart.addActionListener(this);
        jBExit.addActionListener(this);

    jBStart.addActionListener(new ActionListener() {
                    public void actionPerformed(ActionEvent evt) {
                        jBStartActionPerformed(evt);
                    }
                });
        jBExit.addActionListener(new ActionListener() {
                    public void actionPerformed(ActionEvent evt) {
                        jBExitActionPerformed(evt);
                    }
                });

Also für mich sieht es nicht so aus. 😉

 

[mrBrown]

    // add Listeners
        jBStart.addActionListener(this);
        jBExit.addActionListener(this);

    jBStart.addActionListener(new ActionListener() {
                    public void actionPerformed(ActionEvent evt) {
                        jBStartActionPerformed(evt);
                    }
                });
        jBExit.addActionListener(new ActionListener() {
                    public void actionPerformed(ActionEvent evt) {
                        jBExitActionPerformed(evt);
                    }
                });

Also für mich sieht es nicht so aus. 😉

In dem einem Fall sehe ich direkt, was eigentlich gemacht wird, in dem anderen muss ich mich erst durch eine weitere Methode wühlen 😉

Aber wie schon gesagt: so gut wie jeder formt das zu Methodenreferenzen um (bzw, jede IDE macht das für einen), dann sieht es so aus:

jBStart.addActionListener(this::jBStartActionPerformed);
jBExit.addActionListener(this::jBExitActionPerformed);

In den meisten IDEs dürfte das sowieso schon als

jBExit.addActionListener(ActionListener(ActionEvent e) {jBExitActionPerformed(e)}

oder vergleichbar dargestellt sein.

 

[Blender3D]

jBStart.addActionListener(this::jBStartActionPerformed);
jBExit.addActionListener(this::jBExitActionPerformed);

Stimmt so sieht das ganze viel besser aus.
😉

 

[ralfb1105]

@mrBrown: Wenn Du schreibst

In den meisten IDEs dürfte das sowieso schon als

Was wäre denn Deiner Meinung nach ein gutes IDE, da Jigloo es ja anscheinend nicht so macht.

Gruß

Ralf

 

[mrBrown]

Ist Jigloo nicht nur ein Eclipse-Plugin? Eclipse sollte sowas eigentlich können, IntelliJ IDEA macht's aber auf jeden Fall

 

[ralfb1105]

Ja, richtig - Jigloo ist ein Eclipse Plugin. Ich werde mir bei Gelegenheit mal IntelliJ ansehen, aber erst einmal mit Eclipse und Jigloo weiter machen denn ich jabe ja mit Java schon genug Baustellen 😉

Gruß

Ralf