java - ワーカースレッドが GUI の再描画をブロックする

Question

アプリケーションのログイン画面を作ろうとしています。ログイン中、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();

とりあえずダミーのイベントをいくつか入れたのですが、ステータスラベルが変わらない...何か提案はありますか?

score 4 · Accepted Answer

シンプルなスイングワーカーがそれを解決しました。私はグーグルで十分ではなかったと思います

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));
    }

}

score 3 · Accepted Answer

理想的には、SwingWorker を実装して、GUI 以外の重いタスクをすべて実行します。イベント駆動型プログラミングの利点を最大限に活用する必要があります。

あなたがする必要があるのは次のようなものです：

GUI クラスに 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;
    }
}

score 1 · Accepted Answer

イベントディスパッチスレッドで実行時間の長い操作を実行しないでください。完了までに 3 秒かかる操作を実行しています。この 3 秒間は EDT を独占しており、他の GUI 更新は発生しません。

score 0 · Accepted Answer

あなたは善意で始めました...

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は）、はるかに簡単な解決策です。

java - ワーカー スレッドが GUI の再描画をブロックする

4 に答える 4

Related

Reference

java - ワーカースレッドが GUI の再描画をブロックする