AutoResetEvent.WaitOne() 导致死锁

Question

我正在编写一个带有关键区域的应用程序。

我决定使用 AutoResetEvent 来实现互斥。 这是代码

    public class MyViewModel
    {
        private AutoResetEvent lock = new AutoResetEvent(true);
        private aync Task CriticalRegion()
        {
            Dosomething();
        }


        public async Task Button_Click()
        {
            Debug.WriteLine("Entering Button_Click");
            lock.WaitOne();
            try
            {
                await CriticalRegion();
            }
            finally
            {
                lock.Set();
                Debug.WriteLine("Leaving Button_Click");

            }
        }

    }

我有一个按钮，其点击事件调用Button_Click() 方法

正常工作。但是，如果我足够快地在第一次调用 Button_Click() 完成之前再次单击该按钮，则整个应用程序将停止响应。

在“调试”窗口中，我找到了类似的内容

Entering Button_Click
Entering Button_Click

看起来该方法永远不会完成。

我挣扎了一下，发现如果我把lock.WaitOne();改成

   if (!sync.WaitOne(TimeSpan.FromSeconds(1)))
   {
       return;
   }

在这种情况下，我的应用程序能够避免死锁，但我不知道它为什么会起作用。

我只知道我的操作系统课程中的 IPC 和 C# 中的 async 和 await 模式，我对 .Net 世界中的线程不太熟悉。

我真的很想了解幕后的真实情况。 感谢您的任何回复；）

Answer 1

AutoResetEvent.WaitOne() 将无限阻塞，直到您调用 AutoResetEvent.Set()，除了在WaitOne() 调用之后，您似乎从未这样做。

引用AutoResetEvent.WaitOne() 文档：

阻塞当前线程直到当前 WaitHandle 收到信号。

Answer 2

你有死锁是因为WaitOne阻塞了主线程（按钮点击处理程序在主线程上执行），而你在调用await时没有调用ConfigureAwait(false)，这意味着它试图运行主线程await之后的代码，即使被阻塞，也会导致死锁。

我建议阅读this post 以获得对死锁情况的更彻底的解释。

对于您的代码，我建议将锁放得更深，可能在异步任务中，并尝试使用更合适的锁定模式，最好是lock statement，因为使用Event 对象对于互斥来说很尴尬，正如汉斯在评论中所说的那样。