Swing 线程安全样板

Question

为了简单起见，想象一个下载​​文件的应用程序。有一个简单的 GUI，带有一个显示进度的标签。为了避免 EDT 违规，像每个合法公民一样，我在一个线程（主线程）中下载文件，并在另一个线程（EDT）中更新 GUI。所以，这里是相关的伪代码块：

class Downloader {
    download() {
        progress.startDownload();
        while(hasMoreChunks()) {
            downloadChunk();
            progress.downloadedBytes(n);
        }
        progress.finishDownload();
    }
}

class ProgressDisplay extends JPanel {
    JLabel label;
    startDownload() {
        SwingUtilities.invokeLater(new Runnable() {
            public void run() {
                label.setText("Download started");
            }
        });
    }
    downloadedBytes(int n) {
        SwingUtilities.invokeLater(new Runnable() {
            public void run() {
                label.setText("Downloaded bytes: " + n);
            }
        });
    }
    finishDownload() {
        SwingUtilities.invokeLater(new Runnable() {
            public void run() {
                label.setText("Download finished");
            }
        });
    }
}

我喜欢 Java 不支持闭包这一事实，而且代码对我来说非常清晰。抛开笑话不谈，我想知道……我做错了吗？是否有可能使用 SwingUtilities、每个方法中 Runnable 的匿名实现等消除所有这些丑陋的样板？

我的情况比这稍微复杂一些，但我试图不通过实现代理或类似的东西来过度设计。

Answer 1

有一个用于该任务的特殊类： SwingWorker< /代码>。它允许您在单独的线程中运行代码并在工作结束时更新 UI。

Answer 2

如果不在其他地方引入大量冗余代码，您无法做太多事情来避免样板代码。但是您可以通过一个小的抽象帮助器类和一些不寻常的格式使其变得更好一点。

public abstract static class SwingTask implements Runnable
{
    public void start()
    {
         SwingUtilities.invokeLater( this );
    }
}

startDownload() {
    new SwingTask() { public void run() {
        label.setText("Download started");
    } }.start();
}

代码格式的目的是强调这一行中的所有代码基本上都是无趣的样板代码，可以轻松安全地复制到其他需要的地方。我们已经使用这种格式样式有一段时间了，结果证明它非常有用。

Answer 3

我会使用类似的辅助方法。

 public void setLabelText(final JLabel label, final String text) {
    SwingUtilities.invokeLater(new Runnable() {
        public void run() {
            label.setText(text);
        }
    });
 }

Answer 4

几年前，我们在一个项目中使用了 Spin。

Answer 5

只是一个原始想法：

public class ThreadSafeJLabel extends JLabel {

    @Override
    public void setText(final String text) {
        if (SwingUtilities.isEventDispatchThread()) {
            super.setText(text);
        } else {
            SwingUtilities.invokeLater(new Runnable() {
                @Override
                public void run() {
                    setText(text);
                }
            });
        }
    }

}

还没有尝试过，但我想当从 EDT 调用 setText() 时，将使用 super ；但是当涉及其他线程时，重写的 (ThreadSafeJLabel.setText()) 将稍后调用，这次是在 EDT 内。