for循环+goroutine的坑

01. 调查

初学者经常会使用如下代码来并行处理数据：

for val := range values {
    go val.MyMethod()
}

或者使用闭包（closure）：

for val := range values {
    go func() {
        fmt.Println(val)
    }()
}

这里的问题在于 val 实际上是一个遍历了切片中所有数据的单一变量。由于闭包只是绑定到这个 val 变量上，因此极有可能上面的代码的运行结果是所有 goroutine 都输出了切片的最后一个元素。这是因为很有可能当 for-loop 执行完之后 goroutine 才开始执行，这个时候 val 的值指向切片中最后一个元素。

02. 解决方法

以上代码正确的写法为：

for val := range values {
    go func(val interface{}) {
        fmt.Println(val)
    }(val)
}

在这里将 val 作为一个参数传入 goroutine 中，每个 val 都会被独立计算并保存到 goroutine 的栈中，从而得到预期的结果。

另一种方法是在循环内定义新的变量，由于在循环内定义的变量在循环遍历的过程中是不共享的，因此也可以达到同样的效果：

for i := range valslice {
    val := valslice[i]
    go func() {
        fmt.Println(val)
    }()
}

 转载：

https://segmentfault.com/a/1190000010884717

Common Mistake - Using goroutines on loop iterator variables