后台工作人员总是很忙

Question

我是使用事件处理程序和后台工作者的新手，所以这里可能缺少一些非常明显的东西。尽管如此，我已经试着解决这个问题两天了，所以我想我还不如看看别人会说什么。

我有一个叫SqlExpressDownloader的后台工作人员。它在我的程序开始时开始运行，其余的工作都在运行，然后它应该等待SqlExpressDownloader_DoWork()方法中的操作完成后再继续。唯一的问题是，由于某种原因，每当我执行while(SqlExpressDownloader.IsBusy)时，它总是响应繁忙，因此将永远等待。

事件处理程序的代码如下：

    private void SqlExpressDownloader_DoWork(object sender, DoWorkEventArgs e)
    {
        string sSource = string.Format("{0}\{1}", Paths.Settings_Common, "sqlexpr_x64_enu.exe");
        Debug.WriteLine(sSource);
        Debug.WriteLine("http://www.elexioamp.com/Install/redistributables/sql2008r2express/sqlexpr_x64_enu.exe");
        if (!System.IO.File.Exists(sSource))
        {
            WebClient oWebClient = new WebClient();
            oWebClient.DownloadProgressChanged += DownloadProgressChanged;
            oWebClient.DownloadDataCompleted += DownloadComplete;
            oWebClient.DownloadFileAsync(new System.Uri("http://www.elexioamp.com/Install/redistributables/sql2008r2express/sqlexpr_x64_enu.exe"), sSource);
            while (oWebClient.IsBusy)
            {
                Thread.Sleep(100);
            }
            e.Result = "";
            DownloadFinished = true;
        }
    }

我看过代码，也看过它完成这个方法。我甚至在DownloadFinished = true之后添加了一个return，但它的响应仍然很忙。我想知道的是如何让后台工作人员认为自己不忙。

编辑所有事件都添加到构造函数中，如下所示：

        SqlExpressDownloader = new BackgroundWorker();
        SqlExpressDownloader.DoWork += new DoWorkEventHandler(this.SqlExpressDownloader_DoWork);
        SqlExpressDownloader.RunWorkerCompleted += new RunWorkerCompletedEventHandler(this.SqlExpressDownloader_RunWorkerCompleted);

RunWorkerCompleteEventHandler看起来像这样：

    private void SqlExpressDownloader_RunWorkerCompleted(object sender, RunWorkerCompletedEventArgs e)
    {
        if (e.Error != null)
        {
            Debug.WriteLine("The actions are complete.");
        }
        else
        {
            Debug.WriteLine("Error in completed work.");
        }
    }

但是，当我上次调试它时，它实际上并没有触发。

Answer 1

不要在循环中查询SqlExpressDownloader.IsBusy，而是尝试订阅BackgroundWorker的RunWorkerCompleted事件，并将您的代码放在那里，这应该只在DoWork事件完成后发生。

您还可以访问RunWorkerCompletedEventArgs，可以对其进行检查以确保BackgroundWorker的DoWork部分没有抛出错误。

    ...
    ...
    SqlExpressDownloader.RunWorkerCompleted += SqlExpressDownloader_RunWorkerCompleted;
    SqlExpressDownloader.RunWorkerAsync();
}
private void SqlExpressDownloader_RunWorkerCompleted(object sender, RunWorkerCompletedEventArgs e)
{
    if (e.Error != null)
    {
        // do something in response to the error
    }
    // stuff to do after DoWork has completed
}

当我学习如何使用这些教程时，我发现Joe Albahari的教程很有帮助。

Answer 2

您可以用像这样更优雅的异步/等待解决方案来替换您的代码

private async Task SqlExpressDownloadAsync()
{
    string sSource = string.Format("{0}\{1}", Paths.Settings_Common, "sqlexpr_x64_enu.exe");
    Debug.WriteLine(sSource);
    Debug.WriteLine("http://www.elexioamp.com/Install/redistributables/sql2008r2express/sqlexpr_x64_enu.exe");
    if (!System.IO.File.Exists(sSource))
    {
        WebClient oWebClient = new WebClient();
        oWebClient.DownloadProgressChanged += DownloadProgressChanged;
        oWebClient.DownloadDataCompleted += DownloadComplete;
        await oWebClient.DownloadFileTaskAsync(new System.Uri("http://www.elexioamp.com/Install/redistributables/sql2008r2express/sqlexpr_x64_enu.exe"), sSource);  
    }   
}

Answer 3

我也遇到了类似的问题。DownloadASync会启动，但是.IsBusy将始终保持为true。

这可能不是一个常见的问题，只是想我也有同感。

我使用

MessageBox.Show(new Form() { TopMost = true }, "", "")

这就是原因。我也试过：

var t = new Form() { TopMost = true };
MessageBox.Show(t, "", "");
t.Dispose();

这也引发了同样的问题。

我的代码有多个线程，我认为其中一个线程一定卡住了，或者MessageBox（新的Form（）｛TopMost=true；｝）调用创建了一个卡住的线程。

我一拆下那个零件，例如

MessageBox.Show("", "");

一切如愿以偿。

因此，也许您正在某个地方创建另一个线程，从而导致您的问题。