哪个更适合在课堂上进行多项操作？匿名内部类或者if语句在actionPerformed（）中？

Question

我正在制作java JFrame中的菜单。我看到了在同一个班级中对不同事件做出不同反应的两种方式。一种是使用匿名内部类是从几年前在这个答案说明： https://*.com/a/10472395/5960074

public static void createMenuBar() 
{ 
    //Main Menu Items 

    menubar.add(fileMenu); 
    MainWindow.window.setJMenuBar(menubar); 

    //Sub menu items 
    saveItem = new JMenuItem("Save"); 
    loadItem = new JMenuItem("Load"); 

    //Add to the "File" menu item 
    fileMenu.add(saveItem); 
    fileMenu.add(loadItem); 

    //Anonymous Inner classes for actionListeners 


    //Action event for saving 
    saveItem.addActionListener(new ActionListener(){ //adding the listener 
     @Override 
     public void actionPerformed(ActionEvent e){ //creating the specific action for save 
      System.out.println("You tried to save."); 
     } 
    }); 

} 

在我的代码看起来这和它的作品（如在这一点上打印“你想救。“到控制台）。

我见过其他人在类中实现ActionListener，然后在方法actionPerformed（）中使用if语句来跟踪事件的来源。所以有一个事件，但是代码在触发它的时候发生了变化。

第一种方法似乎凌乱，第二种方法更容易阅读。第二种方法是否有理由采用第一种方法？

谢谢。

Answer 1

对非匿名类使用Action类（尽管它们可以是静态内部类）。它给你很好的封装，使代码可读，并且在很多方面比使用ActionListener更好。

Answer 2

首先，避免使用static，你真的（真的）不需要它，它会促进糟糕的代码设计。

假设你使用Java 8，从而可以使用拉姆达的：

public void createMenuBar() { 
    ... 
    saveItem.addActionListener(this::save); 
    loadItem.addActionListener(this::load); 
} 

private save(ActionEvent e) { 
    .... 
} 

private load(ActionEvent e) { 
    .... 
} 

如果您不能使用通过内部类是比实现该接口，并使用if报表更好的lambda表达式。

public class MyGui { 
    private class SaveAction implements ActionListener { 
     @Override 
     public void actionPerformed(ActionEvent e) { 
      .... 
     } 
    } 

    private class LoadAction implements ActionListener { 
     @Override 
     public void actionPerformed(ActionEvent e) { 
      .... 
     } 
    } 

public void createMenuBar() { 
    ... 
    saveItem.addActionListener(new SaveAction()); 
    loadItem.addActionListener(new LoadAction()); 
} 
}