Java：我应该在哪里放置匿名侦听器逻辑代码？

Question

我们在工作中争论过在 java 中使用侦听器的最佳实践是什么：侦听器逻辑是否应该保留在匿名类中，或者应该在单独的方法中，例如：

button.addActionListener(new ActionListener() {
    public void actionPerformed(ActionEvent e) {
        // code here
    }
});

或者

button.addActionListener(new ActionListener() {
    public void actionPerformed(ActionEvent e) {
        buttonPressed();
    }
});

private void buttonPressed() {
    // code here
}

就可读性而言，哪种方式是推荐的方式和可维护性？我更喜欢将代码保留在侦听器内，只有当代码太大时，才将其设为内部类。这里我假设代码在其他地方没有重复。

谢谢。

Answer 1

我自己制定的规则是：

1. 如果侦听器有超过 2 个方法，则创建一个命名类。
2. 如果侦听器跨越 10 行以上，请创建一个命名类。

非常简单，易于遵循并生成或多或少可读的代码。但我不得不承认我从来没有想过你的例子表明了什么。

Answer 2

这个问题在这里得到了部分回答：

对于听众来说是否有更好的实践

我也不喜欢匿名方式，原因有两个：
1）你不能轻易地重复使用代码，所以一段时间后你可能会发现你有重复的代码2）我发现它破坏了代码的阅读（其他人不同意......个人品味）。我认为每个人都会同意，如果您执行的代码超过 5-10 行，那么匿名内部类不是一个好主意（我认为超过 2 行就太多了）。

Answer 3

我个人认为，“视情况而定”。如果侦听器仅添加到单个组件中，非常简单并且是 GUI 的组成部分，那么匿名内部类会很好地工作。如果监听器很复杂，会被添加到多个组件中，会有自己独立的状态，那么单独的独立类更好。介于两者之间的是私有内部类。 HTH。

编辑：我的错。我正在回答一个不同的问题——是否为侦听器使用单独的独立类。至于是否将匿名内部类代码保持内联还是在方法中，我同意其他发帖人的观点，这将取决于侦听器代码的大小和复杂性。

Answer 4

如果需要从匿名内部类之外的任何内容访问方法 buttonPressed()，那么请使用方法。否则只需将代码放在 actionPerformed() 中即可。

Answer 5

根据我个人的经验，最好遵循 MVC 模式。这样，就有一个代表模型的单独类，它创建所有相关的操作、操作侦听器等。最好将操作表示为最终类字段，并在构造函数中实例化。

例如：

public class Model {
    private final Action buttonAction;
    ...

    public Model(final IController controller) {
       buttonAction = createButtonAction(controller);
       ...
    }

    private Action createButtonAction(final IController controller) {
        Action action = new Action("Action") {
            public void actionPerformed(final ActionEvent e) {
                // do the required action with long running ones on a separate Thread
                controller.run();
            }
        };
        action.set...// other initialisation such as icon etc
        ...
        return action;
    }
    ...

    public Action getButtonAction() {
        return buttonAction;
    }
}

视图也由一个单独的类表示，该类将模型作为其构造函数参数，实际的按钮实例化发生在其中。例如：

public class View extends JPanel {
    public View(final Model model) {
       ...
       JButton button = new JButton(model.getButtonAction();
       ...
    }
}

使用这种方法，将 actionPerformed 的逻辑实现为匿名类的一部分非常方便，因为它几乎没有什么可重用的。所有逻辑都封装到控制器中，因此操作实际上充当控制器调用的包装器，以用作按钮的模型。

Answer 6

是的，这很大程度上取决于您想要做什么。我认为匿名内部类因为两个误区而受到了不好的批评。一是无法重用匿名代码。第二，内存泄漏。但这些问题可以通过简单的方法轻松解决。保存对实例的引用。为了共享代码，只需创建对匿名内部类的引用。

Action action = new AbstractAction("Open") {...};
JButton button = new JButton( action );
JMenuItem menuItem = new JMenuItem( action );
panel.getActionMap().put("openFile", action );

现在您可以在多个组件中重复使用该操作。对于后面的内存泄漏问题，您可以使用该引用来取消注册它，或者第二个也是更简单的选项是 WeakListeners。 WeakListener 的优点是在创建后不需要进行太多管理。

至于风格，我发现匿名侦听器非常方便，并且在某些情况下在处理 Swing 中的线程时更易于阅读，因为它将代码保留在一种方法中（invokeLater、executeInBackground 等）。当您将匿名侦听器委托给实例方法时，我认为它将代码分开，您无法在一个屏幕中读取侦听器之前发生的情况以及与侦听器关联的逻辑。他们往往会被分开，并且很难跟随。

需要注意的是，如果您使用 ActionMap，大多数与键盘监听相关的内存泄漏都会消失。不幸的是，诸如焦点侦听器或向中央系统注册的侦听器之类的事情仍然是一个问题。 （再次强调，WeakListener 在这里很棒）。而且您已经有一个位置来保存每个组件的操作，因此无需创建额外的实例变量来保存它。如果您需要跨两个组件（例如在菜单栏和控件中）重用，请创建一个单独的类。