gometalinter / errcheck返回有关推迟返回变量的func的警告

一尘不染

gometalinter / errcheck返回有关推迟返回变量的func的警告

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

Web请求中的示例：

defer r.Body.Close()

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

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

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

阅读 212

2020-07-02

共1个答案

一尘不染

如果延迟函数具有任何返回值，则在函数完成时将其丢弃（有关更多详细信息，请参见Spec：Defer
statement）。

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

一种实现方法是像您一样使用匿名函数，该函数可能会稍作简化：

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()执行传递给函数的值第一。

2020-07-02