工作线程和 UI 线程调用相同方法时死锁，包括调用 ui 控件

Question

这是我描述问题的代码：

public partial class Form1 : Form
{
    private readonly  object lockObject = new object();
    public Form1()
    {
        InitializeComponent();
    }

    private void btnWorkerThread_Click(object sender, EventArgs e)
    {
        Task.Factory.StartNew(() => SomeLongRunningThread());
    }

    private void SomeLongRunningThread()
    {
        lock (lockObject)
        {
            Thread.Sleep(3000);
            if (txtResult.InvokeRequired)
            {
                Thread.Sleep(3000);
                txtResult.Invoke((MethodInvoker) delegate { txtResult.Text = DateTime.Now.ToShortTimeString(); });
            }
            else
            {
                Thread.Sleep(3000);
                txtResult.Text = DateTime.Now.ToShortTimeString();
            }
        }

    }

    private void btnUIThread_Click(object sender, EventArgs e)
    {
        SomeLongRunningThread();
    }
}

当工作线程启动然后btnUIThread被点击时发生死锁：

txtResult.InvokeRequried 上好像发生了死锁：

• 工作线程将控制权传递给 ui 线程
• 由于 ui 线程轮到，它同时更新 txtResult 并尝试启动 btnUIThread 点击。
• 在线txtResult.Invoke((MethodInvoker) delegate { txtResult.Text = DateTime.Now.ToShortTimeString(); }); 发生死锁

如果我错了，请纠正我。

问题是如何处理死锁？

编辑：经过各位大侠的宝贵解答，我的实际项目比预期的要复杂，锁块有更多的代码，我应该保证一次一个线程执行。

Answer 1

为避免死锁，您应该以不同的方式lock。锁定所有线程代码是没有意义的。 此外，此锁 (lockobject) 不是必需的，因为在您的示例中 SomeAction() 已经同步以在主/UI 线程中执行。

private void SomeLongRunningThread()
{

    Thread.Sleep(3000);
    if (txtResult.InvokeRequired)
    {
        Thread.Sleep(3000);
        txtResult.Invoke((MethodInvoker) delegate { SomeAction });
    }
    else
    {
        Thread.Sleep(3000);
        SomeAction();
    }


}

private void SomeAction(){
    // This lock is not needed as long this method is only called from SomeLongRunningThread()
    //lock (lockObject)
    //{
        txtResult.Text = DateTime.Now.ToShortTimeString();
    //}
}

Answer 2

这里根本不需要lock。

调用 txtResult.Invoke((MethodInvoker) delegate { txtResult.Text = DateTime.Now.ToShortTimeString(); }); 会将委托推送到 UI 消息循环队列。当循环准备好时，它会弹出委托并执行。 UI 一次只能运行一件事 - 没有竞争，也没有死锁。

Answer 3

.NET 4.5 引入了IProgress< T> 接口来报告线程和任务的进度，以及Progress< T> 实现在创建它的线程中运行回调或引发事件。有了这个和async/await，您就不需要调用BeginInvoke 或`Invoke。

这段代码可以使用Progress来简化：

public partial class Form1 : Form
{    
    IProgress<string> _progress;
    public Form1()
    {
        InitializeComponent();
        _progress = new Progress<string>(UpdateUI);
    }

    void UpdateUI(string message)
    {
        txtResult.Text = message;
    }

    private void btnWorkerThread_Click(object sender, EventArgs e)
    {
        Task.Run(() => SomeLongRunningThread());
    }

    private void SomeLongRunningThread()
    {
        Thread.Sleep(3000);
        _progress.Report(DateTime.Now.ToShortTimeString());
    }

    private void btnUIThread_Click(object sender, EventArgs e)
    {
        SomeLongRunningThread();
    }
}