Swing: jLabel.setText вызывалась слишком поздно

Question

У меня есть приложение с:

• один JTextField для пользовательского ввода,
• один JLabel сюда, показывающий состояние занятости,
• один JTextArea для печати результатов поиска.

Я хочу, чтобы пользователь написал текст в текстовое поле, нажмите enter и просмотрите результаты. У меня есть слушатель, как это:

private void searchForPattern(java.awt.event.ActionEvent evt) { 
     textArea.setText(""); 
     busyLabel.setText("Searchnig ..."); 
     doSearch(); 
     busyLabel.setText("Idle"); 
    } 

В doSearch существует довольно сложный алгоритм, который открывает много файлов XML и поиск по заданному шаблону, это занимает некоторое время. Текст busyLabel изменен на поиск ... только после завершения doSearch. В doSearch нет второго потока, только много операций ввода-вывода.

Как это исправить?

Answer 1

У вас есть классическая проблема параллелизма Swing (учебник: Concurrency in Swing), где doSearch связывает поток событий Swing. Поскольку этот поток - это то, где Swing выполняет всю свою рисование/рисование и взаимодействует с пользователями, если он привязан кодом, который требует сколько-нибудь заметного количества времени, все приложение «зависает» без обновления компонентов и взаимодействия с пользователем игнорируется.

Решение: сделайте это на фоновом потоке, таком как предоставленный объектом SwingWorker. Установите строку «Idle» в JLabel в методе SwingWorker done().

т.е.

private void searchForPattern(java.awt.event.ActionEvent evt) { 
    textArea.setText(""); 
    busyLabel.setText("Searching ..."); 
    new SwingWorker<Void, Void>() { 
    @Override 
    protected Void doInBackground() throws Exception { 
     doSearch(); 
     return null; 
    } 

    @Override 
    protected void done() { 
     busyLabel.setText("Idle"); 
    } 
    }.execute(); 
} 

Answer 2

Вы также можете использовать invokeLater метод в соответствии с SwingUtilities, чтобы обновить компоненты GUI распашные из потоке приложения.

private void searchForPattern(java.awt.event.ActionEvent evt) { 

    SwingUtilities.invokeLater(
    new Runnable(){ 
     public void run(){   
     textArea.setText(""); 
     busyLabel.setText("Searchnig ..."); 
     } 
    } 
); 

    doSearch(); 

    SwingUtilities.invokeLater(
    new Runnable(){ 
     public void run(){ 
     busyLabel.setText("Idle"); 
     } 
    } 
); 
} 

EDIT PS: здесь я предполагаю, что вы не называется функцией searchForPattern() из EDT, что это значит, если вы называете это от ActionListener вы должны назвать так:

new Thread(
    new Runnable(){ 
     public void run(){ 
     searchForPattern(....) 
     } 
    } 
).start();