Где я должен разместить прослушиватель событий?

Я не уверен, читали ли вы это, но я бы начал с чтения The Java™ Tutorials на Написание прослушивателей событий. Это руководство очень помогло мне, когда я учился правильно обрабатывать события GUI. Поскольку необходимо обрабатывать так много событий, Java предоставляет основной интерфейс, называемый EventListener, который не имеет методов (это то, что известно как Интерфейс маркера). Каждый JComponent определяет, какое событие он должен обрабатывать. Например, объекты JButton связаны с другим набором событий, чем объекты JFrame.

Лучшее, что вы можете сделать, это понять основную предпосылку того, для чего нужен этот API. Графические интерфейсы должны управляться событиями, потому что не существует эффективного способа зафиксировать взаимодействие пользователя прямо в момент его выполнения. Например, объект JButton должен прослушивать, по крайней мере, события «щелчка». Это должно быть очевидно для вас. Но есть и другие события, о которых вы, вероятно, не подумали. Например, наведите указатель мыши на кнопку, чтобы отобразить подсказку.

На ваш вопрос,

If listener uses other objects, that objects must be created before listener code... What should I do for this?

Есть несколько способов справиться с этим. Я собираюсь показать вам очень простые случаи, которые, надеюсь, вы сможете использовать для расширения. Предположим, у вас есть панель с двумя кнопками OK и Cancel. Очевидно, вы знаете, что при нажатии на эти две кнопки будут выполняться разные функции. Для этой иллюстрации работа, которую они будут выполнять, не имеет значения. Итак, я просто собираюсь распечатать какое-то сообщение при нажатии. Фрагмент кода ниже показывает только соответствующие фрагменты. Чтобы обрабатывать нажатия кнопок, вы должны добавить ActionListener к каждой из кнопок.

public class MyActionListener implements ActionListener {
    public void actionPerformed(ActionEvent e) {
        JButton button = (JButton)e.getSource();
        String name = button.getName(); // Assuming you set "OK" and "Cancel" as the names
        if (name.equals("OK")
            System.out.println("I will comply with your command!");
        else
            System.out.println("You cancelled your request.");
    }
}

Затем вы можете добавить слушателя к своим кнопкам следующим образом...

MyActionListener listener = new MyActionListener();
cancelBtn.addActionListener(listener);
okBtn.addActionListener(listener);

Это решение должно работать нормально, но оно не очень хорошо масштабируется. Если вы добавите больше кнопок в свое приложение, наличие одного такого слушателя будет плохим, потому что ваш слушатель будет перегружен работой. Несколько лучшим решением является предоставление каждому экземпляру собственного слушателя. Вы можете сделать это, воспользовавшись Анонимные классы. Вот как:

cancelBtn.addActionListener(new MyActionListener(){
    public void actionPerformed(ActionEvent e) {
        System.out.println("I will comply with your command!");
    }
});
okBtn.addActionListener(new MyActionListener(){
    public void actionPerformed(ActionEvent e) {
        System.out.println("You cancelled your request.");
    }
});

Некоторым людям не очень нравятся анонимные классы, потому что это влияет на читабельность вашего кода. Мне нравится тот факт, что у каждого экземпляра объекта есть свой персонализированный прослушиватель, который выполняет именно ту работу, которую должен выполнять экземпляр объекта, когда происходит действие.

Я думаю, что этот подход должен работать для вашего случая просто отлично. Есть и другие способы, но я думаю, что вы должны начать использовать этот простой подход, пока не получите больше знаний о том, как обрабатывать события. Дайте мне знать, что вы думаете.

Вы должны разделить свой ActionListener и свой бизнес-код: вы не должны выполнять длительный запрос в ActionListener, потому что это заморозит ваше приложение, заблокировав EDT (поток отправки событий).

Я бы предпочел сделать следующее при работе с Swing (или другой структурой, такой как Java FX, SWT,...):

• Создайте компонент GUI, такой как поле,...
• Настройте или «инициализируйте» макет, то есть добавьте компонент к их родителю и т. д.
• Настройте любой прослушиватель событий

Это будет (замените «init» на «configure», если хотите):

private JCheckBox checkBox;

MyClass() { // constructor
  this.init();
}

void init() {
  initComponents();
  initEventListeners();
}

void initComponents() {
  this.checkBox = new JCheckBox("..."); // or in the constructor if you want final field.

  // layout for the parent component 
  this.add(new JLabel("Label 1"));
  this.add(checkBox);
}

void initEventListeners() {
  this.checkBox.addActionListener(System.out::println);
}

Кроме того, событие, которое вы прикрепляете к компоненту Swing, должно быть максимально простым, если это лямбда-выражения или анонимные классы: вместо этого создайте закрытый метод и используйте его ссылку:

  btnSearch.addActionListener(this::searchActionListener);

И метод, в котором я использую другой поток (ForkJoinPool.commonPool()) для выполнения длительной работы вне EDT (поток отправки событий). В противном случае интерфейс будет заморожен. Бизнес-метод выполняется снаружи, в другом объекте (здесь он называется business).

Кнопка поиска должна быть отключена, иначе пользователь (в этом случае) может спамить кнопку, что приведет к нежелательным проблемам в базе данных...

private void searchActionListener(ActionEvent event) {
    btnSearch.setEnabled(false);  // user should not be able to search while it's running

    // extract Swing data before running outside the EDT
    String query = txtSearch.getText()
    boolean useName = chkName.isSelected();
    boolean useAddress = chkAddress.isSelected();
    // ... and so on

    // do work OUTSIDE the EDT
    ForkJoinPool.commonPool().execute(() -> {
      Result result = business.find(query, useName, useAddress);
      SwingUtilities.invokeLater(() -> {
        processResult(result); // do whatever is needed
        btnSearch.setEnabled(true);
      });
    });
  }

Как видите, если вы создали свой компонент Swing (chkName, ...) до прослушивателя событий, вы не можете ссылаться на них в методе: вы должны создать их как поле для класса, а не как переменную.

В противном случае вы обязаны создать их перед прослушивателем событий.

Кроме того, хотя это и не обсуждалось в вопросе, вы должны переписать свой запрос, потому что вы вводите SQL:

"(SELECT " + tableName + "_SID FROM " + tableName + " WHERE " + tableName + "_NM LIKE '%" + txtSearch.getText() + "%') UNION ";

'%" + txtSearch.getText() + "%' следует заменить на ?, и вы должны использовать PreparedStatement для передачи параметра, а не вводить его.