如何使我的SwingWorker示例正常工作？

Question

我已经制作了自己的SwingWorker示例来熟悉它的工作方式。我想做的是：当单击按钮时，我希望进度条出现，直到任务完成，然后简单地删除进度条并在对话框中添加一个字符串。

当单击按钮时，进度条出现，但永远不会消失。（10秒后从未删除进度条，也从未将标签放置在上面）

这是一个SSCCE：

package swingtesting;

import java.awt.event.ActionEvent; 
import java.awt.event.ActionListener;
import javax.swing.JButton;
import javax.swing.JDialog;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JProgressBar;
import javax.swing.SwingWorker;

public class SwingTesting {

    /**
     * Creates a frame that will hold a simple button to make use of SwingWorker
     */
     public static void main(String[] args) {
         // TODO code application logic here
         JFrame frame = new JFrame();
         JButton button = new JButton();

         button.addActionListener(new ActionListener() {
            public void actionPerformed(ActionEvent e) {
                new GuiWorker().execute();
            }
         });
         button.setText("Test Me");
         frame.getContentPane().add(button);
         frame.pack();
         frame.setVisible(true);
    }
}

class GuiWorker extends SwingWorker<Integer, Integer> {

    /*
     * This should just create a frame that will hold a progress bar until the
     * work is done. Once done, it should remove the progress bar from the dialog
     * and add a label saying the task complete.
     */

    private JFrame frame = new JFrame();
    private JDialog dialog = new JDialog(frame, "Swingworker test", true);
    private JProgressBar progressBar = new JProgressBar();


    public GuiWorker() {
        progressBar.setString("Waiting on time");
        progressBar.setStringPainted(true);
        progressBar.setIndeterminate(true);
        dialog.getContentPane().add(progressBar);
        dialog.pack();
        dialog.setVisible(true);
    }

    @Override
    protected Integer doInBackground() throws Exception {
        Thread.sleep(10000);
        return 0;
    }

    @Override
    protected void done() {
        JLabel label = new JLabel("Task Complete");
        dialog.getContentPane().remove(progressBar);
        dialog.getContentPane().add(label);
    }

}

- WilliamShatner

---

你可以看一下这个答案，并将其与你的代码进行比较。Swing教程中还包含一个示例（链接在JProgressBar类的javadoc中可用）。 - Robin

Answer 1

这里是您的代码的更新版本，可以正常运行

import java.awt.EventQueue;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import javax.swing.JButton;
import javax.swing.JDialog;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JProgressBar;
import javax.swing.SwingWorker;

public class SwingTesting {

  public static void main(String[] args) {
    EventQueue.invokeLater( new Runnable() {
      @Override
      public void run() {
        JFrame frame = new JFrame();
        JButton button = new JButton();
        button.addActionListener(new ActionListener() {
          @Override
          public void actionPerformed(ActionEvent e) {
            new GuiWorker().execute();
          }
        });
        button.setText("Test Me");
        frame.getContentPane().add(button);
        frame.pack();
        frame.setVisible(true);
      }
    } );

  }
}

class GuiWorker extends SwingWorker<Integer, Integer> {

  /*
  * This should just create a frame that will hold a progress bar until the
  * work is done. Once done, it should remove the progress bar from the dialog
  * and add a label saying the task complete.
  */

  private JFrame frame = new JFrame();
  private JDialog dialog = new JDialog(frame, "Swingworker test", true);
  private JProgressBar progressBar = new JProgressBar();


  public GuiWorker() {
    progressBar.setString("Waiting on time");
    progressBar.setStringPainted(true);
    progressBar.setIndeterminate(true);
    dialog.getContentPane().add(progressBar);
    dialog.pack();
    dialog.setModal( false );
    dialog.setVisible(true);
  }

  @Override
  protected Integer doInBackground() throws Exception {
    System.out.println( "GuiWorker.doInBackground" );
    Thread.sleep(1000);
    return 0;
  }

  @Override
  protected void done() {
    System.out.println("done");
    JLabel label = new JLabel("Task Complete");
    dialog.getContentPane().remove(progressBar);
    dialog.getContentPane().add(label);
    dialog.getContentPane().validate();
  }

}

关键点是设置模态对话框可见会阻塞，直到对话框被处理。因此，将其设置为非模态可以解决问题，并在切换组件时对内容面板进行validate调用。我还调整了您的主方法以在EDT上运行，并添加了一些System.out调用。如果删除setModal(false)调用，则会发现这些语句直到关闭对话框后才会打印出来。

- Robin

---

谢谢你指出这个问题！我想我应该先看一下是否有setModal方法。我也注意到了主函数的变化，感谢你指出。我在这里读到了相关信息：http://java.sun.com/developer/technicalArticles/javase/swingworker/ 不过我还有一个问题，通过将Modal更改为false，它允许用户多次点击按钮。 - WilliamShatner

@WilliamShatner: 没有必要使对话框非模态。只需在启动SwingWorker后显示对话框即可。 这可以从调用类中完成，也可以从执行SwingWorker的类中完成，首先调用execute，然后显示对话框，或者可以从SwingWorker中完成，但如果是后者，则必须制作自己的伪执行方法来调用超级execute，然后显示对话框。 请注意，您无法覆盖execute()本身，因为它是最终的。 - Hovercraft Full Of Eels

@WilliamShatner 当你启动 SwingWorker 时，可以禁用该按钮，并在此后重新启用它。 - Robin

@Robin 我想我会坚持你的方法，但是我不确定除非按钮是全局的，否则我如何在其自身的actionPerformed中禁用该按钮？ - WilliamShatner

@WilliamShatner 两种解决方案：您可以从ActionEvent＃getSource（）中检索按钮，或者您可以将按钮设置为final。 - Robin

Answer 2

无需使对话框非模态。在启动SwingWorker后直接显示对话框即可。可以从调用类中执行此操作，通过首先调用execute，然后显示对话框，也可以从SwingWorker中执行此操作，但如果从后者执行，您必须自己创建一个伪execute方法，该方法调用super的execute，然后显示对话框。请注意，您无法覆盖execute()本身，因为它是final的。

例如...

import java.awt.CardLayout;
import java.awt.Window;
import java.awt.Dialog.ModalityType;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.beans.PropertyChangeEvent;
import java.beans.PropertyChangeListener;
import java.util.concurrent.ExecutionException;

import javax.swing.*;

@SuppressWarnings("serial")
public class SwingTesting2 {

   private static void createAndShowGui() {
      final JFrame frame = new JFrame("SwingTesting2");
      final JDialog dialog = new JDialog(frame, "Dialog",
            ModalityType.APPLICATION_MODAL);
      final DialogPanel dialogPanel = new DialogPanel();
      dialog.getContentPane().add(dialogPanel.getMainPanel());
      dialog.pack();
      dialog.setLocationRelativeTo(frame);

      JButton button = new JButton(new AbstractAction("Test Me") {

         @Override
         public void actionPerformed(ActionEvent actEvt) {
            final GuiWorker2 guiWorker = new GuiWorker2();
            guiWorker.addPropertyChangeListener(new PropertyChangeListener() {

               @Override
               public void propertyChange(PropertyChangeEvent pcEvt) {
                  if (pcEvt.getPropertyName().equals("state")) {
                     if (pcEvt.getNewValue() == SwingWorker.StateValue.DONE) {
                        try {
                           dialogPanel.done(guiWorker.get());
                        } catch (InterruptedException e) {
                           e.printStackTrace();
                        } catch (ExecutionException e) {
                           e.printStackTrace();
                        }
                     }
                  } else if (pcEvt.getPropertyName().equals("progress")) {
                     dialogPanel.setProgress((Integer)pcEvt.getNewValue());
                  }
               }
            });

            guiWorker.execute();
            dialogPanel.start();
            dialog.setVisible(true);
         }
      });

      frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
      frame.getContentPane().add(button);
      frame.pack();
      frame.setLocationByPlatform(true);
      frame.setVisible(true);
   }

   public static void main(String[] args) {
      SwingUtilities.invokeLater(new Runnable() {
         public void run() {
            createAndShowGui();
         }
      });
   }
}

class GuiWorker2 extends SwingWorker<Integer, Integer> {
   private static final int MAX_COUNT = 20;
   private static final long SLEEP_TIME = 100;
   private int count = 0;

   @Override
   protected Integer doInBackground() throws Exception {
      while (count < MAX_COUNT) {
         Thread.sleep(SLEEP_TIME);
         count++;
         setProgress((100 * count) / MAX_COUNT);
      }
      return count;
   }
}

@SuppressWarnings("serial")
class DialogPanel {
   public static final String PROGRESS_BAR = "Progress Bar";
   public static final String DONE = "Done";
   private static final int TIMER_DELAY = 2000;
   private CardLayout cardLayout = new CardLayout();
   private JPanel mainPanel = new JPanel(cardLayout);
   private JLabel doneLabel = new JLabel("Done", JLabel.CENTER);
   private JProgressBar progressBar = new JProgressBar();

   public DialogPanel() {
      progressBar.setString("Waiting on time");
      progressBar.setStringPainted(true);
      progressBar.setIndeterminate(false);

      mainPanel.add(progressBar, PROGRESS_BAR);
      mainPanel.add(doneLabel, DONE);
   }

   public void setProgress(Integer newValue) {
      progressBar.setValue(newValue);
   }

   public void start() {
      cardLayout.show(mainPanel, PROGRESS_BAR);
      progressBar.setValue(0);
   }

   public void done(int countValue) {
      doneLabel.setText(DONE + ". Count: " + countValue);
      cardLayout.show(mainPanel, DONE);
      new Timer(TIMER_DELAY, new ActionListener() {

         @Override
         public void actionPerformed(ActionEvent e) {
            Window win = SwingUtilities.getWindowAncestor(mainPanel);
            win.dispose();
         }
      }){{setRepeats(false);}}.start();
   }

   public JPanel getMainPanel() {
      return mainPanel;
   }

}

- Hovercraft Full Of Eels

---

@Hovercraft 的例子很好。我测试了一下，效果不错。虽然我不太熟悉代码的全部功能，但我已经尝试进行了调整。虽然我不是很喜欢在图片中添加另一个类，但我很高兴你发了这篇文章。我点了个赞 :) - WilliamShatner

@William：感谢您的评论。祝您的项目好运！ - Hovercraft Full Of Eels

@HovercraftFullOfEels，你在我的每篇帖子中都非常有帮助，我真希望能够给予更多的回报，而不仅仅是感谢。我还在学习中，希望能够回来并完全理解上面的所有代码。我希望你能在未来继续提供帮助，因为我知道我还会有很多问题。再次感谢！ - WilliamShatner

@HovercraftFullOfEels 不知道你是否在意，但我想说一下，你的 propertyChangeListener 对我刚开始的一个新项目来说是个很好的例子 :) 再次感谢！ - WilliamShatner