Блог

Привет, я пытался узнать, как использовать SwingWorker для обновления графического интерфейса приложений swing.

Мой вопрос заключается в следующем: если у меня есть 2 swingworkers, которые могут выполняться одновременно (каждый запускается с другой кнопки), как убедиться, что их метод done вызывается в EDT (для обновления графического интерфейса пользователя) в том же порядке, в котором они были запущены?

Мои наблюдения и идеи: я попытался синхронизировать их методы doInBackground() с блокировкой, чтобы убедиться, что один должен дождаться, пока другой завершит выполнение своей задачи, прежде чем продолжить. Проблема в том, что я смог (всего несколько раз) увидеть, как swingworker_1 завершает doInBackground() (пока 2-й рабочий ожидает в синхронизированном блоке), но метод done swingworker_2 смог быть вызван EDT до метода done swingworker_1, хотя swingworker_2 закончил после. Я думал об использовании publish непосредственно перед возвратом doInBackground(), это, кажется, всегда работает, но может быть плохой практикой, поскольку, похоже, он используется для промежуточных результатов, а не для конечных результатов? Другой возможностью было бы использовать SwingUtilities.invokeLater внутри doInBackground() для обновления графического интерфейса перед возвратом, опять же, я боюсь, что это может быть и плохой практикой. Заранее спасибо.

Тест, который я использовал, нажав первую кнопку, а затем сразу вторую:

     //Code that runs in the first button
    SwingWorker<Void, Void> swingworker = new SwingWorker<Void, Void>() {

        @Override
        protected Void doInBackground() throws Exception {
            synchronized (lock) {
                System.out.println("Thread 1");

                Thread.sleep(1000);
                return null;
            }

        }

        @Override
        protected void done() {

            System.out.println("Done with thread 1 ");
            //update GUI    
        }

    };
    swingworker.execute();



    //Code that runs in the second button
    SwingWorker<Void, Void> swingworker = new SwingWorker<Void, Void>() {

        @Override
        protected Void doInBackground() throws Exception {
            synchronized (lock) {
                System.out.println("Thread 2");

                return null;
            }

        }

        @Override
        protected void done() {
            System.out.println("Done with thread 2 ");
            //update GUI
        }

    };
    swingworker.execute();
  

ОБНОВЛЕНИЕ: просто чтобы уточнить, меня не интересует, кто запускается первым (swingworker_1 или swingworker_2), это зависит только от того, какая кнопка будет нажата первой. Что я хочу гарантировать, так это то, что если рабочий из кнопки был первым для выполнения (и, следовательно, завершается, поскольку рабочие синхронизируются в методе doInBackground()), то он также должен быть первым в очереди для обновления графического интерфейса пользователя в EDT. Я обнаружил, что это не обязательно происходит, даже если рабочие синхронизированы иногда рабочему, который запускает задачу позже, все равно удается обновить графический интерфейс перед первым.

Если вы согласны с тем, что рабочие выполняются последовательно, что и произойдет, если вы синхронизируете блокировку в соответствии с вашим образцом кода, и вы не используете какие-либо другие функции SwingWorker , кроме doInBackground() and done() , вам может быть проще использовать однопоточный исполнитель и SwingUtilities.invokeLater(...) представить результаты в графическом интерфейсе.

Таким образом, вам гарантируется, что вещи, отправленные исполнителю, будут выполняться в том порядке, в котором они были отправлены.

Если это работает для вас, вы можете попробовать что-то вроде этого:

 final ExecutorService executor = Executors.newSingleThreadExecutor();

// Code that runs in the first button
Runnable worker1 = new Runnable() {
  @Override
  public void run() {
    System.out.println("Thread 1");
    try {
      Thread.sleep(1000);
    } catch (InterruptedException e) {
      e.printStackTrace(System.err);
    }
    SwingUtilities.invokeLater(this::updateGUI);
  }

  private void updateGUI() {
    System.out.println("Done with thread 1");
    //update GUI
  }
};
executor.execute(worker1);

// Code that runs in the second button
Runnable worker2 = new Runnable() {
  @Override
  public void run() {
    System.out.println("Thread 2");
    SwingUtilities.invokeLater(this::updateGUI);
  }

  private void updateGUI() {
    System.out.println("Done with thread 2");
    //update GUI
  }
};
executor.execute(worker2);