16.1 误用短声明导致变量覆盖

  1. var remember bool = false
  2. if something {
  3. remember := true //错误
  4. }
  5. // 使用remember

在此代码段中,remember 变量永远不会在 if 语句外面变成 true,如果 somethingtrue,由于使用了短声明 :=if 语句内部的新变量 remember 将覆盖外面的 remember 变量,并且该变量的值为 true,但是在 if 语句外面,变量 remember 的值变成了 false,所以正确的写法应该是:

  1. if something {
  2. remember = true
  3. }

此类错误也容易在 for 循环中出现,尤其当函数返回一个具名变量时难于察觉,例如以下的代码段:

  1. func shadow() (err error) {
  2. x, err := check1() // x 是新创建变量,err 是被赋值
  3. if err != nil {
  4. return // 正确返回 err
  5. }
  6. if y, err := check2(x); err != nil { // y 和 if 语句中 err 被创建
  7. return // if 语句中的 err 覆盖外面的 err,所以错误的返回 nil !
  8. } else {
  9. fmt.Println(y)
  10. }
  11. return
  12. }

链接