Question

我有一个通道，它接收从日志文件解析的事件，而另一个用于同步。出于测试目的，共有8个事件。

使用for range语法时，我得到4个事件。使用已知数字（8）时，我可以全部获得。

func TestParserManyOpinit(t *testing.T) {
    ch := make(chan event.Event, 1000)
    done := make(chan bool)
    go parser.Parse("./test_data/many_opinit", ch, done)
    count := 0
    exp := 8
    evtList := []event.Event{}

    <-done
    close(ch)
    //This gets all the events
    for i := 0; i < 8; i++ {
            evtList = append(evtList, <-ch)
            count++
    }

    //This only gives me four
    //for range ch {
    //        evtList = append(evtList, <-ch)
    //        count++
    //}

    if count != exp || count != len(evtList) {
            t.Errorf("Not proper lenght, got %d, exp %d, evtList %d", count, exp, len(evtList))
    }

func Parse(filePath string, evtChan chan event.Event, done chan bool) {
    log.Info(fmt.Sprintf("(thread) Parsing file %s", filePath))
    file, err := os.Open(filePath)
    defer file.Close()

    if err != nil {
            log.Error("Cannot read file " + filePath)
    }
    count := 0
    scan := bufio.NewScanner(file)
    scan.Split(splitFunc)
    scan.Scan() //Skip log file header

    for scan.Scan() {
            text := scan.Text()
            text = strings.Trim(text, "\n")
            splitEvt := strings.Split(text, "\n")
            // Some parsing ...
            count++
            evtChan <- evt
    }

    fmt.Println("Done ", count) // gives 8
    done <- true
}

我肯定缺少与通道上的for循环相关的东西。

我尝试在time.Sleep部分之前添加一个done <- true。它并没有改变结果。

Answer 1

使用for range时，每个循环迭代都从通道读取，并且您没有使用读取的值。因此，将丢弃一半的值。应该是：

for ev := range ch {
        evtList = append(evtList, ev)
        count++
}

为了真正利用循环迭代器中读取的值。

Tour of Go中展示了频道范围，而Go spec中则详细介绍了

。

对于范围与静态通道长度golang

1 个答案: