JFrame-Unterklasse und ActionListener-Schnittstellenimplementierung

Question

Warum ist extends JFrame implements ActionListener eine schlechte Designentscheidung?

Answer 1

Ein paar Punkte:

• ein JFrame ist wahrscheinlich eine falsche Ansatz Erweiterung
• eine ActionListener auf einem JFrame Die Implementierung wird wahrscheinlich zu nicht-OOP Code führen.

Falscher Ansatz?

Wenn die Idee ist, dass eine GUI-Anwendung gemacht wird, dann macht man nicht eine Erweiterung von JFrame, sondern schreibt tatsächlich eine Anwendung.

Daher wäre die JFrame Teil einer Anwendung, nicht die Anwendung selbst. Daher sollte JFrame ein Objekt in der Klasse sein.

eine ActionListener auf einem JFrame Implementierung wahrscheinlich

Betrachten Sie den folgenden Fall zu nicht-OOP Code führen - wie die GUI-Anwendung beginnt, groß zu bekommen, und wir beginnen, eine Menge Tasten hinzufügen und Menüs, die ActionEvent s ergeben.

Wenn die JFrame selbst die Ereignisse erhalten würde, wie würde dann die Methode actionPerformed aussehen?

Wahrscheinlich so etwas wie dieses:

public void actionPerformed(ActionEvent e) { 
    Object source = e.getSource(); 

    // Find out which component fired the event 
    if (source == masterButton) { 
    // ... do something 
    } else if (source == anotherButton) { 
    // ... do something else 
    } else if (...) 
    // ... and so on ... 
    } else if (...) 
    // ... and so on ... 
    } 
} 

Huch.

Wir werden anfangen, Code zu bekommen, der eng an alle Komponenten in der Anwendung gekoppelt ist, und die Wartbarkeit wird in kürzester Zeit aus dem Fenster sein.

---

Wenn zum Beispiel die GUI Anwendungsinstanzen von ActionLister s hatte, die auf jede Komponente reagiert, dann würden wir in der Lage sein, die Aktionen und die Kopplung des actionPerformed Verfahren mit allen Komponenten in der aufzubrechen GUI.

Zum Beispiel:

JButton masterButton = new JButton(); 
masterButton.addActionListener(new MasterButtonActionListener()); 

JButton anotherButton = new JButton(); 
anotherButton.addActionListener(new AnotherButtonActionListener()); 

Auf diese Weise wird es ActionListeners für jede Taste sein, die vermutlich unterschiedliche Funktionen haben. Die Verantwortung des MasterButtonActionListener wäre, Ereignisse von der masterButton zu behandeln - es muss nicht über andere Knöpfe in der Anwendung wissen.

Dies würde auch die Wiederverwendbarkeit der Komponenten in anderen Anwendungen oder anderen Teilen der Anwendung fördern, ohne Teile der monolithischen if-else-Anweisung in der actionPerformed-Methode kopieren und einfügen zu müssen.

Answer 2

Wenn Sie Effective Java lesen, Artikel 16 heißt es:

"Favor Zusammensetzung über inheritence"

. Lesen Sie die Erklärung gegeben here

Answer 3

Ich glaube, dass es ein schlechtes Design .. Weil Sie Controller-Code mit Blick mischen .. Wenn Sie MVC-Muster folgen wollen, sollten Sie sie nicht mischen ..

Answer 4

Zusätzlich A JFrame erhält im Allgemeinen keine Aktionen. Seine Kindobjekte wären die Wohltäter einer ActionEvent.

Answer 5

Ein besserer und mehr OO-Ansatz wäre, getrennte Aktionsobjekte zu haben. Auf diese Weise können Sie Funktionalität und Status von einer Komponente trennen. Dadurch wird Ihr Code wiederverwendbar, da Sie beispielsweise die gleiche Aktion für eine Schaltfläche und einen Menüeintrag verwenden können.

Beispiel:

class ExitAction extends AbstractAction{  
    public ExitAction(){ 
     super("Exit"); 
    } 
    @Override 
    public void actionPerformed(ActionEvent e) { 
     System.out.println("Exiting"); 
    } 
} 

JButton exitButton = new JButton(new ExitAction()); 

Werfen Sie einen Blick auf How to Use Actions in den Java-Tutorial: