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这些值的地址逃逸出循环。使用匿名函数 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 不要在内部使用:= 来什么新变量.