检查当前线程是否拥有锁

假设我有以下代码:

public class SomeClass()
{
    private readonly object _lock = new object();

    public void SomeMethodA()
    {
        lock (_lock)
        {
            SomeHelperMethod();

            //do something that requires lock on _lock
        }
    }

    public void SomeMethodB()
    {
        lock (_lock)
        {
            SomeHelperMethod();

            //do something that requires lock on _lock
        }
    }

    private void SomeHelperMethod()
    {
        lock (_lock)
        {
            //do something that requires lock on _lock
        }
    }
}

锁定内部SomeHelperMethod似乎是多余和浪费的,因为所有呼叫者都已经锁定.但是,简单地删除锁SomeHelperMethod似乎很危险,因为我们以后可能会重构代码并忘记_lock在调用之前锁定对象SomeHelperMethod.

理想情况下,我可以通过断言当前线程在_lock内部拥有一个锁来解决这个问题SomeHelperMethod:

private void SomeHelperMethod()
{
    Debug.Assert(Monitor.HasLock(_lock));

    //do something that requires lock on _lock
}

但似乎没有这样的方法.Monitor.TryEnter并没有帮助,因为锁是可重入.因此,如果当前线程已经拥有锁,TryEnter仍然会成功并返回true.它失败的唯一时间是另一个线程拥有锁并且呼叫超时.

那么,这种方法存在吗？如果没有,为什么？它对我来说似乎没什么危险,因为它只是告诉你当前线程(不是另一个线程)是否拥有锁.

.NET 4.5使用Monitor.IsEntered(object)支持此功能.

锁定或重新锁定实际上是免费的.这种低成本的缺点是您没有其他同步机制那么多的功能.如上所述,只要锁是至关重要的,我就会锁定.

如果你绝对想要省略"不必要的"锁定而没有潜在重构所涉及的风险,请添加注释.如果有人更改了您的代码并且它已经中断,那么就会有一条注释可以解释原因.如果他们仍然无法弄明白,那就是他们的问题,而不是你的问题.另一种方法是创建一个类用作锁定对象,该类包含一个可以轻弹的布尔值.但是,这样做的开销(包括try/finally块,这些都不是免费的)可能不值得.