Question

在众所周知的Joseph Albahari's article on Threading中，多个线程使用的所有类变量都声明为static fields =＆gt;所有线程都可以访问同一个变量。这需要在所有阅读/写作场所通过lock()机制进行配备，并完成工作。

我的问题是关于类属性的实现。我知道如果我使用Timeout后备存储实现（例如）属性static，它是有效的：

class MyClassWithWorkerThread {
    private readonly object _locker = new object();
    private static int _Timeout = false;
    private int Timeout {
        get {
            lock (_locker) {
                return _Timeout;
            }
        }
        set {
            lock (_locker) {
                _Timeout = value;
            }
        }
    }
}

这将使变量_Timeout在所有类实例之间共享。

但在我的情况下，多线程处理对类实例是私有的。它以New()开头，以Dispose()结尾。主线程和工作线程访问Timeout属性（但_Timeout后备存储从不在属性getter / setter外部访问）。

我不希望_Timeout值在应用程序范围内。我希望每个类实例都具有唯一性。 我的问题是：我可以安全地从static变量中移除_Timeout以实现此目标吗？

注意：如果代码中有任何错误，我很抱歉，我实际上正在使用VB.NET并使用工具转换它。我希望主要问题仍然清楚。

Answer 1

绝对安全且非常推荐（静态变量即使在需要时也很难测试）。假设安全你也意味着有效你不能忘记的是：

this.Timeout = 0; // This is safe and valid
++this.Timeout; // This is safe but not valid

因为++运算符不是原子的（这就是为什么我们有Interlocked类）。当然同样适用于这种情况：

if (this.Timeout == 0)
    Timeout = 10;

因为即使每次访问安全（我会说读取属性总是安全的，但你可能会读到没有lock障碍的旧值）它不是原子的操作和价值可能会在测试后和新任务之前发生变化。更复杂的是什么？

if (this.Timeout == 0)
    Timeout = Timeout * 2;

在这种情况下，每次阅读Timeout时，您都可能获得不同的价值。出于这个原因，我说除非它是一个只读属性，否则该属性中的锁很少有用。最好从属性get / set中删除该锁，并将代码包装在lock语句中：

lock (_locker) {
    if (this.Timeout == 0)
        Timeout = Timeout * 2;
}

另请注意，对于int _Timeout（我假设与false的作业只是一个拼写错误），您可以简单地删除锁定并将其设为volatile：

private volatile int _Timeout;

当然这不会解决其他描述的问题，但它对于只读属性可能是有用的（并且更快）（或者对于非常受控的情况，volatile修饰符可能很棘手并且它具有与C相比具有不同的含义，并且很容易忘记访问它们是原子的，但仅此而已。）

Answer 2

static关键字是正交的（即：独立或无关）线程。

只有当您只需要该属性/ field / class / method / etc的单个实例时，才应使用static关键字。

在您的情况下，如果您希望该值对实例是唯一的，则在_Timeout上不需要静态。

我没有看过您链接的文章，但我认为您可能误解了它（或者作者误解了静态的使用）

在多线程期间共享数据 - 是非静态变量吗？

2 个答案: