go 例程未从通道收集所有对象

我有一个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，不知道为什么。我对 Go 例程有点初学者。那么有人可以帮我找到原因吗？谢谢。

问题是updateWorkergoroutine 都从以下位置收集结果hits通道（到目前为止一切顺利），并且它们都将结果存储到temp局部变数不同步。这是不行的。

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

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

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

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

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

With a sync.Mutex https://golang.org/pkg/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对于 goroutine，添加上述同步（锁定）甚至会使其性能低于仅使用其中之一的性能。

为了正确分配工作和收集结果，请查看以下答案：这是 Go 中惯用的工作线程池吗？ https://stackoverflow.com/questions/38170852/is-this-an-idiomatic-worker-thread-pool-in-go/38172204#38172204