假设我在某个类中有以下属性，其目的是用作锁。
protected object SyncRoot { get; private set; }
无论如何，不管这是如何设置的，如果它确实被设置了，最佳做法是如何使用它？
由于锁不能与空对象一起使用，我应该像这样处理吗？
lock (SyncRoot ?? new object())
SomeMethod();
或者我应该像这样检查 null 吗？
if (SyncRoot != null)
lock (SyncRoot)
SomeMethod();
else
SomeMethod();
通常我使用私有成员变量而不是属性，例如
private static object MyLock = new object();
这种方式可以确保它始终被初始化。
您也可以使用非静态版本，例如
private readonly object MyLock = new object();
同步
SyncRoot ?? new object()
如果SyncRoot为null，那么每个线程每次都会得到一个新的对象，因此毫无意义。在单独的对象上同步没有效果：线程将立即继续执行，因为没有其他人可能在同一个new对象上同步。
您应该在构造函数中初始化SyncRoot，在第一个线程尝试获取锁之前进行初始化。
new object()上同步的性能影响已经无关紧要了，因为它是一个完全无用的习惯用法。 - Kirk Woll第一个问题是，它不会导致任何良好的同步：
lock (SyncRoot ?? new object())
SomeMethod();
SyncRoot，它将被放置在堆上，但没有对它的引用。因此，当另一个线程到来时，它将找不到它... 它变得完全无用，并且它不会阻止对关键部分的任何访问。来自文档： https://msdn.microsoft.com/zh-cn/library/c5kehkcz.aspx
通常情况下，避免锁定公共类型或您代码控制之外的实例。常见的构造 lock (this)、lock (typeof(MyType)) 和 lock ("myLock") 违反了此指南：
- 如果可以公开访问该实例，则 lock (this) 会出现问题。
- 如果 MyType 是公开可访问的，则 lock (typeof(MyType)) 会出现问题。
- 因为进程中使用相同字符串的任何其他代码都将共享相同的锁，所以 lock("myLock") 会出现问题。
最佳实践是定义一个私有对象进行锁定，或者定义一个私有静态对象变量以保护所有实例的公共数据。
示例：
class Account
{
decimal balance;
private Object thisLock = new Object();
public void Withdraw(decimal amount)
{
lock (thisLock)
{
if (amount > balance)
{
throw new Exception("Insufficient funds");
}
balance -= amount;
}
}
}
class Processor
{
readonly Queue<int> processQueue = new Queue<int>();
public void AddToQueue(int index)
{
lock (processQueue)
{
processQueue.Enqueue(index);
}
}
}
new object()--为什么你会有不初始化的情况呢？（为什么不在声明时就初始化它，从而默认消除这个问题呢？） - Kirk Woll