Question

我想使用ThreadPool在更短的时间内完成长时间运行的作业。我的方法当然做更多的工作但我准备了一个简单的例子供你理解我的情况。如果我运行此应用程序，它会在注释行上抛出ArgumentOutOfRangeException。它还表明我等于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的值。 / p>

按如下方式更改循环：

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

这里有什么问题？

Closure。您正在捕获i变量，该变量未按预期执行。

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

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

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

2 个答案: