gometalinter/errcheck 在延迟返回变量的函数时返回警告

Question

gometalinter 和 errcheck 向我返回有关延迟返回变量的函数的警告。

网络请求中的示例:

defer r.Body.Close()

在这种情况下，Close 会返回一个错误变量，并且不会对其进行检查。

将它延迟到另一个函数中是最好的方法/惯用方法吗？

defer func() {
    err := r.Body.Close()
    if err != nil {
        // fmt, panic or whatever
    }
}()

Answer 1

如果延迟函数有任何返回值，它们将在函数完成时被丢弃(有关更多详细信息，请查看 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() 的最后一个函数值将首先执行。