在 SwingWorker 中运行 ExecutorService 是一种好习惯吗？

Question

考虑以下代码：

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

        @Override
        protected Void doInBackground() throws Exception {
            ExecutorService executor = Executors.newFixedThreadPool(5);
            try {
                for (int j = 0; j < 5; j++) {
                    Callable<Object> worker = new MyCallableImpl();
                    Future<Object> future = executor.submit(worker); 
                    array[j] = future.get();
                }
            } catch (InterruptedException e) {
                // some code here
            } catch (ExecutionException e) {
                // some code here
            }
                // some code here
            executor.shutdown();
            return null;
        }

    };
    sworker.execute();

正如我在标题中所说：这是在 SwingWorker 的 doInBackground() 方法中调用 ExecutorService 的好习惯吗？它适用于我（JDK1.7），GUI 没有被阻塞，并且来自 Executor 池的多个线程在后台运行，但我仍然有一些疑问......

Answer 1

进一步mre的回应。这没有任何意义，因为您的执行实际上是单线程的。 doInBackground 将提交给执行者并等待该单个任务完成然后提交另一个。

您应该以相同的方式提交，但将返回的Futures 存储在某种列表中，然后在提交所有任务后获取每个列表。

我不介意 doInBackground 像 mre 那样异步提交这些作业。如果您尝试提交多个任务并且在任何给定时间只提交了 N 个任务，那么您绝对不应该通过SwingWorker.doInBackground 执行此操作。我认为使用ExectorService + SwingUtilities.invokeLater 是更好的方法。

为了澄清任何混淆，invokeLater 应该只在 ExecutorService 中的任务完成并且它需要做的只是更新 UI 组件时在这里使用。

编辑：解决您的评论的示例

protected Void doInBackground() throws Exception {
    ExecutorService executor = Executors.newFixedThreadPool(5);
    List<Future> futures = ...;
    try {
        for (int j = 0; j < 5; j++) {
            Callable<Object> worker = new MyCallableImpl();
            futures.add(executor.submit(new Callable<Object>(){
                 public Object call(){
                    //expensive time consuming operation
                    final String result = ...;//result from consuming operation
                    SwingUtilities.invokeLater(new Runnable(){
                        public void run(){
                             jLabel.setText(result);
                        }
                    });
                    return new Object();
                 }
            ));
        }
        for(Future<Object> f :futures)f.get();
        executor.shutdown();
    return null;
}

注意invokeLater 是如何进行简单更新的？这不会导致您的 EDT 冻结。

Answer 2

• 可以从 Executor 执行 SwingWorkers 实例

• 必须接受 Executor 不关心 SwingWorkers 生命周期，反之亦然

• 必须为 SwingWorker 实现 PropertyChangeListener

• 例如here

Answer 3

上面的代码对我来说没有多大意义。

如果此处的目标是确保在执行长时间运行的任务时 GUI 保持响应，则无需使用 ExecutorService，因为 SwingWorker 已经提供了该机制。