Go使用全局变量要小心闭包
zenlife ·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的。
David
Life is hard but please enjoy it!
这个问题和闭包没什么关系。
var logger = AppLog()
这句话使全局变量的初始化,会在`main`执行之前被执行,此时`InitConf`显然还没有执行,从而`__logger`必然是`nil`,通过`AppLog()`返回给了`logger`导致其为`nil`。之后`InitConf`虽然把`__logger`初始化了,但是不会改变`logger`的值。
正确的做法是把`InitConf`改名成`init`,变成 package 初始化的一部分。
#2
更多评论
执行main函数之前,所有的全局变量应该已经被初始化了,此时 logger为nil(因为__logger 为 nil)。即使 在main函数里,InitConf 执行后, __logger 非nil,但是 logger 仍然是nil。另外,在 http.HandleFunc 之前,调用logger.Debug() ,必然panic,因为 logger为nil。
#1
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的。