BackgroundWorker无法按预期工作的问题求助

测试代码 #

using System;
using System.ComponentModel;
using System.Threading;
using System.Windows.Forms;

namespace BackgroundWorkingTest
{
    public partial class Form1 : Form
    {
        BackgroundWorker backgroundWorker1 = new BackgroundWorker();
        int counter;

        public Form1()
        {
            InitializeComponent();

            backgroundWorker1 = new BackgroundWorker
            {
                WorkerReportsProgress = true,
                WorkerSupportsCancellation = true
            };

            backgroundWorker1.DoWork += BackgroundWorker1_DoWork;
            backgroundWorker1.ProgressChanged += BackgroundWorker1_ProgressChanged;
            backgroundWorker1.RunWorkerCompleted += BackgroundWorker1_RunWorkerCompleted;
        }

        private void StartButton_Click(object sender, EventArgs e) // Start
        {
            counter = 0;
            backgroundWorker1.RunWorkerAsync();
        }

        private void CancelButton_Click(object sender, EventArgs e) // Cancel
        {
            backgroundWorker1.CancelAsync();
            progressLabel.Text = "0";
        }

        private void BackgroundWorker1_DoWork(object sender, DoWorkEventArgs e) // DoWork
        {
            int i;

            for (i = 0; i < 123456; i++)
            {
//              Application.DoEvents();    // This line doesn't help
//              Thread.Sleep(1);           // Enable this line solve the problem

                backgroundWorker1.ReportProgress(i + 1);
                counter++;

                if (backgroundWorker1.CancellationPending)
                {
                    e.Cancel = true;
                    return;
                }
            }
        }

        private void BackgroundWorker1_ProgressChanged(object sender, ProgressChangedEventArgs e) // Progress
        {
            progressLabel.Text = counter.ToString();
            progressLabel.Refresh();
        }

        private void BackgroundWorker1_RunWorkerCompleted(object sender, RunWorkerCompletedEventArgs e) // Completed
        {
            if (e.Cancelled)
                MessageBox.Show("Operation Cancelled");
            else
                MessageBox.Show("Operation Completed");
        }
    }
}

问题逐一解答 #

1. 禁用Thread.Sleep(1)后Cancel按钮无响应 #

后台线程循环速度极快，疯狂调用ReportProgress，导致UI线程被大量ProgressChanged事件占满，完全没时间处理Cancel按钮的点击消息。Application.DoEvents()没用是因为它是在后台线程调用的，UI消息队列的处理必须在UI线程执行，后台线程调用这个方法起不到任何作用。

Thread.Sleep(1)能解决是因为给后台线程加了微小延迟，降低了ReportProgress的触发频率，让UI线程有空闲处理按钮点击。更优的方案是直接降低进度报告的频率，比如每循环100次才报告一次，既不影响进度显示流畅度，又能释放UI资源：

for (i = 0; i < 123456; i++)
{
    counter++;
    // 每100次报告一次进度
    if (i % 100 == 0)
    {
        backgroundWorker1.ReportProgress(i + 1);
    }

    if (backgroundWorker1.CancellationPending)
    {
        e.Cancel = true;
        return;
    }
}
// 循环结束后补一次完整进度报告
backgroundWorker1.ReportProgress(123456);

2. 使用progressLabel.Refresh()后完成提示延迟弹出 #

后台线程触发了超大量ProgressChanged事件，这些事件会在UI线程的消息队列里排队。哪怕后台线程已经结束，UI线程还在处理队列里剩余的进度事件，必须等所有事件处理完，才会触发RunWorkerCompleted显示提示。

解决办法和上面一致：降低ReportProgress的触发频率，减少UI线程的事件队列压力。

3. 移除Refresh()后看不到中间计数 #

WinForms控件默认有延迟绘制机制，不会每次修改Text就立刻刷新界面。如果不想用Refresh()，可以改用ProgressBar控件来显示进度，它的更新机制更高效；或者保留Refresh()同时配合降低报告频率的方案，既能看到中间过程，又不会导致完成延迟。

4. 多个BackgroundWorker更新同一标签和计数器无报错 #

这属于线程不安全操作，只是没触发明显错误而已。多个后台线程同时修改全局counter，可能因线程竞争导致计数错误；如果直接在后台线程操作UI控件（而非通过ProgressChanged），随时可能触发跨线程操作异常——这次没报错只是巧合。

正确做法：

• 每个后台线程维护自己的局部计数，不要共享全局变量；
• 所有UI更新必须通过ProgressChanged事件（该事件自动在UI线程触发）；
• 如果需要合并多线程进度，在UI线程汇总各线程的进度数据。

内容的提问来源于stack exchange，提问作者Xenon