golang几个注意点

译者注:今天看到这个文章,感觉非常有用,我们平常写代码中也是经常遇到,所以翻译一下,让大家平常写Go代码的时候引起关注,同时在翻译的过程中也加入了一些自己的理解注释。

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这些值的地址逃逸出循环。使用匿名函数是一个微妙的解决地址逃逸问题的方法,因为在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变量的影子变量. 所以上面的代码解决方案是添加一个来什么新变量.