Golang定时递归调用的内存问题：两种实现方案优劣分析

我写了一个需要每5秒调用一次的缓存刷新函数，最初实现如下：

func (s *cacheService) ReloadCache() {
    // 执行缓存刷新逻辑...

    <- time.After(time.Second * 5)

    s.ReloadCache()
}

这种递归写法看起来比循环简洁，但我担心会有内存问题或者调用栈阻塞的情况。想问问下面用goroutine的实现是不是更优：

func (s *cacheService) ReloadCache() {
    // 执行缓存刷新逻辑...

    <- time.After(time.Second * 5)

    go s.ReloadCache()
}

两种实现的核心问题 #

• 第一种递归实现的风险：
  Go的栈虽然支持动态扩容，但这种无终止的递归会持续堆积调用栈帧。每次调用ReloadCache都会在栈上生成新的帧，随着运行时间增加，栈内存会持续膨胀，最终可能触发内存溢出。而且整个调用链处于阻塞状态，只要初始goroutine不退出，所有递归产生的栈帧都无法被回收，必然导致长期内存泄漏。

• 第二种goroutine实现的问题：
  这种写法避免了栈溢出，但每次递归都会启动新的goroutine，导致goroutine数量无限增长。每个goroutine初始占用约2KB栈内存，长期运行后会消耗大量内存，同时增加调度器的调度负担，拖慢程序整体性能。

更稳妥的实现方案 #

最合理的写法是循环+定时器，既不会堆积栈帧，也不会无限创建goroutine：

func (s *cacheService) ReloadCache() {
    ticker := time.NewTicker(5 * time.Second)
    defer ticker.Stop()

    for {
        // 执行缓存刷新逻辑...

        <- ticker.C
    }
}

这种写法在单个goroutine内循环执行，复用同一个定时器，内存占用稳定，没有栈或goroutine膨胀的问题。如果需要支持主动停止，还可以添加退出通道：

func (s *cacheService) ReloadCache(stop chan struct{}) {
    ticker := time.NewTicker(5 * time.Second)
    defer ticker.Stop()

    for {
        select {
        case <- ticker.C:
            // 执行缓存刷新逻辑...
        case <- stop:
            return
        }
    }
}

内容的提问来源于stack exchange，提问作者evrei666