我想使用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;
}
这里有什么问题?
解决方法:
问题是i == 10,但你的列表有10个项目(即最大索引为9).
这是因为您在委托运行之前更改了捕获的变量的竞争条件.循环的下一次迭代是否会在委托运行之前递增值,还是在循环递增值之前运行委托?这完全取决于特定运行的时间.
你的直觉是我的值为0-9.但是,当循环到达终止时,我的值为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的“私有”独立副本,而不是引用在委托的所有调用之间共享的单个更改值.
我不会太担心调试和非调试模式之间的行为差异.这就是种族条件的本质.