在尝试启动多个线程时,索引超出了数组的范围

时间:2013-05-30 18:34:16

标签: c# arrays multithreading indexoutofboundsexception

我有这个代码,它给了我一个“索引超出了数组的范围”。我不知道为什么会发生这种情况,因为变量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();
    }
}

有人能告诉我如何解决这个问题,为什么会发生这种情况。谢谢!

2 个答案:

答案 0 :(得分: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别无选择,只能在循环结束前接受它。这将为您提供预期的结果,也不例外。

答案 1 :(得分: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();
    }
}