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

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

在此代码段中，remember变量永远不会在if语句外面变成true，如果something为true，由于使用了短声明:=，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. }

链接