执行例程不从通道收集所有对象答案

【问题标题】：go routine not collecting all objects from channel执行例程不从通道收集所有对象
【发布时间】：2018-03-21 17:49:52
【问题描述】：

我有一个go-routine 将对象添加到通道中，然后我有4 个go-routines 来处理通道的对象。处理只不过是将对象添加到数组中。但有时，最终数组中缺少对象。所以我假设通道在某些时候会停止收集对象。我有以下代码：

package main

import (
    "log"
    "sync"
)

func main() {
    j := 0
    for {
        if j == 10 {
            break
        }
        wg := sync.WaitGroup{}
        months := []string{"Jan", "Feb", "Mar", "Apr", "May", "Jun", "Jul"}
        hits := make(chan string)
        i := 0
        wg.Add(1)
        go func() {
            defer close(hits)
            for {
                if i == 25 {
                    wg.Done()
                    return
                }
                for _, month := range months {
                    hits <- month
                }
                i++
            }
        }()

        temp := []string{}
        for updateWorker := 1; updateWorker <= 4; updateWorker++ {
            wg.Add(1)
            go func() {
                for hit := range hits {
                    temp = append(temp, hit)
                }
                wg.Done()
                return
            }()
        }

        wg.Wait()

        log.Printf("length of temp %+v\n", len(temp))
        j++
    }
}

我正在使用sync 库来同步例程。我将相同的过程循环 10 次以测试输出是否一致。我期待这样的输出：

length of temp 175

这是 175，因为我要发送 7 个月的字符串 25 次。但是有时输出小于175，我不知道为什么。我对 goroutines 有点初学者。那么有人可以帮我在这里找到原因吗？谢谢。

【问题讨论】：

标签： go concurrency channel goroutine

【解决方案1】：

问题是updateWorker goroutines 都从hits 通道收集结果（到目前为止一切都很好），它们都将结果存储到temp 局部变量未同步。这不行。

必须同步对来自多个 goroutine（其中至少一个是写入）的所有变量的访问。

如果您在启用竞争检测器的情况下运行它，它会尖叫数据竞争 (go run -race app.go)。

如果您将updateWorker goroutine 的数量减少到 1，它会立即产生有效结果，因为这样我们就消除了您应用的单一数据竞争源：

for updateWorker := 1; updateWorker <= 1; updateWorker++ {
    // ...
}

如果你想保留多个updateWorker goroutine，它们对共享temp 变量的访问必须同步。

sync.Mutex:

var (
    mu   sync.Mutex
    temp []string
)
for updateWorker := 1; updateWorker <= 4; updateWorker++ {
    wg.Add(1)
    go func() {
        for hit := range hits {
            mu.Lock()
            temp = append(temp, hit)
            mu.Unlock()
        }
        wg.Done()
        return
    }()
}

还要注意，在这个简单的示例中，使用多个 updateWorker goroutines 没有任何好处，与仅使用其中一个相比，添加上述同步（锁定）甚至会降低性能。

为了正确分配工作和收集结果，请查看此答案：Is this an idiomatic worker thread pool in Go?

【讨论】：