作为一个 golang 新手,时常遇到一个编译报错:
err is shadowed during return
比如说下面这段代码:
func fun() (err error) {
if true {
x, err := makeErr()
if err != nil {
return
}
fmt.Println(x, err.Error())
}
return
}
func makeErr() (x int, err error) {
err = fmt.Errorf("error")
return
}
因为内层初始化的 err 不是 fun 函数的返回值。在返回时,内层的 err 就会被忽视。
如果将 if 语句去掉,编译错误会消失。
func fun() (err error) {
x, err := makeErr()
if err != nil {
return
}
return
}
当然,这样的话代码的逻辑也改变了。通常来说,处理方法是新建一个 err 变量,再赋值:
func fun() (err error) {
if true {
x, err1 := makeErr()
if err1 != nil {
err = err1
return
}
fmt.Println(x, err1.Error())
}
return
}
但是个人觉得这种写法比较丑陋。查阅 go 源码,可以发现源码中的通常做法是:
func fun() error {
var err error
if true {
x, err := makeErr()
if err != nil {
return err
}
fmt.Println(x, err.Error())
}
return err
}
看来返回值里定义的变量并不能当作一般的局域变量看待啊。