发布/处理使 SwingWorker 变慢答案

【问题标题】：publish/process makes the SwingWorker slow发布/处理使 SwingWorker 变慢
【发布时间】：2025-12-21 00:20:08
【问题描述】：

我尝试使用 SwingWorker 来更新进度条。对我来说没有什么新鲜事。我有点懒，所以我没有覆盖 process 方法。这里是一个简化的例子：

protected Void doInBackground() throws Exception {
    for (int i = 0; i < 10000; i++) {
        progressBar.setValue(i+1);            
    }
    return null;
}

在我的电脑上，将进度条更新到 100% 大约需要 6 秒。现在是星期五，所以我想让我们以一种不同的方式来做这件事，就像经常描述的那样。让我们使用 publish 和 process 方法来更新进度条：

@Override
protected Void doInBackground() throws Exception {
    for (int i = 0; i < maximum; i++) {            
        publish(i+1);
    }
    return null;
}

@Override
protected void process(List<Integer> chunks) {        
    progressBar.setValue(chunks.get(chunks.size()-1));
}

但是现在使用发布方法需要 21（而不是 6）秒才能将进度条更新到 100%。

为什么？

这里是整个代码（带有“开始”按钮和进度条的框架）：

public class Gui extends JFrame {

    public static void main(String[] args) {
        new Gui();
    }

    public Gui() {
        setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);

        final JProgressBar progressBar = new JProgressBar();
        progressBar.setStringPainted(true);

        JButton btnStart = new JButton("Start");
        btnStart.addActionListener(new ActionListener() {

            @Override
            public void actionPerformed(ActionEvent e) {
                new MySwingWorker(progressBar).execute();
            }
        });

        getContentPane().setLayout(new BorderLayout(3, 3));
        getContentPane().add(btnStart, BorderLayout.CENTER);
        getContentPane().add(progressBar, BorderLayout.SOUTH);
        pack();
        setVisible(true);
        setLocationRelativeTo(null);
    }

    private class MySwingWorker extends SwingWorker<Void, Integer> {

        private final int MAXIMUM = 100000000;
        private final JProgressBar progressBar;

        public MySwingWorker(JProgressBar progressBar) {
            this.progressBar = progressBar;
            this.progressBar.setMaximum(MAXIMUM);
            this.progressBar.setValue(0);
        }

        @Override
        protected Void doInBackground() throws Exception {
            for (int i = 0; i < MAXIMUM; i++) {
                //----------------------------------
                // TOGGLE BETWEEN THESE LINES:
                //progressBar.setValue(i+1);
                publish(i + 1);
                //-----------------------------------
            }
            return null;
        }

        @Override
        protected void process(List<Integer> chunks) {
            progressBar.setValue(chunks.get(chunks.size() - 1));
        }
    }
}

感谢您的建议...

【问题讨论】：

没仔细看 - 但您的第一个版本错误：您不得访问 doInBackground 中的 swing 组件
是的，我知道，但它可以正常工作，而且速度提高了 70%！ 6 秒而不是 21 秒。所以我的问题是：真正会发生什么？
违反 EDT 可能发生的所有邪恶事情 ;-) 难以检测，虚假（但肯定会在你最不想要的时候抛出），不可预测。所以干脆不要（而且大多数生产代码都应该防范它）。这里的问题似乎是您在不需要的情况下淹没了系统：在现实世界的场景中，您不会尝试以高于屏幕分辨率的频率更新进度条，对吧:-)
我将在我的生产系统上检查这一点。分析 500.000.000 条消息。时间差是多少。用户是否需要因为进度条而多等几分钟？我会检查一下..
我的意思是你不想要那些数以百万计的通知：填满屏幕的进度条最多有几千个像素，所以如果通知也不会改变任何可见的东西细粒度。工人不会替换大脑 :-) 在 doInBackground 中将其分解为合理的块并仅每 100.000 个左右通知一次

标签： java swing process publish swingworker

【解决方案1】：

您不应该发布每一个更改，而应该每 10 次发布一次：

for (int i = 0; i < maximum; i++) {
    doTheWork();
    if(i % 10 == 0) {
        publish(i+1);
    }
}
publish(maximum);

【讨论】：