锁定属性，好方法？

Question

在我的多线程应用程序中，我使用了一些可以被多个实例同时更改的变量。这很奇怪，但它工作得很好，没有任何问题..但我当然需要让它成为线程安全的。我只是从锁开始，所以我很欣赏你的建议：

当客户端连接时，会创建类 Client，其中每个 Client 都有自己的“A”变量。

有时，客户端会这样调用方法：

Client selectedClient SelectOtherClientClassByID(sentID);

selectedClient.A=5;

即使有 5 个类同时执行（线程池），到目前为止也没有问题，但我在想如何为 A 属性添加锁？

喜欢：

A {
    get { return mA; }
    set {
        // use lock here for settting A to some value
    }    
}

可以吗？

Answer 1

您需要在获取和设置中都使用锁。这个锁必须是同一个对象。例如：

private object mylock = new object();

public int A {

  get {
    int result;
    lock(mylock) {
    result = mA; 
    }
    return result;
  } 

  set { 
     lock(mylock) { 
        mA = value; 
     }
  }
}

Answer 2

锁定对访问器内部属性的访问可能会导致虚假结果。例如，看下面的代码：

class C {
    private object mylock = new object();

    public int A {

      get {
        int result;
        lock(mylock) {
        result = mA; 
        }
        return result;
      } 

      set { 
         lock(mylock) { 
            mA = value; 
         }
      }
    }
}
C obj = new C;
C.A++;

（是的，我从第一个答案中复制了它） 这里有一个比赛条件！操作“C.A++”实际上需要对 A 进行两次单独的访问，一次获取值，另一次设置更新的值。没有什么可以确保这两个访问将一起执行，而无需在它们之间进行上下文切换。竞争条件的经典场景！

所以，小心！将锁放在访问器中不是一个好主意，应该显式获取锁，就像前面的答案所暗示的那样（尽管它不必与 SyncRoots 一起，任何对象都可以）

Answer 3

当您只需要设置一个属性时，这是非常罕见的。更多时候selectedClient.A = 5 将成为更大逻辑操作的一部分，其中涉及多个分配/评估/等。在整个操作过程中，您更希望selectedClient 处于一致状态，而不是引入死锁/竞争条件。因此，最好在 Client 类中公开 SyncRoot 属性并从调用代码中锁定它：

Client selectedClient = GetClient(...);

lock(selectedClient.SyncRoot)
{
    selectedClient.A = 5;
    selectedClient.B = selectedClient.A * 54;
}