Question

以下代码：

package main

import (
    "fmt"
    "strings"
)

var data = []string{
    "The yellow fish swims slowly in the water",
    "The brown dog barks loudly after a drink ...",
    "The dark bird bird of prey lands on a small ...",
}

func main() {
    histogram := make(map[string]int)
    words := make(chan string)

    for _, line := range data {
        go func(l string) {
            for _, w := range strings.Split(line, " ") {
                words <- w
            }
        }(line)
    }

    defer close(words)
    for w := range words {
        histogram[w]++
    }

    fmt.Println(histogram)
}

以死锁结束：

fatal error: all goroutines are asleep - deadlock!

goroutine 1 [chan receive]:
main.main()
    /tmp/sandbox780076580/main.go:28 +0x1e0

我的理解是频道词会阻止作者和读者实现某些同步。我正在尝试对所有goroutine（编写器）和main中的单个读取器使用单个通道（使用“ range”命令）。我也尝试过缓冲通道-类似的失败。我有问题要弄清楚为什么这不起作用。有了解技巧吗？
谢谢。

Answer 1

如对问题的评论中所述，推迟直到主程序返回才执行。结果，words上的范围将永远被阻止。

要解决此问题，所有goroutines发送完成后，应用程序必须关闭words。一种方法是使用wait group。每个goroutine的等待组增加，goroutine退出时，等待组减少。另一个goroutine在组中等待并关闭通道。

func main() {
    histogram := make(map[string]int)
    words := make(chan string)

    var wg sync.WaitGroup
    for _, line := range data {
        wg.Add(1)
        go func(l string) {
            for _, w := range strings.Split(l, " ") {
                words <- w
            }
            wg.Done()
        }(line)
    }

    go func() {
        wg.Wait()
        close(words)
    }()

    for w := range words {
        histogram[w]++
    }

    fmt.Println(histogram)
}

奖金修复：问题中的goroutine引用了循环变量iine而不是参数l。常见问题解答explains why this is a problem。

了解golang频道：死锁

1 个答案: