2015-03-16
Go使用全局变量要小心闭包
下面是一个简单复现:
package main
import (
"fmt"
"net/http"
)
type Logger struct{}
func (this *Logger) Debug() {
if this == nil {
panic("fuck")
}
fmt.Println("hello world")
}
var __logger *Logger
func AppLog() *Logger {
return __logger
}
func InitConf() {
__logger = &Logger{}
}
// ------内上内容应该是写在log包中,这里是方便演示------
var logger = AppLog()
func HelloWorld(w http.ResponseWriter, r *http.Request) {
AppLog().Debug() // ok
logger.Debug() // panic
}
func main() {
InitConf()
http.HandleFunc("/", HelloWorld)
http.ListenAndServe(":8088", nil)
}
会崩。为什么?明明有调InitConf初始化过全局变量__logger
呀!
都是闭包惹的祸。http.HandlerFunc其实是闭包。
最终HelloWorld中使用的logger不是全局变量的logger,而是闭包里的自由变量。
在闭包生成的时候InitConf()还没被调用,所以__logger
是nil的,于是闭包里面的自由变量logger是nil的。
有疑问加站长微信联系(非本文作者)

执行main函数之前,所有的全局变量应该已经被初始化了,此时 logger为nil(因为logger 为 nil)。即使 在main函数里,InitConf 执行后, logger 非nil,但是 logger 仍然是nil。另外,在 http.HandleFunc 之前,调用logger.Debug() ,必然panic,因为 logger为nil。
这个问题和闭包没什么关系。
这句话使全局变量的初始化,会在
main
执行之前被执行,此时InitConf
显然还没有执行,从而__logger
必然是nil
,通过AppLog()
返回给了logger
导致其为nil
。之后InitConf
虽然把__logger
初始化了,但是不会改变logger
的值。正确的做法是把
InitConf
改名成init
,变成 package 初始化的一部分。正解,我差点相信楼主
楼主一本正经的胡说八道
和闭包没任何关系,就是全量初始化顺序的问题,var logger = AppLog()优先于InitConf()函数执行,等InitConf()执行后,logger并没有更新取值,故触发宕机,要么在InitConf中重复执行logger = AppLog(),要么修改logger和AppLog使用二级指针