Golang app使用sync.WaitGroup&渠道永不退出

时间:2015-12-14 22:03:28

标签: go goroutine

我使用sync.WaitGroupdefer wg.Close()wg.Wait()等待我的goroutines完成。

程序会等待,但它永远不会退出。

这是我的程序(可运行):

package main

import (
    "fmt"
    "io"
    "log"
    "net/http"
    "os"
    "sync"
)

var symbols = []string{
    "ASSA-B.ST",
    "ELUX-B.ST",
    "HM-B.ST",
}

func main() {

    fmt.Println("fetching quotes...")

    fetchedSymbols := make(chan string)
    var wg sync.WaitGroup
    wg.Add(len(symbols))

    for _, symbol := range symbols {
        go fetchSymbol(symbol, &wg, fetchedSymbols)
    }

    for response := range fetchedSymbols {
        fmt.Println("fetched " + response)
    }

    wg.Wait()

    fmt.Println("done")

}

func fetchSymbol(symbol string, wg *sync.WaitGroup, c chan<- string) {
    defer wg.Done()
    resp, err := http.Get("http://ichart.yahoo.com/table.csv?s=" + symbol + "&a=0&b=1&c=2000")
    defer resp.Body.Close()

    if err != nil {
        log.Fatal(err)
    }

    out, err := os.Create("./stock-quotes/" + symbol + ".csv")
    defer out.Close()

    if err != nil {
        log.Fatal(err)
    }

    io.Copy(out, resp.Body)
    c <- symbol
}

下载所有引号后,该程序是否应该退出? (仅供参考:我刚开始学习GO)

1 个答案:

答案 0 :(得分:13)

您永远不会关闭fetchedSymbols频道,因此范围循环永远不会退出。

处理此问题的一种方法是使用您已经拥有的WaitGroup来发出何时关闭频道的信号。范围超过fetchedSymbols足以阻止主进程的进展,您不需要另一个频道或WaitGroup。

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

for response := range fetchedSymbols {
    fmt.Println("fetched " + response)
}

...