Выполняет ли ExecutorService внутри SwingWorker хорошую практику?

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

Как я уже говорил в названии: это хорошая практика, чтобы вызвать ExecutorService внутри doInBackground метода() из SwingWorker? Это работает для меня (JDK1.7), GUI не заблокирован, и несколько потоков из пула Исполнителей работают в фоновом режиме, но все же у меня есть некоторые сомнения ...

Answer 1

Вышеприведенный код для меня не имеет смысла.

Если целью является обеспечение того, чтобы GUI оставался отзывчивым при выполнении долговременной задачи, тогда нет необходимости использовать ExecutorService, поскольку SwingWorker уже предоставляет этот механизм.

Answer 2

Для дальнейшего ответа от mre. Это не имеет смысла, потому что ваше исполнение на самом деле однопоточное. doInBackground отправит исполнителю и дождитесь завершения этой отдельной задачи, а затем отправьте другое.

Вы должны отправить таким же образом, но сохраните возвращенный Future s в списке какого-либо рода, затем перейдите по каждому из них после отправки всех заданий.

Мне не так нравится doInBackground, чтобы отправить эти задания асинхронно, как это делает mre. Если вы пытаетесь отправить несколько заданий и только в течение определенного времени подали только N, вам определенно не следует делать это через SwingWorker.doInBackground. Использование ExectorService + SwingUtilities.invokeLater Я думаю, что это лучший способ.

И как раз для того, чтобы разъяснить любую путаницу, invokeLater следует использовать здесь только тогда, когда задача в ExecutorService завершена, и все, что нужно сделать, это обновить компонент пользовательского интерфейса.

Edit: Пример решения Ваш комментарий

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 3

• может SwingWorkers выполнение экземпляра из Исполнителю

• должны принимать, что Исполнитель не заботится о SwingWorkers жизненного цикла, и наоборот

• должны реализовать PropertyChangeListener для SwingWorker

• exmple here