ThreadPool在调试模式和运行时的行为不同

Question

我想使用ThreadPool在更短的时间内完成长时间运行的作业。我的方法当然会做更多的工作，但我准备了一个简单的例子让你理解我的处境。如果我运行这个应用程序，它会在注释行上抛出ArgumentOutOfRangeException。它还表明i等于10。如果是10，它如何进入for循环？

如果我不运行应用程序并调试此代码，它不会引发异常，并且工作正常。

public void Test()
{
List<int> list1 = new List<int>();
List<int> list2 = new List<int>();
for (int i = 0; i < 10; i++) list1.Add(i);
for (int i = 0; i < 10; i++) list2.Add(i);
int toProcess = list1.Count;
using (ManualResetEvent resetEvent = new ManualResetEvent(false))
{
for (int i = 0; i < list1.Count; i++)
{
ThreadPool.QueueUserWorkItem(
new WaitCallback(delegate(object state)
{
// ArgumentOutOfRangeException with i=10
Sum(list1[i], list2[i]);
if (Interlocked.Decrement(ref toProcess) == 0)
resetEvent.Set();
}), null);
}
resetEvent.WaitOne();
}
MessageBox.Show("Done");
}
private void Sum(int p, int p2)
{
int sum = p + p2;
}

这里有什么问题？

Answer 1

问题是i==10，但您的列表有10个项目(即最大索引为9)。

这是因为您对在代理运行之前正在更改的捕获变量具有竞争条件。循环的下一次迭代是在代理运行之前增加值，还是在循环增加值之前运行代理？这一切都取决于具体跑步的时间安排。

您的直觉是i的值为0-9。然而，当循环到达其终止时，i将具有值10。因为委托捕获了i，所以在循环终止之后可以很好地使用i的值。

更改您的循环如下：

for (int i = 0; i < list1.Count; i++)
{
var idx=i;
ThreadPool.QueueUserWorkItem(
new WaitCallback(delegate(object state)
{
// ArgumentOutOfRangeException with i=10
Sum(list1[idx], list2[idx]);
if (Interlocked.Decrement(ref toProcess) == 0)
resetEvent.Set();
}), null);
}

现在，您的委托获得了i的"私有"、独立的副本，而不是引用在委托的所有调用之间共享的单个、不断变化的值。

我不会太担心调试模式和非调试模式之间的行为差异。这就是种族状况的本质。

Answer 2

这里有什么问题？

结束。您正在捕获i变量，该变量没有执行您期望的操作。

您需要在for循环中创建一个副本：

var currentIndex = i:
Sum(list1[currentIndex], list2[currentIndex]);