返回 登录
0

Go语言的三个雷区

Go语言中有三个常犯的错误,我将他们以最容易犯的方式记下来。注意这里用的例子是最容易碰到的,而不是最容易理解的。

下面的例子都是在Kubernetes代码中的,而且据我所知,都不止一次地通过了代码审查。

1. 循环中变量的作用域问题

观察下面的代码,预测其功能是什么?

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) // OK; prints 9 ... 0
  defer func(){ fmt.Println(i) }() // WRONG; prints "10" 10 times
  defer func(i int){ fmt.Println(i) }(i) // OK
  defer print(&i) // WRONG; prints "10" 10 times
  go fmt.Println(i) // OK; prints 0 ... 9 in unpredictable order
  go func(){ fmt.Println(i) }() // WRONG; totally unpredictable.
}

for key, value := range myMap {
  // Same for key & value as i!
}

大家都认为这些变量能正常工作,但实际上,Go语言随着迭代会重用相同的内存。这意味着,你永远不能让key,value,i的值在循环外使用。匿名函数func() { /* do something with i */ }(一个“闭包”)是解决这个问题的巧妙办法。

2. Nil接口并不是有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!")
  }
}

猜到了吗?上面的代码不会发现Nil指针! 这是因为,接口作为一个指针,所以GetACat实际上将一个指针返回给了空指针。永远不要写上面这种代码,这样你的同事会很高兴的。用错误值来代替更好。(http://golang.org/doc/faq#nil_error

3. 有害的变量名

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变量覆盖了函数的作用域变量,使用 var result string,不使用:=可以解决这个问题。

原文The Three Go Landmines.markdown
作者:lavalamp 翻译:赖信涛 责编:仲培艺

评论