어떤 수업에서 여러 행동을하는 것이 더 낫습니까? actionPerformed()의 익명 내부 클래스 또는 if 문?

Question

자바 JFrame에서 메뉴를 만드는 중입니다. 같은 클래스에서 서로 다른 이벤트에 대해 서로 다른 응답을하는 두 가지 방법을 보았습니다. https://stackoverflow.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을 사용하지 마십시오. 정말로 필요하지 않습니다. 나쁜 코드 디자인을 촉진합니다. 당신을 가정

람다의를 사용하여 자바 8을 사용하고 :

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

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

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

당신이 인터페이스를 구현하고 if 문을 사용하는 것보다 더 나은 내부 클래스를 사용하여 람다를 사용할 수없는 경우.

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()); 
} 
}