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

Question

gometalinter并errcheck返回一个关于延迟返回变量的函数的警告.

Web请求中的示例:

defer r.Body.Close()

在这种情况下,Close返回一个错误变量并且不会检查它.

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

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

Answer 1

如果延迟函数具有任何返回值,则在函数完成时将丢弃它们(有关更多详细信息,请检查Spec:Defer语句).

因此,检查返回值的唯一方法是存储它,并且只有在函数本身不被延迟的情况下才有可能,但是只有另一个函数可以调用它.

一种方法是使用匿名函数,这可能会略微简化:

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()将被执行第一.