Рабочий поток блокирует перерисовку графического интерфейса

Я пытаюсь сделать экран входа в приложение. Во время входа в базу данных MySQL будет выполнено множество SQL-запросов, и настройка всех параметров может занять несколько секунд. Я хотел бы отобразить экран состояния с помощью макета карты и обновить JLabel во время работы фонового потока.

Вот суть того, что у меня есть для моего рабочего потока:

public class LoginPrepThread extends Thread {

    private final UIMain parent;

    public LoginPrepThread(UIMain w){
        parent = w;
    }

    public void exec(){
        EventQueue.invokeLater(this);
    }

    public void run(){
        try{
            //SqlHelper sql = SqlHelper.instance;
            sleep(500);
            parent.getLoadingLable().setText("Fetching preferences...");
            parent.getMainFrame().revalidate();
            sleep(500);
            parent.getLoadingLable().setText("Scanning workbench...");
            parent.getMainFrame().revalidate();
            sleep(500);
            parent.getLoadingLable().setText("Updating permissions...");
            parent.getMainFrame().revalidate();
            sleep(500);
            parent.getLoadingLable().setText("Finished...Please wait");
            parent.getMainFrame().revalidate();
            sleep(1000);
            parent.getLayout().show(parent.getMainFrame().getContentPane(), "view.main");
        }catch(Exception e){

        }
    }

}

Вот как я это называю (внутри события JButton, после аутентификации):

setActiveProfile(user);
layout.show(frame.getContentPane(), "view.loading");
frame.repaint();
LoginPrepThread pt = new LoginPrepThread(thisTrick);
pt.exec();

Я добавил несколько фиктивных событий, но метка состояния не меняется... есть предложения?

nlowe 12.10.2012 источник

comment

Рад, что ты понял это. Еще одно наблюдение: в вашем примере вы создаете поток, но никогда его не запускаете (вы используете его исключительно как Runnable), поэтому вы никогда не запускаете фоновый поток, что является частью проблемы. - JimN 13.10.2012

Ответы (4)

arrow_upward
4
arrow_downward

Простой рабочий свинг решил это. Думаю, я недостаточно хорошо разбирался в гугле.

public class LoginPrepThread extends SwingWorker<String,String> {

    private final UIMain parent;

    public LoginPrepThread(UIMain w){
        parent = w;
    }

    @Override
    protected String doInBackground() throws Exception {
        try{
            publish("Fetching preferences...");
            Thread.sleep(1000);
            publish("Updating permissions...");
            Thread.sleep(1000);
            publish("Scanning workbench...");
            Thread.sleep(1000);
            publish("Finalizing...");
            Thread.sleep(2000);
            publish("Finished...Please wait");
            Thread.sleep(1000);
            parent.getLayout().show(parent.getMainFrame().getContentPane(), "view.main");
        }catch(Exception e){

        }
        return null;
    }

    protected void process(List<String> item) {
        parent.getLoadingLable().setText(item.get(0));
    }

}

nlowe 12.10.2012

comment

Приятно видеть, что вы научились использовать SwingWorker. В своем ответе я предоставил один из способов использования PropertyChangeListener. Это был бы отличный способ воспользоваться преимуществами программирования, управляемого событиями, и сохранить модульность вашей бизнес-логики и логики обновления пользовательского интерфейса. Надеюсь, это поможет :) - Sujay; 13.10.2012

comment

Рад, что ты разобрался. Пожалуйста, примите ответ, когда у вас появится такая возможность. - Andrew Thompson; 13.10.2012

comment

Осторожно: не обращайтесь к любому экземпляру, связанному с представлением, в doInBackground (как вы делаете в самой последней строке ..) - kleopatra; 13.10.2012

arrow_upward
3
arrow_downward

В идеале вы должны реализовать SwingWorker для выполнения всех ваших тяжелых задач, не связанных с графическим интерфейсом. Вы должны воспользоваться преимуществами событийно-ориентированного программирования.

