Ersetze viele "else if" durch etwas anderes

Question

Ich benutze einen ActionListener und habe viele andere if-Anweisungen, um zu wissen, welche Taste gedrückt wird und je nach der Schaltfläche Code ausführen.

Gibt es eine Möglichkeit, den Code schöner zu machen? Ich habe fast 10 andere, wenn Aussagen aufeinander folgen, gibt es etwas anderes, das ich stattdessen verwenden könnte?

Probe Code:

class BtnListener implements ActionListener { 
      @Override 
     public void actionPerformed(ActionEvent e) { 
      if (e.getSource() == menu.getOpen()) { 
       getFile(); 
      } else if (e.getSource() == btnPlay) { 

      } else if (e.getSource() == btnQuit)) { 
     } 
} 

Dank.

Answer 1

Sie füllen Map<Object, Consumer<ActionEvent>> vor der Verwendung von Listener, z. B. im Konstruktor, wobei der Schlüssel source ist und Wert ein Consumer für Aktionsereignis ist. In Aktion führen Sie einfach Consumer by Key und rufen Sie es auf.

Answer 2

Sie können den Aktionsbefehl der Schaltfläche, und einen Schalter-case-Block:

public void actionPerformed(ActionEvent e){ 

switch(e.getActionCommand()) { 
    case "Open": 
      open(); 
      break; 
    case "Delete": 
      delete(); 
      break; 
    default : 
      break; 

    } 
} 

Natürlich haben Sie die Aktion Befehl jeder Taste zuerst setzen, wie:

openButton.setActionCommand("Open"); 

Beachten Sie, dass Schalter-Fall mit String Objekte existiert erst seit JDK 7: Strings in switch Statements

Answer 3

Sie können Anweisung stattdessen viele andere wechseln, wenn Leiter.

Answer 4

Der beste Weg, um alle "Gemeinkosten" zu vermeiden, ist Lambda-Expressioned die Tasten. Beispiel: