了解golang频道:死锁

问题描述 投票:1回答:1

以下代码:

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”命令)。我也尝试过缓冲通道 - 类似的故障。我有问题要明白为什么这不起作用。理解的任何提示? 谢谢。

go deadlock
1个回答
1
投票

正如对问题的评论中所述,延迟在主要回归之前不会执行。结果,words的范围永远阻止。

要解决此问题,应用程序必须在所有goroutine完成发送后关闭words。一种方法是使用wait group。等待组针对每个goroutine递增,当goroutines退出时递减。还有另一个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

© www.soinside.com 2019 - 2024. All rights reserved.