应该在属性上使用同步锁吗？

Question

我一直在阅读，关于是否应该在属性上使用同步锁得到了相互矛盾的答案。

我有一个多线程应用程序，它需要跨线程获取/设置实例对象的属性。它目前是在没有使用同步锁的情况下实现的，到目前为止我还没有注意到任何问题。我在常用静态方法上使用同步锁，但我想以线程安全的方式正确实现我的实例类。

任何反馈都将不胜感激。

Answer 1

一个好的经验法则是，如果以下任何一个条件成立，您就需要锁定：

• 如果要在多个线程上修改对象的任何字段
• 如果任何修改涉及访问多个字段
• 如果任何可修改字段是 Double、Decimal 或结构化值类型
• 如果任何修改涉及读取-修改-写入（即添加到字段或使用另一个字段的值设置一个字段）

那么您可能需要锁定访问这些字段的每个方法或属性。

编辑：请记住，在一个类中锁定很少是足够的——您需要做的是确保在整个 的范围内事情不会出错逻辑运算.

正如@Bevan 指出的那样，如果调用代码需要多次访问一个对象，客户端代码应该在其工作的整个过程中对对象进行自己的锁定，以确保另一个线程不会获得“在两者之间”它的访问并弄乱它的逻辑。

您还需要注意，如果需要一次取出多把锁，它们总是以相同的顺序。如果线程 1 在实例 A 上锁定并尝试锁定实例 B，而线程 2 在实例 B 上锁定并尝试在实例 A 上获得锁定，则两个线程都被卡住并且无法继续 - 你有一个死锁。

Answer 2

你不能仅仅通过用锁包围单个方法来使对象线程安全。你最终要做的就是序列化（减慢）对对象的访问。

考虑这个小例子：

var myObject = ...
var myThreadSafeList = ...
if (!myThreadSafeList.Contains(myObject))
{
    myThreadSafeList.Add(myObject);
}

即使 myThreadSafeList 已锁定所有方法，这也不是线程安全的，因为另一个线程可以在 调用 Contains() 和 Add() 之间更改列表的内容。

在这个列表的情况下，需要一个额外的方法：AddIfMissing():

var myObject = ...
var myThreadSafeList = ...
myThreadSafeList.AddIfMissing(myObject);

只有将逻辑移动到对象中，才能将这两个操作都用锁包围并使其安全。

没有更多细节，很难进一步评论，但我建议如下：

• 将所有属性设为只读，并允许任何人随时读取它们
• 提供 mutator 方法，这些方法采用一组一起修改的属性，并在锁内以原子方式进行更改

举例说明：

public class Person {
    public string FullName { get; private set; }
    public string FamilyName { get; private set; }
    public string KnownAs { get; private set; }

    public void SetNames( string full, string family, string known) {
        lock (padLock) {
            ...
        }
    }
}