Monitor.Exit() 有时会挂起

Question

我在使用 Monitor.Enter 和 Monitor.Exit 时似乎遇到了并发问题。有时我的代码会挂在下面的 Monitor.Exit 语句上：

public void EndInit()
{
    Monitor.Enter(this.lockObj);
    this.initCount--;
    if (this.initCount == 0) {
        this.IsInitializing = false;
        this.IsInitialized = true;
        this.OnInitialized();
    }
    // sometimes, this Exit will never return ...
    Monitor.Exit(this.lockObj);
}

只有另外一个地方使用了我的 lockObj：

public void BeginInit()
{
    Monitor.Enter(this.lockObj);
    this.initCount++;
    this.IsInitializing = true;
    this.IsInitialized = false;
    Monitor.Exit(this.lockObj);
}

这就是我声明这个同步对象的方式：

private readonly object lockObj = new object();

我竭尽全力想知道这里发生了什么，但没有成功。我希望Monitor.Enter() 会阻止直到我的同步对象被释放，但为什么Monitor.Exit() 会被阻止？我在 MSDN 中也找不到对这种行为的任何解释。

注意我无法重现这种行为，它是随机发生的（好吧，我知道“随机”不是正确的措辞）。

非常感谢任何想法或有用的提示！

索斯滕

Answer 1

我正在根据我之前的评论做出回答。因为您应该使用try finally 构造来正确地在OnInitialize() 中发生异常时调用Monitor.Exit。

所以代码会变成

public void EndInit()
{
    Monitor.Enter(this.lockObj);
    try
    {
        this.initCount--;
        if (this.initCount == 0) {
            this.IsInitializing = false;
            this.IsInitialized = true;
            this.OnInitialized();
        }
    } 
    finally
    {
        Monitor.Exit(this.lockObj);    
    }
}

第二种方法也是如此。

那么也可以这样写

public void EndInit()
{
    lock(this.lockObj)
    {
        this.initCount--;
        if (this.initCount == 0) {
            this.IsInitializing = false;
            this.IsInitialized = true;
            this.OnInitialized();
        }
    }
}

编辑

可以在here 找到由 Joe Albahari 编写的关于线程的非常好的解释。值得一读。

编辑 2（为了完整性）

正如 Damien_The_Unbeliever 所说，还有一个 overload。

这仅适用于 .NET 4 及更高版本。使用监视器的代码将变为：

public void EndInit()
{
    bool lockAcuired = false;
    try
    {
        Monitor.Enter(this.lockObj, ref lockAquired);
        this.initCount--;
        if (this.initCount == 0) {
            this.IsInitializing = false;
            this.IsInitialized = true;
            this.OnInitialized();
        }
    } 
    finally
    {
        if(lockAquired)
            Monitor.Exit(this.lockObj);    
    }
}