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()的最后一个函数值。

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

1 个答案: