这会导致死锁吗？ BeginInvoke() & thread.Join()

Question

我有许多线程可以调用来更新 GUI 的代码：

MethodInvoker del = () => { lblInfo.Text = tmp; };
lblInfo.BeginInvoke(del);

（lblInfo由GUI线程创建）

我也有这个方法在 GUI 线程执行的按钮单击时调用：

public void Stop()
{
    isStopping = true;
    crawler.Join();
    foreach (Thread t in txtWorkers)
    {
        t.Join();
    }
    indexer.Join();     
    lblStatus.Text = "Stopped";
    lblInfo.Text = "";
}

1 次超过 100 次运行程序在单击停止按钮时死锁。当我看到死锁时我没有调试，所以我不能确定各种线程的状态，但我几乎可以肯定我加入的所有线程最终都会到达他们检查的点 isStopping 值并终止。这使我认为BeginInvoke 可能存在问题，但无法真正找到它。它应该是异步的，因此调用它的线程（爬虫和索引器）不应该阻塞。如果 GUI 线程正在执行 Stop() 并且还必须执行来自 BeginInvoke 的调用，会发生什么情况？这可能是问题吗？我正在加入的线程有什么我看不到的吗？

编辑： 建议更改后的代码是什么样的：

public void Stop()
{
    /*
     ...disable GUI
     */

    isStopping = true; // Declared as volatile
    lblStatus.Text = "Stopping...";

    // Creating a thread that will wait for other threads to terminate
    Task.Factory.StartNew(() =>
    {    
        crawler.Join();
        foreach (Thread t in txtWorkers)
        {
            t.Join();
        }
        indexer.Join();

        // Adjust UI now that all threads are terminated
        MethodInvoker del = () =>
        {
            /*
            ...enable GUI
            */
            lblStatus.Text = "Not Running";
            isStopping = false;
        };
        lblStatus.BeginInvoke(del);
    });
}

它似乎在工作，我希望僵局已经过去了......

Answer 1

我不认为这应该是个问题，因为您使用的是BeginInvoke 而不是Invoke - 后台线程将直接越过该行而无需等待 GUI跟上来。如果您在任何地方使用Control.Invoke，那可能会导致死锁。

更重要的是，在您的 GUI 线程中使用 Join 从根本上来说是个坏主意 - UI 将被冻结，直到一切都完成。最好禁用任何可以启动新事物的控件，设置 isStopping 标志，然后创建一个新线程以等待所有线程停止 - 当所有线程都完成时，然后 em> 再次使用 BeginInvoke 更新 UI。 （如果您使用的是 .NET 4.5，您还可以为此使用异步方法，创建并等待任务以等待所有线程。）

最后，如果isStopping 只是一个bool 字段，则无法保证您的后台线程会“看到”来自UI 线程的更改。将字段设为 volatile 可能会解决此问题，但 volatile 的确切含义让我感到害怕。另一种方法是使用Interlocked 类，或将其设为一个属性，该属性可同时获得读取和写入锁定 - 确保适当的内存屏障到位。