22

如果我在一个股票通道上进行范围并调用 stop() 通道将停止但未关闭。

在这个例子中:

package main

import (
    "time"
    "log"
)

func main() {
    ticker := time.NewTicker(1 * time.Second)
    go func(){
        for _ = range ticker.C {
            log.Println("tick")
        }
        log.Println("stopped")
    }()
    time.Sleep(3 * time.Second)
    log.Println("stopping ticker")
    ticker.Stop()
    time.Sleep(3 * time.Second)
}

运行产生:

2013/07/22 14:26:53 tick
2013/07/22 14:26:54 tick
2013/07/22 14:26:55 tick
2013/07/22 14:26:55 stopping ticker

所以那个 goroutine 永远不会退出。有没有更好的方法来处理这种情况?我应该关心 goroutine 从未退出吗?

4

4 回答 4

20

正如沃尔克建议的那样,使用了第二个频道。这就是我最终运行的结果:

package main

import (
    "log"
    "time"
)

// Run the function every tick
// Return false from the func to stop the ticker
func Every(duration time.Duration, work func(time.Time) bool) chan bool {
    ticker := time.NewTicker(duration)
    stop := make(chan bool, 1)

    go func() {
        defer log.Println("ticker stopped")
        for {
            select {
            case time := <-ticker.C:
                if !work(time) {
                    stop <- true
                }
            case <-stop:
                return
            }
        }
    }()

    return stop
}

func main() {
    stop := Every(1*time.Second, func(time.Time) bool {
        log.Println("tick")
        return true
    })

    time.Sleep(3 * time.Second)
    log.Println("stopping ticker")
    stop <- true
    time.Sleep(3 * time.Second)
}
于 2013-07-22T23:30:35.877 回答
14

在第二个通道上发出“done”信号,并在你的 goroutine 中选择ticker 和 done 通道。

根据您真正想做的事情,可能存在更好的解决方案,但这很难从简化的演示代码中看出。

于 2013-07-22T22:20:03.603 回答
7

你可以这样做。

package main

import (
    "fmt"
    "time"
)

func startTicker(f func()) chan bool {
    done := make(chan bool, 1)
    go func() {
        ticker := time.NewTicker(time.Second * 1)
        defer ticker.Stop()
        for {
            select {
            case <-ticker.C:
                f()
            case <-done:
                fmt.Println("done")
                return
            }
        }
    }()
    return done
}

func main() {
    done := startTicker(func() {
        fmt.Println("tick...")
    })
    time.Sleep(5 * time.Second)
    close(done)
    time.Sleep(5 * time.Second)
}
于 2015-08-11T07:55:30.073 回答
0

如果您需要节省更多空间,请使用struct{}不占用内存的空结构 ( ) 的通道。如上所述,不要在其中发送任何东西 - 只是关闭,它实际上发送零值。

于 2014-10-23T08:17:05.090 回答