Go中有三种最容易犯的错误,我将以最常见的方式来展现而不是最容易理解的方式.
这三种错误都是在Kubernetes的代码中出现过的,这个之前在code review的时候最长见到
循环中的变量逃逸出循环的.
下面这些代码会输出什么呢?大家先预测一下然后再往下翻滚看答案
func print(pi *int) {
fmt.Println(*pi)
}
for i := 0; i < 10; i++ {
defer fmt.Println(i)
defer func(){ fmt.Println(i) }()
defer func(i int){ fmt.Println(i) }(i)
defer print(&i)
go fmt.Println(i)
go func(){ fmt.Println(i) }()
}
答案是:
func print(pi *int) {
fmt.Println(*pi)
}
for i := 0; i < 10; i++ {
defer fmt.Println(i) // 预期的结果; 打印 9 ... 0
defer func(){ fmt.Println(i) }() // 错误的结果; 打印10个"10"
defer func(i int){ fmt.Println(i) }(i) // 正确的预期结果
defer print(&i) // 错误的结果; 打印10个"10"
go fmt.Println(i) // 正确; 但是打印 0-9之间的顺序不保证
go func(){ fmt.Println(i) }() // 错误; 完全不可预知.
}
for key, value := range myMap {
// Same for key & value as i!
}
每个人期望这些值和循环内的值相同,但是Go重用了每次循环的变量. 这就意味着你绝对不要让key,value,i这些值的地址逃逸出循环。使用匿名函数 func() { /* do something with i */ } (a “closure”) 是一个微妙的解决地址逃逸问题的方法,因为在Go里面函数调用,这个参数是值copy。
Nil interface 如果有nil 指针
type Cat interface {
Meow()
}
type Tabby struct {}
func (*Tabby) Meow() {
fmt.Println("meow")
}
func GetACat() Cat {
var myTabby *Tabby = nil
// Oops, we forgot to set myTabby to a real value
return myTabby
}
func TestGetACat(t *testing.T) {
if GetACat() == nil {
t.Errorf("Forgot to return a real cat!")
}
}
大家猜猜会怎么样? 上面的这个测试讲永远不会检测到空指针。这是因为interface作为一个指针的容器来使用了,因此GetACat可以返回一个指针或者空指针. 所以最好不要像GetACat这样来写代码,让你和你的小伙伴们都感到开心。而且这种错误在err里面也是经常见.详见 http://golang.org/doc/faq#nil_error
影子变量.
var ErrDidNotWork = errors.New("did not work")
func DoTheThing(reallyDoIt bool) (err error) {
if reallyDoIt {
result, err := tryTheThing()
if err != nil || result != "it worked" {
err = ErrDidNotWork
}
}
return err
}
上面的这个代码将永远返回nil的错误, 因为循环内的err变量是一个函数返回err变量的影子变量. 所以上面的代码解决方案是添加一个 var result string 不要在内部使用:= 来什么新变量.