索引超出范围错误,在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())
}
答案 0 :(得分:7)
i
确实变为2,但仅在循环的最后。但是因为你的goroutine函数关闭了i
,所以在goroutine启动时它并没有看到i
的值,而是{{1}的值在它执行时。换句话说,您的所有goroutine都共享相同的变量i
。
要解决此问题,一种方法是将i
复制到循环体内声明的辅助变量。一个更好的方法是将它作为参数传递给goroutine:
i