在 get 方法内部或外部添加侦听器

Question

我正在学习 Swing，并使用一系列 get 方法组成了一个界面来添加组件。如下在 get 方法中添加监听器是一种好习惯吗？我想让事情尽可能地解耦。

 private JButton getConnectButton() {
  if (connectButton == null) {
   connectButton = new JButton();
   connectButton.setText("Connect");
   connectButton.setSize(new Dimension(81, 16));
   connectButton.setLocation(new Point(410, 5));

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

  }
  return connectButton;
 }

Answer 1

根据我作为 Swing 开发人员的广泛实践，我可以告诉您，以这种方式（通过 getter）获取组件实例并不是一个好习惯。我通常在 initComponents() 等方法中为 Frame/Dialog 设置 UI，然后在 addListeners() 等方法中添加所有侦听器。我不确定是否有一个最佳实践作为如何做事 - 有很多选择和个人喜好。但是，通常不需要您无论如何都需要的组件的惰性初始化（比如我认为的这个按钮）。

另外 - 你真的应该考虑使用一些布局管理器，例如 MiG，并避免硬编码组件大小。

Answer 2

您似乎正在对connectButton 进行一些延迟初始化。这可能很好，不过，我会这样做：

private void createButton() {
    connectButton = new JButton(new AbstractAction("Connect") {
        public void actionPerformed(ActionEvent e) {
            // actionPerformed code goes here
        }
    });
    connectButton.setText("Connect");

    // Rely on some LayoutManager!
    //connectButton.setSize(new Dimension(81, 16));
    //connectButton.setLocation(new Point(410, 5));
}

private synchronized JButton getConnectButton() {
    if (connectButton == null)
        createButton();

    return connectButton;
}

注意synchronized 的使用。它确保不会发生以下情况：

• 线程 1 调用 getConnectButton() 并看到 connectButton == null
• 线程 2 调用 getConnectButton() 并看到 connectButton == null
• 线程 1 调用 createButton
• 线程 2 调用 createButton。

可能有更好的方法来同步按钮构造，但这是一种方法。