指数超出范围goroutine golang

时间:2018-03-11 19:26:30

标签: go goroutine

索引超出范围错误,在for循环生成的goroutine中似乎不可能

这是我正在执行的代码。 问题是索引超出范围错误是在对goroutine中countlinks调用的一个参数的评估中产生的。我们正在迭代的切片的长度总是2.基于我对go中的for循环的理解,i永远不应该评估为2(在表达式grph.entryPoints[i]中产生超出范围的索引),并且但事实确实如此。请告诉我我是多么疯狂。

func main() {
    grph := structures.ConfigGraphDefault()
    counter := structures.NewCounter()
    queue := structures.NewQueue()
    tracker := structures.NewTracker()
    fmt.Printf("entries: %v\nnodes: %v\n", grph.EntryPoints, grph.Nodes)
    wg := sync.WaitGroup{}
    fmt.Println(len(grph.EntryPoints))
    // this will always evaluate to 2
    l := len(grph.EntryPoints)
    for i := 0; i < l; i++ {
        wg.Add(1)
        go func() {
            fmt.Printf("index: %v, length of slice: %v\n", i, len(grph.EntryPoints))
            structures.CountLinks(&grph, counter, queue, tracker, grph.EntryPoints[i], i)
            wg.Done()
        }()
    }
    wg.Wait()
    fmt.Printf("counter: %v", counter.GetCounts())
}

1 个答案:

答案 0 :(得分:7)

i确实变为2,但仅在循环的最后。但是因为你的goroutine函数关闭了i ,所以在goroutine启动时它并没有看到i的值,而是{{1}的值在它执行时。换句话说,您的所有goroutine都共享相同的变量i

要解决此问题,一种方法是将i复制到循环体内声明的辅助变量。一个更好的方法是将它作为参数传递给goroutine:

i