创建任务时列出索引超出范围异常

Question

确切的错误：

索引超出范围。必须为非负数且小于集合的大小。

我无数次索引数组和列表。我已经无数次使用带有数组和列表的循环。数据在那里，它有效。除非我尝试为我的函数创建任务。请注意，我使用 foreach 循环成功地做到了这一点，以实现类似的功能；这个新的需要两个参数，所以我不能正确使用 foreach 循环。至少我认为我做不到。

这是错误的代码：

if (addressList != null) {
    textBox1.Text += ("Address List Length: " + addressList.Count + Environment.NewLine);

    for (int i = 0; i < addressList.Count; i++) {
        textBox1.Text += ("Task for " + addressList[i] + ":" + portList[i] + " initiated." + Environment.NewLine);

        Task.Factory.StartNew(() => PingTaskAdapted(addressList[i], portList[i]));
    }                
}
else textBox1.Text = ("No IPs have been added.");

假设 addressList[0] 是 google.com，portList[0] 是 80， 输出：

Address List Length: 1
Task for google.com:80 initiated.

然后程序中断，Visual Studio 告诉我在 PingTaskAdapted() 我调用了一个超出范围的索引，而实际上它只是打印了有问题的索引，因为它们存在。

为了清楚起见，如果我打电话给PingTaskAdapted(addressList[0], pingList[0]);，它可以正常工作。

Answer 1

你是访问修改闭包的受害者，因为它被简洁地称为。基本上，由于您使用的是任务 - 以及启动的委托 - i 的值不能保证是您期望的值。但是，如果您将 i 复制到一个局部变量，特定于一次迭代的范围，您应该没问题。

for (int i = 0; i < addressList.Count; i++)
{
    textBox1.Text += ("Task for " + addressList[i] + ":" + portList[i] + " initiated." + Environment.NewLine);

    var iCopy = i;
    Task.Factory.StartNew(() => PingTaskAdapted(addressList[iCopy], portList[iCopy]));
}

但是，正如this answer by nvoigt 中所指出的，如果您复制将要使用的值而不是迭代器值，那么在可读性和可维护性方面会更加清晰。

Answer 2

闭包捕获变量，而不是值。

将代码更改为以下内容，您将看到问题消失：

for (int i = 0; i < addressList.Count; i++) {
    textBox1.Text += ("Task for " + addressList[i] + ":" + portList[i] + " initiated." + Environment.NewLine);

    var temp = i;
    Task.Factory.StartNew(() => PingTaskAdapted(addressList[temp], portList[temp]));
    }                

Answer 3

当任务运行时，您的任务将访问列表。在您在循环中查看的代码行中不是按顺序排列的。为了确保在闭包中捕获正确的值（并且列表仍然存在并且具有相同的值），请在任务之外制作本地副本，以确保在循环运行的那个时间点捕获值：

var localAddress = addressList[i];
var localPort = portList[i];
Task.Factory.StartNew(() => PingTaskAdapted(localAddress , localPort));