3

我有这段代码，它给了我一个“索引超出了数组的范围”。我不知道为什么会这样，因为变量i应该总是小于数组的长度，bla因此不会导致这个错误。

private void buttonDoSomething_Click(object sender, EventArgs e)
{
    List<Thread> t = new List<Thread>();

    string[] bla = textBoxBla.Lines;

    for (int i = 0; i < bla.Length; i++)
    {
        t.Add(new Thread (() => some_thread_funmction(bla[i])));
        t[i].Start();
    }
}

有人可以告诉我如何解决这个问题以及为什么会这样。谢谢！

4

2 回答 2

10

闭包是你的问题。

基本上，不是在创建 lambda（在循环中）时获取值，而是在需要时获取它。计算机的速度如此之快，以至于当它发生时，它已经脱离了循环。值为 3。这是一个示例（先不要运行它）：

private void buttonDoSomething_Click(object sender, EventArgs e)
{
    List<Thread> t = new List<Thread>();
    for (int i = 0; i < 3; i++)
    {
        t.Add(new Thread (() => Console.Write(i)));
        t[i].Start();
    }
}

想想你期望的结果是什么。会不会是012你在想？

现在运行它。

结果将是333。

这是一些修改后的代码，可以修复它：

private void buttonDoSomething_Click(object sender, EventArgs e)
{
    List<Thread> t = new List<Thread>();
    string[] bla = textBoxBla.Lines;
    for (int i = 0; i < bla.Length; i++)
    {
        int y = i; 
        //note the line above, that's where I make the int that the lambda has to grab
        t.Add(new Thread (() => some_thread_funmction(bla[y]))); 
        //note that I don't use i there, I use y.
        t[i].Start();
    }
}

现在它可以正常工作了。这一次，当循环结束时，值超出范围，因此 lambda 没有选择，只能在循环结束之前获取它。这将为您带来预期的结果，也不例外。

于 2013-05-30T18:39:47.347 回答
2

您所看到的是竞争条件。您的for循环在线程实际开始之前完成。因此，当线程实际启动时， 的值i超出了数组的范围。

尝试复制索引值并改为传递副本。

private void buttonDoSomething_Click(object sender, EventArgs e)
{
    List<Thread> t = new List<Thread>();

    string[] bla = textBoxBla.Lines;

    for (int i = 0; i < bla.Length; i++)
    {
        int index = i;
        t.Add(new Thread (() => some_thread_funmction(bla[index])));
        t[i].Start();
    }
}
于 2013-05-30T18:39:55.243 回答