スイングワーカーでの優雅な例外処理

Question

私はSwing Workerクラスを通してアプリケーションでスレッドを使用しています。それはうまく動作しますが、try-catchブロックにエラーメッセージダイアログを表示するのは悪い思いです。潜在的にアプリケーションをブロックできますか？これは現在の外観です：

SwingWorker<Void, Void> worker = new SwingWorker<Void, Void>() { 

    // Executed in background thread 
    public Void doInBackground() { 
     try { 
      DoFancyStuff(); 
     } catch (Exception e) { 

      e.printStackTrace(); 

      String msg = String.format("Unexpected problem: %s", e 
        .toString()); 

      //TODO: executed in background thread and should be executed in EDT? 
      JOptionPane.showMessageDialog(Utils.getActiveFrame(), 
        msg, "Error", JOptionPane.ERROR_MESSAGE, 
        errorIcon); 

     }//END: try-catch 

     return null; 
    } 

    // Executed in event dispatch thread 
    public void done() { 
     System.out.println("Done"); 
    } 
}; 

Swing Workerフレームワークを使用して安全に行うことはできますか？ publish（）メソッドのオーバーライドはここで良いリードですか？ EDIT

：

はこのようにそれをやった：

} catch (final Exception e) { 

    SwingUtilities.invokeLater(new Runnable() { 

     public void run() { 

      e.printStackTrace(); 

      String msg = String.format(
        "Unexpected problem: %s", e.toString()); 

      JOptionPane.showMessageDialog(Utils 
        .getActiveFrame(), msg, "Error", 
        JOptionPane.ERROR_MESSAGE, errorIcon); 

     } 
    }); 

} 

計算部分が例外をスローするように、2つのtry-catchブロックにつながるdoneメソッドで取得呼び出すので、私はこれがあると思います最後にクリーナー。

Answer 1

1つのオプションは、EDT

SwingUtilities.invokeLater(new Runnable(){ 
    @Override 
    public void run(){ 
     JOptionPane.showMessageDialog(
      Utils.getActiveFrame(), 
      msg, 
      "Error", 
      JOptionPane.ERROR_MESSAGE, 
      errorIcon); 
    } 
}); 

上のアクションを投稿するSwingUtilities.invokeLater(...)を使用することですそして、あなたが述べたように、SwingWorkerは、中間結果を報告することが可能であるが、あなたはときに呼び出される、process(...)をオーバーライドする必要がありますpublish(...)が呼び出されます。

例外が発生した場合にフラグを設定するだけで、そのフラグが設定されている場合、EDTで安全に実行されるので、done()でダイアログを表示してください。

Answer 2

あなたは正しいですが、イベントディスパッチスレッド以外の場所でGUIを変更しないSwingの基本ルールに違反しています。

私の場合は、GUIがエラーメッセージを表示するためにリッスンするイベントをスローします。または、SwingWorkerの呼び出しをtry catchでラップし、そこでダイアログを表示することができます。

Answer 3

次のようにそれを行うための正しい方法は次のとおりです。

SwingWorker<Void, Void> worker = new SwingWorker<Void, Void>() { 
    // Executed in background thread 
    protected Void doInBackground() throws Exception { 
     DoFancyStuff(); 
     return null; 
    } 

    // Executed in EDT 
    protected void done() { 
     try { 
      System.out.println("Done"); 
      get(); 
     } catch (ExecutionException e) { 
      e.getCause().printStackTrace(); 
      String msg = String.format("Unexpected problem: %s", 
          e.getCause().toString()); 
      JOptionPane.showMessageDialog(Utils.getActiveFrame(), 
       msg, "Error", JOptionPane.ERROR_MESSAGE, errorIcon); 
     } catch (InterruptedException e) { 
      // Process e here 
     } 
    } 
} 

あなたがバックグラウンドスレッドで例外をキャッチするのではなく、彼らがSwingWorkerの自体に通過させ、その後、あなたがそれらを得ることができますしようとするべきではありませんdone()メソッドを呼び出してget()を呼び出すと、通常doInBackground()（状況に応じてVoid）という結果が返されます。バックグラウンドスレッドで例外がスローされた場合はの中にラップされたget()がスローされます。

また、無効にされたSwingWorkerのメソッドはprotectedであり、それらを作成する必要はありません。public