c# - 在许多属性中使用 ReaderWriterLockSlim 的更简洁的方法

Question

我正在尝试实现许多利用 ReaderWriterLockSlim 实现线程安全的属性。所以像大多数人一样，我最终在我的所有属性中都有这样的东西:

public string Name
{
    get
    {
        rwLock.EnterReadLock();
        try {
            return name;
        }
        finally {
            rwLock.ExitReadLock();
        }
    }
    set
    {
        rwLock.EnterWriteLock();
        try {
            name = value;                    
        }
        finally {
            rwLock.ExitWriteLock();
        }
    }
}

这在 10 个属性中感觉非常冗长和重复，所以我正在寻找更 DRY 的实现。

显而易见的解决方案是将其包装到一个释放处置锁的类中，并允许我将我的线程安全操作放在 using 语句中。根据 this 显然这不是很安全以及其他一些来源。

所以我尝试使用 lambda 表达式和匿名方法想出一个漂亮的解决方案:

private TResult ThreadSafeRead<TResult>(Func<TResult> value)
{
    rwLock.EnterReadLock();
    try {
        return value();
    }
    finally {
        rwLock.ExitReadLock();
    }            
}

private void ThreadSafeWrite(Action value)
{
    rwLock.EnterWriteLock();
    try {
        value();
    }
    finally {
        rwLock.ExitWriteLock();
    }
}

public string Name
{
    get { return ThreadSafeRead(() => name); }
    set { ThreadSafeWrite(() => { name = value; }); }
}       

虽然这消除了让我烦恼的重复代码，但我不知道它是否像我最初的冗长实现一样线程安全。

有没有人对 MSIL 这将生成和多线程有更深入的了解理论能够告诉我我的实现是否安全？

Answer 1

我见过人们解决这个问题的通常方法是将锁包装在一个对象中并按照以下方式做一些事情:

using (var obj = new ReadOnlyLock(this.rwLock))
{
    this.name = value;
}

我承认我不这样做(为什么要创建另一个对象只是为了获取锁？)但它确实实现了一些人认为最重要的更好的样式目标......