gometalinter / errcheck returns 关于延迟 returns 变量的函数的警告

Question

gometalinter 和 errcheck return 给我一个关于延迟 return 变量的函数的警告。

Web 请求中的示例：

defer r.Body.Close()

在这种情况下，Close return 是一个错误变量，未被检查。

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

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

Answer 1

如果延迟函数有任何 return 值，它们将在函数完成时被丢弃（有关更多详细信息，请查看 Spec: Defer statements）。

所以检查 return 值的唯一方法是存储它，并且只有在不是函数本身被延迟，而是调用它的另一个函数时才有可能。

一种方法是像您一样使用匿名函数，这可能会稍微简化：

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 returns 关于延迟 returns 变量的函数的警告

gometalinter / errcheck returns a warning on deferring a func which returns a variable

return-value

go

deferred-execution