如果我不在 WinForms 上使用 Thread.Sleep，BackgroundWorker 将不起作用答案

【问题标题】：BackgroundWorker isn't working if I don't use Thread.Sleep on WinForms如果我不在 WinForms 上使用 Thread.Sleep，BackgroundWorker 将不起作用
【发布时间】：2021-11-24 02:47:35
【问题描述】：

这是我的简单代码：

private void button1_Click(object sender, EventArgs e)
{
    if (backgroundWorker1.IsBusy) return;
    backgroundWorker1.RunWorkerAsync();
}

private void backgroundWorker1_DoWork(object sender, DoWorkEventArgs e)
{
    var worker = sender as BackgroundWorker;

    for (var i = 1; i <= 10000; i++)
    {
        Thread.Sleep(1);
        worker?.ReportProgress(i);
    }
}

private void backgroundWorker1_ProgressChanged(object sender, ProgressChangedEventArgs e)
{
    label1.Text = e.ProgressPercentage.ToString();
}

private void backgroundWorker1_RunWorkerCompleted(object sender, RunWorkerCompletedEventArgs e)
{
    label1.Text = "Completed.";
}

此代码可以正常工作。但是如果我从代码中删除Thread.Sleep(1);，BackgroundWorker 将不起作用。这么简单的事情我都找不到错误的来源。

【问题讨论】：

不起作用是什么意思？如果你删除了 Sleep，你会用太多的连续更新来淹没 UI 线程。它不能像空的 for 循环那样快速更新。
@Steeeve - 可以这么说。我怎么知道？事实是：1000 条记录来自数据库。我将它们添加到对象列表中，并显示在 Gridview 中。在这种情况下也会发生同样的事情。
不知道真正的代码我不能说太多。但是，在 GridView 中显示 1000 条记录可能比从数据库中获取它们需要更多时间。

标签： c# backgroundworker

【解决方案1】：

没有必要这么频繁地报告进度。

做这样的事情：

for (var i = 1; i <= 10000; i++)
{
    Thread.Sleep(1);
    if (i % 100 == 0)
    {
        worker?.ReportProgress(i);
    }
}

或者您可以检查时间或滴答计数，当您看到 50 毫秒的时间已经过去时，请执行 ReportProgress。比这更频繁是没有意义的。

此外，您可以批量更新您的 GridView，以提高效率/减少浪费 - 一次发送这么多行 1 行将会产生更多开销。

【讨论】：