0

我最初试图在 Java Action Listener 中多次更新 JFrame 和 JPanel，但两者都只会在 Action Listener 完成所有任务时更新。这是我原来的问题的链接（Refreshing a JFrame while in an Action Listener）。

我在对该问题的反馈中被告知 Swing Worker 应该解决我的问题。但是，当我实现 Swing Worker（如下所示）时，没有任何改变。只有当 Action Listener 完成所有任务时，JFrame 和 JPanel 才会更新。我的问题是，我是否遗漏了以下内容？如果没有，我如何在动作侦听器中实现这一点以及时正确更新框架和面板？

@Override
protected Integer doInBackground() throws Exception{
    //Downloads and unzips the first video.  
    if(cameraBoolean==true)
        panel.add(this.downloadRecording(camera, recording));
    else
        panel.add(new JLabel("Could not contact camera "+camera.getName()));

    panel.repaint();
    jframe.repaint();
    return 1;
}

private JLabel downloadRecording(Camera camera, Recording recording){
    //does a bunch of calculations and returns a jLabel, and works correctly
}

protected void done(){
    try{
        Date currentTime = new Timestamp(Calendar.getInstance().getTime().getTime());
        JOptionPane.showMessageDialog(jframe, "Camera "+camera.getName()+" finished downloading at "+currentTime.getTime());
    }catch (Exception e){
    e.printStackTrace();
    }
}
4

3 回答 3

6

您对如何工作有误解SwingWorker。此类旨在提供一种在执行繁重任务时更新 GUI 的方法。所有这一切都是因为Swing组件更新发生在事件调度线程（又名 EDT）中，它是一个特定的线程。

例如，如果您单击一个按钮并在 EDT 中执行一项耗时的任务，则该线程将阻塞，直到该任务完成。因此，您会看到您的 GUI 被冻结。

记住这一点，doInBackground()方法在另一个不是 EDT 的不同线程中运行，这是可以的。所以不要Swing在那里调用任何方法：

protected Integer doInBackground() throws Exception{
    //Downloads and unzips the first video.  
    if(cameraBoolean==true) // just use if(cameraBoolean), since this is a boolean
        panel.add(this.downloadRecording(camera, recording)); // NO!
    else
        panel.add(new JLabel("Could not contact camera "+camera.getName())); //NO!

    panel.repaint(); //NO, never!
    jframe.repaint();//NO, never!
    return 1;
}

在执行之前添加 aJLabel并使用publish()和process()方法更新其文本：panel SwingWorker

JPanel panel = new JPanel();
final JLabel progressLabel = new JLabel("Some text before executing SwingWorker");
panel.add(progressLabel);

SwingWorker<Integer, String> worker = new SwingWorker<Integer, String>() {    
    @Override
    protected Integer doInBackground() throws Exception {
        if(cameraBoolean){
            pubish("Starting long process...");
            //Some processing here
            publish("Intermediate result to be published #1");
            //Some other processing stuff
            publish("Intermediate result to be published #2");
            //And so on...
            return 0;
        } else {
            publish("Could not contact camera "+camera.getName());
            return -1;
        }
    }

    @Override
    protected void process(List<String> chunks) {
        for(String string : chunks){
            progressLabel.setText(string);
        }
    }

    @Override
    protected void done() {
        progressLabel.setText("Finished!!!");
    }
};

worker.execute();

process()和done()方法都发生在 EDT 中，因此在那里进行 GUI 更新是安全的。看看这个优秀的例子：Swing Worker Example了解更多细节。

于 2013-11-05T21:27:44.960 回答
0

也许是因为您在同步调用this.downloadRecording(camera, recording)完成时重新绘制了面板/框架？

尝试只将此调用放入doInBackground()方法中，因为（所以我猜）这是需要很长时间的调用，并且在所有这些时间内 JFrame 都不会刷新。

于 2013-11-05T19:00:53.213 回答
0

您无法通过以下方式更新 UI：

 panel.repaint();
 jframe.repaint();

在您的doInBackground方法中，您必须调用publish(V... chunks)方法，即Sends data chunks to the process(java.util.List<V>) method.（根据文档），而不是在方法中process(List<V> chunks)您可以更新您的 UI（根据文档处理方法 - Receives data chunks from the publish method asynchronously on the Event Dispatch Thread.）。SwingWorker 文档。

因此，覆盖process更新方法，并调用publish方法。

您也可以将Executors用于后台进程。在这种情况下，您的 UI 将在 EDT 中运行，而您的后台进程将在另一个线程中运行。例子：

Executors.newSingleThreadExecutor().execute(new Runnable() {

        @Override
        public void run() {
            // run background process

        }
    });

编辑：SwingWorker 的好例子

于 2013-11-05T19:30:53.273 回答