Что вам нужно сделать, будет примерно так:

Внедрите PropertyChangeListner в свой класс графического интерфейса, прослушивая обновления. На основе изменений свойств обновите метки. Всегда рекомендуется, чтобы один класс GUI обрабатывал все действия по обновлению, связанные с GUI.
Создайте SwingWorker, в котором вы будете выполнять фоновые задачи. Как только появятся доступные обновления, запустите событие изменения свойства и сообщите классу GUI, что есть обновление.

Вот небольшой SSCCE пример того, что вы можете сделать:

import java.beans.PropertyChangeEvent;
import java.beans.PropertyChangeListener;
import javax.swing.SwingWorker;

/**
 *
 * @author Sujay
 */
public class SimpleWorkerUI extends javax.swing.JFrame implements PropertyChangeListener{

    /**
     * Creates new form SimpleWorkerUI
     */
    public SimpleWorkerUI() {
        initComponents();
        Worker worker = new Worker();
        worker.addPropertyChangeListener(this);
        worker.execute();
    }

    /**
     * This method is called from within the constructor to initialize the form.
     * WARNING: Do NOT modify this code. The content of this method is always
     * regenerated by the Form Editor.
     */
    @SuppressWarnings("unchecked")
    // <editor-fold defaultstate="collapsed" desc="Generated Code">
    private void initComponents() {

        jPanel1 = new javax.swing.JPanel();
        jLabel1 = new javax.swing.JLabel();
        jLabel2 = new javax.swing.JLabel();

        setDefaultCloseOperation(javax.swing.WindowConstants.EXIT_ON_CLOSE);

        jLabel1.setText("Current Status: ");

        javax.swing.GroupLayout jPanel1Layout = new javax.swing.GroupLayout(jPanel1);
        jPanel1.setLayout(jPanel1Layout);
        jPanel1Layout.setHorizontalGroup(
            jPanel1Layout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING)
            .addGroup(jPanel1Layout.createSequentialGroup()
                .addContainerGap()
                .addComponent(jLabel1)
                .addPreferredGap(javax.swing.LayoutStyle.ComponentPlacement.RELATED)
                .addComponent(jLabel2)
                .addContainerGap(327, Short.MAX_VALUE))
        );
        jPanel1Layout.setVerticalGroup(
            jPanel1Layout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING)
            .addGroup(jPanel1Layout.createSequentialGroup()
                .addGap(20, 20, 20)
                .addGroup(jPanel1Layout.createParallelGroup(javax.swing.GroupLayout.Alignment.BASELINE)
                    .addComponent(jLabel1)
                    .addComponent(jLabel2))
                .addContainerGap(20, Short.MAX_VALUE))
        );

        javax.swing.GroupLayout layout = new javax.swing.GroupLayout(getContentPane());
        getContentPane().setLayout(layout);
        layout.setHorizontalGroup(
            layout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING)
            .addComponent(jPanel1, javax.swing.GroupLayout.DEFAULT_SIZE, javax.swing.GroupLayout.DEFAULT_SIZE, Short.MAX_VALUE)
        );
        layout.setVerticalGroup(
            layout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING)
            .addComponent(jPanel1, javax.swing.GroupLayout.PREFERRED_SIZE, javax.swing.GroupLayout.DEFAULT_SIZE, javax.swing.GroupLayout.PREFERRED_SIZE)
        );

        pack();
    }// </editor-fold>

    /**
     * @param args the command line arguments
     */
    public static void main(String args[]) {
        /* Set the Nimbus look and feel */
        //<editor-fold defaultstate="collapsed" desc=" Look and feel setting code (optional) ">
        /* If Nimbus (introduced in Java SE 6) is not available, stay with the default look and feel.
         * For details see http://download.oracle.com/javase/tutorial/uiswing/lookandfeel/plaf.html 
         */
        try {
            for (javax.swing.UIManager.LookAndFeelInfo info : javax.swing.UIManager.getInstalledLookAndFeels()) {
                if ("Nimbus".equals(info.getName())) {
                    javax.swing.UIManager.setLookAndFeel(info.getClassName());
                    break;
                }
            }
        } catch (ClassNotFoundException ex) {
            java.util.logging.Logger.getLogger(SimpleWorkerUI.class.getName()).log(java.util.logging.Level.SEVERE, null, ex);
        } catch (InstantiationException ex) {
            java.util.logging.Logger.getLogger(SimpleWorkerUI.class.getName()).log(java.util.logging.Level.SEVERE, null, ex);
        } catch (IllegalAccessException ex) {
            java.util.logging.Logger.getLogger(SimpleWorkerUI.class.getName()).log(java.util.logging.Level.SEVERE, null, ex);
        } catch (javax.swing.UnsupportedLookAndFeelException ex) {
            java.util.logging.Logger.getLogger(SimpleWorkerUI.class.getName()).log(java.util.logging.Level.SEVERE, null, ex);
        }
        //</editor-fold>

        /* Create and display the form */
        java.awt.EventQueue.invokeLater(new Runnable() {
            public void run() {
                new SimpleWorkerUI().setVisible(true);
            }
        });
    }
    // Variables declaration - do not modify
    private javax.swing.JLabel jLabel1;
    private javax.swing.JLabel jLabel2;
    private javax.swing.JPanel jPanel1;
    // End of variables declaration

    @Override
    public void propertyChange(PropertyChangeEvent evt) {
        if("status".equalsIgnoreCase(evt.getPropertyName())){
            String currentStatus = (String) evt.getNewValue();
            jLabel2.setText(currentStatus);
        }
    }
}

