锁定静态变量

Question

我在服务器端有一个以下课程。

public class Sample
{
    private enum Status
    {
        NotEvaluated,
        Yes,
        No
    }

    private static object _lockObj = new object();
    private static Status _status = Status.NotEvaluated;

    public static Status GetStatus()
    {
        if (_status == Status.NotEvaluated)
        {
            lock (_lockObj)
            {
                if (_status == Status.NotEvaluated)
                {
                    //some evaluation code which sets status to either Yes/No;
                    _status = Status.Yes;
                }
            }
        }

        return _status;
    }

}

上面的锁定机制有问题吗？我需要锁定吗？因为它是服务器端的（会有多个请求）并且变量是静态的，所以我认为应该在评估时锁定它。

如果我错了请纠正我。

谢谢

Answer 1

您不/不应该对“if (_status == Status.NotEvaluated)”进行外部检查。虽然如果您离开它，似乎不会发生任何“坏”事情，但第二个线程可能会在第一个线程将 _status 设置为 Status.Yes 之前不必要地输入该“if”。是的，您需要锁定：

“作为基本规则，您需要锁定访问任何可写共享字段。”
http://www.albahari.com/threading/part2.aspx

lock (_lockObj)
   {
      if (_status == Status.NotEvaluated)
         {
            // some evaluation code which sets status to either Yes/No;
            _status = Status.Yes;
         }
      return _status;    
  }