你有什么理由在这样的布尔属性的 getter 和 setter 周围创建锁？
private _lockObject = new object();
private bool _myFlag;
public bool MyFlag
{
get
{
lock (_lockObject)
{
return _myFlag;
}
}
set
{
lock (_lockObject)
{
_myFlag = value;
}
}
}
【问题讨论】：
标签： c# multithreading properties locking
好吧，您不一定需要 锁 - 但如果您希望一个线程明确读取另一个线程写入的值，您需要锁或 volatile 变量。
我个人已经放弃尝试理解 volatile 的确切含义。我尽量避免编写自己的无锁代码，而是依赖真正了解内存模型的专家。
编辑：作为这可能导致的问题的示例，请考虑以下代码：
using System;
using System.Threading;
public class Test
{
private static bool stop = false;
private bool Stop
{
get { return stop; }
set { stop = value; }
}
private static void Main()
{
Thread t = new Thread(DoWork);
t.Start();
Thread.Sleep(1000); // Let it get started
Console.WriteLine("Setting stop flag");
Stop = true;
Console.WriteLine("Set");
t.Join();
}
private static void DoWork()
{
Console.WriteLine("Tight looping...");
while (!Stop)
{
}
Console.WriteLine("Done.");
}
}
该程序可能会或可能不会终止。我见过这两种情况。无法保证“读取”线程会实际上从主内存中读取 - 它可以将 stop 的初始值放入一个寄存器并永远使用它。我亲眼目睹了这种情况，在现实中。它不会在我当前的机器上发生，但它可能会在我的下一个机器上发生。
根据问题中的代码在属性 getter/setter 中放置锁将使此代码正确且其行为可预测。
有关更多信息，请参阅blog post by Eric Lippert。
【讨论】：
bool 的读写是atomic。
但是，名称“flag”表示单独的线程将读取/写入，直到某些情况发生。为避免由于优化导致的意外行为，您应该考虑在 bool 声明中添加 volatile 关键字。
【讨论】：
没有理由在那儿有锁。
锁定在您的设计中可能很合适，但很怀疑这是正确的粒度。
您需要使您的设计线程安全，而不是单个属性（甚至整个对象）。
【讨论】：