2

問題: すべてのワーカー スレッドが停止したら、アプリケーションのメイン ウィンドウを閉じたいです。簡単だ。しかし、ウィンドウを閉じることによってメッセージが送信される前に、SwingWorker によって送信されたメッセージが処理されることも確認したいと思います。この (そうではない) 小さなサンプル ソース ファイルがここにあります。

/*****************************************************************************
    TestAsyncEvents.java

This example shows that messages from SwingWorker threads get processed
*AFTER* the WINDOW_CLOSED event even if they had been generated
*BEFORE* them.
The question is: HOW DO I MAKE IT RIGHT?
I know the answer, but it doesn't satisfy me. ;)
*****************************************************************************/
import  java.util.List;
import  java.awt.Dimension;
import  java.awt.Toolkit;
import  java.awt.event.WindowAdapter;
import  java.awt.event.WindowEvent;
import  javax.swing.JFrame;
import  javax.swing.SwingUtilities;
import  javax.swing.SwingWorker;

//////////////////////////////////////////////////////////////////////////////
//
public class TestAsyncEvents extends JFrame
{

/*
 *  This class is for closing main window
 */
private class Closer extends WindowAdapter
{
    @Override public void windowClosing( WindowEvent ev ) { handleWindowClosing(); }
    @Override public void windowClosed( WindowEvent ev ) { handleWindowClosed(); }
}

/*
 *  This class represents worker that gets asked to stop
 *  and then reports when it actually stops
 */
private class Ticker extends SwingWorker<Boolean,String>
{
    private boolean stop_flag = false;
    private int     counter = 0;
    private boolean result;
    Ticker() { super(); execute(); }
    @Override protected Boolean doInBackground()
    {
        // body, executed in worker thread
        try {
            while( !stop_flag )
            {
                Thread.sleep(2000);
                publish(String.format("Tick #%d",++counter));
            }
            return result=true;
        } catch( Exception ex ) {
            return result=false;
        }
    }
    @Override protected void process( List<String> chunks )
    {
        // reports progress, executed in gui thread
        for( String chunk: chunks )
            System.out.println(String.format("Chunk processed: %s",chunk));
    }
    @Override protected void done()
    {
        // reports result, executed in gui thread
        System.out.println(String.format("Thread is %s.",isCancelled()?"cancelled":"stopped normally"));
        System.out.println(String.format("Result is %s.",Boolean.toString(result)));
        //postClosing(); // IT IS THE SOLUTION FOR MY PROBLEM! BUT... IT ISN'T GOOD ONE!
    }
    public void askToStop() { stop_flag = true; }
}

/****************************************************************************/
/*  FIELDS                                                                  */
/****************************************************************************/
private static  TestAsyncEvents     self;
private Ticker      worker_object = null;

/****************************************************************************/
/*  CONSTRUCTOR                                                             */
/****************************************************************************/
TestAsyncEvents()
{
    super("Testing Async Events");
    addWindowListener(new Closer());
    setMinimumSize(new Dimension(512,384));
    setVisible(true);
    worker_object = new Ticker();
}

/****************************************************************************/
/*  INNER METHODS                                                           */
/****************************************************************************/
/*
 *  Waiting for worker to finish
 */
private void doStopping()
{
    worker_object.askToStop();
    while( !worker_object.isDone() );
}
private boolean stopInSeparateThread()
{
    try {
        Thread closer = new Thread(new Runnable(){public void run(){doStopping();}});
        closer.start();
        closer.join();
        return true;
    } catch( Exception ex ) {
        return false;
    }
}
private boolean stopHere()
{
    doStopping();
    return true;
}
private boolean stopWorker()
{
    //return stopInSeparateThread();
    return stopHere();
}
private boolean canClose()
{
    return worker_object.isDone();
}
/*
 *  Posting WM_CLOSE events
 */
private void doPostCloseEvent()
{
    Toolkit.getDefaultToolkit().getSystemEventQueue().postEvent(new WindowEvent(this,WindowEvent.WINDOW_CLOSING));
}
private void postInSeparateThread()
{
    SwingUtilities.invokeLater(new Runnable(){public void run(){doPostCloseEvent();}});
}
private void postHere()
{
    doPostCloseEvent();
}
private void postClosing()
{
    //postInSeparateThread();
    postHere();
}
/*
 *  Methods for Closer class
 */
private void handleWindowClosing()
{
    System.out.println("Closing main window...");
    if( canClose() )
    {
        System.out.println("Can close! Disposing...");
        dispose();
    } else {
        System.out.println("Can't close! Now we'll allow it by stopping worker thread...");
        boolean res = stopWorker();
        System.out.println(String.format("Stopping worker thread went %s.",res?"okay":"wrong"));
        postClosing(); // HERE I SIGNAL THE MAIN WINDOW TO CLOSE
    }
}
private void handleWindowClosed()
{
    System.out.println("Main window is closed!");
}

/****************************************************************************/
/*  ENTRY POINT                                                             */
/****************************************************************************/
public static void main( final String[] args )
{
    SwingUtilities.invokeLater(new Runnable(){public void run(){self=new TestAsyncEvents();}});
    System.out.println("All systems are go!");
}

}
//////////////////////////////////////////////////////////////////////////////

その出力は次のとおりです。

F:\C\Java\Test-Frame-Events>java TestAsyncEvents
All systems are go!
Chunk processed: Tick #1
Closing main window...
Can't close! Now we'll allow it by stopping worker thread...
Stopping worker thread went okay.
Closing main window...
Can close! Disposing...
Main window is closed!
Chunk processed: Tick #2
Thread is stopped normally.
Result is true.

私が欲しいものはここにあります:

F:\C\Java\Test-Frame-Events>java TestAsyncEvents
All systems are go!
Chunk processed: Tick #1
Closing main window...
Can't close! Now we'll allow it by stopping worker thread...
Stopping worker thread went okay.
Chunk processed: Tick #2
Thread is stopped normally.
Result is true.
Closing main window...
Can close! Disposing...
Main window is closed!

SwingWorker からのイベントは、ウィンドウ メッセージなどを処理するイベント キューではなく、まったく別のイベント キューで処理されるようです。WINDOW_CLOSING イベントを投稿する前に、ワーカー スレッドが停止してすべてのメッセージを投稿するのを意図的に待ちます。しかし、それは役に立ちません。SwingWorker からのメッセージは、WINDOW_CLOSING および WINDOW_CLOSED イベントの後で処理されます。これにより、多くの小さな不便が生じます。特に、WINDOW_CLOSED ハンドラーですべてのロギングを閉じると、それらがプログラムで実行される最後のオペレーターになることを期待して、ワーカー スレッドからのすべてのメッセージが時間と空間で失われます。

私は自分の問題の解決策を知っています。68 行目と 161 行目のコメントを外す必要があります。ただし、複数の SwingWorker スレッドがある場合は、すべてのワーカーを監視する別のスレッドを生成し、すべてのワーカーが停止したときに終了するようにメイン ウィンドウに通知する必要があることを意味します。そして、それはきちんとしていません、IMHO。では、Java の達人、これを修正する方法を教えてください。

4

1 に答える 1

2

考えられる解決策の 1 つ: PropertyChangeListener の使用を検討してください。

private void handleWindowClosing() {
  System.out.println("Closing main window...");
  if (canClose()) {
     System.out.println("Can close! Disposing...");
     dispose();
  } else {
     System.out
           .println("Can't close! Now we'll allow it by stopping worker thread...");

     worker_object.addPropertyChangeListener(new PropertyChangeListener() {

        @Override
        public void propertyChange(PropertyChangeEvent pcEvt) {
           if (SwingWorker.StateValue.DONE.equals(pcEvt.getNewValue())) {
              postClosing();
           }
        }
     });


     boolean res = stopWorker();
     System.out.println(String.format("Stopping worker thread went %s.",
           res ? "okay" : "wrong")); // !!
     // postClosing(); // HERE I SIGNAL THE MAIN WINDOW TO CLOSE
  }
}

複数のワーカーがある場合は、CountDownLatchまたはCyclicBarrierを使用できます。

于 2012-09-22T15:40:06.147 に答える