ThreadPool 在调试模式和运行时表现不同
ThreadPool behaves different for debug mode and runtime
我想使用 ThreadPool 在更短的时间内完成较长的 运行ning 作业。我的方法
当然做更多的工作,但我准备了一个简单的例子让你理解
我的情况。如果我 运行 这个应用程序,它会在注释行上抛出 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;
}
这里有什么问题?
问题是 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);
}
现在您的委托获得 "private",i
的独立副本,而不是引用在委托的所有调用之间共享的单个不断变化的值。
我不会太担心调试模式和非调试模式之间的行为差异。这就是竞争条件的本质。
What is the problem here?
Closure。您正在捕获 i
变量,它没有按照您的预期执行。
您需要在 for
循环中创建一个副本:
var currentIndex = i:
Sum(list1[currentIndex], list2[currentIndex]);
我想使用 ThreadPool 在更短的时间内完成较长的 运行ning 作业。我的方法 当然做更多的工作,但我准备了一个简单的例子让你理解 我的情况。如果我 运行 这个应用程序,它会在注释行上抛出 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;
}
这里有什么问题?
问题是 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);
}
现在您的委托获得 "private",i
的独立副本,而不是引用在委托的所有调用之间共享的单个不断变化的值。
我不会太担心调试模式和非调试模式之间的行为差异。这就是竞争条件的本质。
What is the problem here?
Closure。您正在捕获 i
变量,它没有按照您的预期执行。
您需要在 for
循环中创建一个副本:
var currentIndex = i:
Sum(list1[currentIndex], list2[currentIndex]);