gometalinter
和 errcheck
向我返回有关延迟返回变量的函数的警告。
网络请求中的示例:
defer r.Body.Close()
在这种情况下,Close
会返回一个错误变量,并且不会对其进行检查。
将它延迟到另一个函数中是最好的方法/惯用方法吗?
defer func() {
err := r.Body.Close()
if err != nil {
// fmt, panic or whatever
}
}()
最佳答案
如果延迟函数有任何返回值,它们将在函数完成时被丢弃(有关更多详细信息,请查看 Spec: Defer statements)。
所以检查返回值的唯一方法是存储它,并且只有在不是函数本身被延迟,而是另一个调用它的函数的情况下才有可能。
一种方法是像您一样使用匿名函数,这可能会稍微简化:
defer func() {
if err := r.Body.Close(); err != nil {
fmt.Println("Error when closing:", err)
}
}()
或者你可以为它创建一个辅助函数:
func Check(f func() error) {
if err := f(); err != nil {
fmt.Println("Received error:", err)
}
}
并使用它:
defer Check(r.Body.Close)
辅助函数当然可以多次使用,例如:
defer Check(r.Body.Close)
defer Check(SomeOtherFunc)
您还可以为此创建一个修改后的辅助函数,它可以接受多个函数:
func Checks(fs ...func() error) {
for i := len(fs) - 1; i >= 0; i-- {
if err := fs[i](); err != nil {
fmt.Println("Received error:", err)
}
}
}
并使用它:
defer Checks(r.Body.Close, SomeOtherFunc)
请注意,我有意在 Checks()
中使用向下循环来模仿延迟函数执行的先进后出 性质,因为最后一个defer
将首先执行,因此使用向下循环,传递给 Checks()
的最后一个函数值将首先执行。
关于gometalinter/errcheck 在延迟返回变量的函数时返回警告,我们在Stack Overflow上找到一个类似的问题: https://stackoverflow.com/questions/40397781/