现在，我们将以上示例重构一下。将实际的工作代码移到一个类型SampleClass中去，该示例要在多个SampleClass实例间操作一个静态字段：
|
private void buttonStartThreads_Click(object sender, EventArgs e)
{
SampleClass sample1 = new SampleClass();
SampleClass sample2 = new SampleClass();
sample1.StartT1();
sample2.StartT2();
}
class SampleClass
{
public static List<string> TempList = new List<string>() { "init0", "init1", "init2" };
static AutoResetEvent autoSet = new AutoResetEvent(false);
object syncObj = new object();
public void StartT1()
{
Thread t1 = new Thread(() =>
{
//确保等待t2开始之后才运行下面的代码
autoSet.WaitOne();
lock (syncObj)
{
foreach (var item in TempList)
{
Thread.Sleep(1000);
}
}
});
t1.IsBackground = true;
t1.Start();
}
public void StartT2()
{
Thread t2 = new Thread(() =>
{
//通知t1可以执行代码
autoSet.Set();
//沉睡1秒是为了确保删除操作在t1的迭代过程中
Thread.Sleep(1000);
lock (syncObj)
{
TempList.RemoveAt(1);
}
});
t2.IsBackground = true;
t2.Start();
}
} |
该示例运行起来抛出异常InvalidOperationException：“集合已修改；可能无法执行枚举”。查看类型SampleClass的方法StartT1和StartT2，方法内部锁定的是SampleClass的实例变量syncObject。实例变量意味着每创建一个SampleClass的实例都会生成一个syncObject对象。在本例中，调用者一共创建了两个SampleClass实例，继而分别调用：
sample1.StartT1();
sample2.StartT2(); |
以上代码锁定的是两个不同的syncObject，这等于完全没有达到两个线程锁定同一个对象的目的。要修正以上的错误，只要将syncObject变成static就可以了。
另外，思考一下lock(this)，我们同样不建议在代码中编写这样的代码。如果两个对象的实例分别执行了锁定的代码，实际锁定的也是两个对象，完全不能达到同步的目的。