class Worker extends SwingWorker<String, String>{

    private static final int FINAL_VALUE = 1000;

    @Override
    protected String doInBackground() throws Exception {
        int counter = 0;

        while(counter < FINAL_VALUE){
            firePropertyChange("status", "", "value is: "+counter);
            try{
                Thread.sleep(100);
            }catch(InterruptedException ixe){
            }
            counter++;
        }
        return null;
    }
}

Sujay 13.10.2012

arrow_upward
1
arrow_downward

Вы не должны выполнять какие-либо длительные операции в потоке диспетчеризации событий. Вы выполняете операцию, которая занимает 3 секунды. В течение этих 3 секунд вы монополизируете EDT, и никакие другие обновления графического интерфейса не могут произойти.

JimN 12.10.2012

arrow_upward
0
arrow_downward

Вы начали с добрыми намерениями...

public class LoginPrepThread extends Thread {

    private final UIMain parent;

    public LoginPrepThread(UIMain w){
        parent = w;
    }

    public void exec(){
        EventQueue.invokeLater(this);
    }

    public void run(){
        try{
            //SqlHelper sql = SqlHelper.instance;
            sleep(500);
            parent.getLoadingLable().setText("Fetching preferences...");
            parent.getMainFrame().revalidate();
            sleep(500);
            parent.getLoadingLable().setText("Scanning workbench...");
            parent.getMainFrame().revalidate();
            sleep(500);
            parent.getLoadingLable().setText("Updating permissions...");
            parent.getMainFrame().revalidate();
            sleep(500);
            parent.getLoadingLable().setText("Finished...Please wait");
            parent.getMainFrame().revalidate();
            sleep(1000);
            parent.getLayout().show(parent.getMainFrame().getContentPane(), "view.main");
        }catch(Exception e){
        }
    }
}

По сути, когда вы вызываете exec, вы фактически отправляете запрос в EDT для вызова метода run, который затем выполняется в EDT, вызывая блокировку.

Кроме того, в вашем исходном примере бессмысленно расширяться от Thread, поскольку вы никогда его не запускаете.

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

Вы правы (в своем ответе), SwingWorker - гораздо более простое решение.

MadProgrammer 13.10.2012

Рабочий поток блокирует перерисовку графического интерфейса

Ответы (4)

Похожие вопросы