Swing Worker中的优美异常处理

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（）方法是一个好的线索吗？

编辑：

这样吗：

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

        }
    });

}

调用get in done方法会导致两个try-catch块，因为计算部分会引发异常，因此我认为这最终会更干净。

Answer 1

一种选择是用于SwingUtilities.invokeLater(...)将操作发布到EDT

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

正如您指出的那样，SwingWorker它能够报告中间结果，但是您需要覆盖process(...)，这是在调用时调用的publish(...)。

无论如何，为什么不只设置一个标志（如果发生异常），并且如果设置了该标志，则显示该对话框，done()因为该对话框已在EDT？中安全地执行了？