使用标志退出 BackgroundWorker

Question

想象一下我有这样的代码：

 private bool flag = false;
 public SomeClass()
 {
     public void setup()
    {
        worker = new BackgroundWorker();
        worker.DoWork += worker_DoWork;

        if(!worker.IsBusy)
            worker.RunWorkerAsync();
    }

    void worker_DoWork(object sender, DoWorkEventArgs e)
    {
        if(flag == true)
          return;
        //whatever You want the background thread to do...
        SomeClass.SomeStaticMethodWhichHasLoopInside();
    }
}

当用户在我的应用程序中单击退出按钮时，我可以在里面设置flag=true - 这是停止由BackgroundWorker 启动的线程的正确方法吗？ 我只想在应用程序想要退出时才这样做 - 这样我就停止了线程。还是会被自动杀死？

Answer 1

没有。不要那样做。它不会按预期工作。您的 flag 字段甚至没有声明为 volatile；该字段的真实值可能尚未刷新到内存中。其他线程仍然会看到 flag 为 false，因为它缓存了自己的值副本，而您的 BackgroundWorker 可能永远不会结束。

相反，只需调用 BackgroundWorker.CancelAsync 并检查 DoWork 事件处理程序中的 BackgroundWorker.CancellationPending 布尔值。

private void OnExit(..)
{
    worker.CancelAsync();
}

所以你的 DoWork 方法会变成

void worker_DoWork(object sender, DoWorkEventArgs e)
{
    while(!worker.CancellationPending)
    {
        //whatever You want the background thread to do...
    }
}

如果您没有循环，则需要不时检查CancellationPending属性。

void worker_DoWork(object sender, DoWorkEventArgs e)
{
    if(worker.CancellationPending)
        return;
    //Do something..
    if(worker.CancellationPending)
        return;
    //Do some other thing..
    if(worker.CancellationPending)
        return;
    //Do something else..
}

我只想在应用程序想要退出时才这样做 - 这样我就停止了 线程。还是会被自动杀死？

由于线程是后台线程，它将被 CLR 自动终止。但是，不要依赖这个事实，因为在 CLR 杀死线程时我们不知道您的线程将执行什么。例如，它可能正在写入一个文件，在这种情况下，您将留下损坏的文件。

后果可能更严重，因此始终建议您自己优雅地停止线程。

在 cmets 中提出的问题：当循环处于某些状态时如何取消 其他方法？

看来您使用的是 .Net4.5。您可以利用CancellationTokenSource 和CancellationToken。您也可以考虑使用 TPL 和 async-await 功能，这将使您的生活更轻松。

private CancellationTokenSource tokenSource = new CancellationTokenSource();

private void OnExit(..)
{
    tokenSource.Cancel();
}

void worker_DoWork(object sender, DoWorkEventArgs e)
{
    //whatever You want the background thread to do...
    SomeClass.SomeStaticMethodWhichHasLoopInside(tokenSource.Token);
}

class SomeClass
{
    public static void SomeStaticMethodWhichHasLoopInside(CancellationToken token)
    {
        while (!token.IsCancellationRequested)
        {
            //Do something
        }
    }
}