Добавить слушателя внутри или снаружи метода 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 я могу сказать вам, что это не очень хорошая практика, чтобы получить экземпляры компонентов таким образом (через getters). Обычно я настраиваю пользовательский интерфейс для Frame/Dialog в каком-то методе, таком как initComponents(), а затем добавляю все слушатели в некоторый метод, например addListeners(). Я не уверен, что есть одна лучшая практика, как что-то делать - есть много вариантов и личных предпочтений. В общем, однако, ленивый init компонентов, которые вам понадобятся в любом случае (например, эта кнопка, которую я предполагаю), не требуется.

Кроме того, вы должны действительно рассмотреть возможность использования какого-либо менеджера компоновки, такого как 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.

Есть, вероятно, более приятные способы синхронизации кнопки-конструкции, но это один из способов.