Go使用全局变量要小心闭包

zenlife · 2015-03-16 11:43:33 · 29777 次点击 · 预计阅读时间 1 分钟 · 大约8小时之前 开始浏览    
这是一个创建于 2015-03-16 11:43:33 的文章,其中的信息可能已经有所发展或是发生改变。

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的。


有疑问加站长微信联系(非本文作者)

本文来自:zenlife的博客

感谢作者:zenlife

查看原文:Go使用全局变量要小心闭包

入群交流(和以上内容无关):加入Go大咖交流群,或添加微信:liuxiaoyan-s 备注:入群;或加QQ群:692541889

29777 次点击  ∙  1 赞  
加入收藏 微博
5 回复  |  直到 2020-03-18 16:27:58
ckrissun
ckrissun · #1 · 10年之前

执行main函数之前,所有的全局变量应该已经被初始化了,此时 logger为nil(因为logger 为 nil)。即使 在main函数里,InitConf 执行后, logger 非nil,但是 logger 仍然是nil。另外,在 http.HandleFunc 之前,调用logger.Debug() ,必然panic,因为 logger为nil。

David
David · #2 · 10年之前

这个问题和闭包没什么关系。

var logger = AppLog()

这句话使全局变量的初始化,会在main执行之前被执行,此时InitConf显然还没有执行,从而__logger必然是nil,通过AppLog()返回给了logger导致其为nil。之后InitConf虽然把__logger初始化了,但是不会改变logger的值。

正确的做法是把InitConf改名成init,变成 package 初始化的一部分。

hexicong
hexicong · #3 · 9年之前
DavidDavid #2 回复

这个问题和闭包没什么关系。 var logger = AppLog() 这句话使全局变量的初始化,会在`main`执行之前被执行,此时`InitConf`显然还没有执行,从而`__logger`必然是`nil`,通过`AppLog()`返回给了`logger`导致其为`nil`。之后`InitConf`虽然把`__logger`初始化了,但是不会改变`logger`的值。 正确的做法是把`InitConf`改名成`init`,变成 package 初始化的一部分。

正解,我差点相信楼主

leenzhu
leenzhu · #4 · 6年之前

楼主一本正经的胡说八道 :laughing:

tianqy
tianqy · #5 · 5年之前

和闭包没任何关系,就是全量初始化顺序的问题,var logger = AppLog()优先于InitConf()函数执行,等InitConf()执行后,logger并没有更新取值,故触发宕机,要么在InitConf中重复执行logger = AppLog(),要么修改logger和AppLog使用二级指针

添加一条新回复 (您需要 登录 后才能回复 没有账号 ?)
  • 请尽量让自己的回复能够对别人有帮助
  • 支持 Markdown 格式, **粗体**、~~删除线~~、`单行代码`
  • 支持 @ 本站用户;支持表情(输入 : 提示),见 Emoji cheat sheet
  • 图片支持拖拽、截图粘贴等方式上传