如果在调用 http.Get(url) 时发生错误，我们是否需要关闭响应对象？

Question

在下面的代码中是否还需要在错误情况下关闭响应体：

res, err := http.Get(url)

if err != nil {

    log.Printf("Error: %s\n", err)

}

defer res.Body.Close()

Answer 1

一般概念是，当一个函数（或方法）有多个返回值时，一个是error，应该首先检查错误，只有在错误是 时才继续nil。如果存在error. 如果函数的行为不同，则应记录在案。http.Get()没有记录这种偏差。所以应该这样处理：res, err := http.Get(url)if err != nil {    log.Printf("Error: %s\n", err)    return}defer res.Body.Close()// Read/work with body笔记：正如 JimB 也确认的那样，如果nil返回非错误，即使响应是非错误nil，我们也不必关闭它。在重定向错误的情况下，无nil响应可能包含上下文和有关重定向失败的位置的进一步信息。请参阅下面的详细信息：http.Get()遵循“大部分时间”的一般概念：nil如果出现错误，它会返回响应：return nil, someError但是检查client.go，未导出的方法Client.doFollowingRedirects()，当前第 427 行：if redirectFailed {    // Special case for Go 1 compatibility: return both the response    // and an error if the CheckRedirect function failed.    // See https://golang.org/issue/3795    return resp, urlErr}因此，由于向后兼容性问题，如果重定向失败，它可能会同时返回无nil响应和nil无错误。另一方面，尝试调用resp.Body.Close()if respisnil会导致运行时恐慌。所以如果我们想在这种情况下关闭响应体，它可能看起来像这样（如果resp不是，只能关闭nil）：res, err := http.Get(url)if err != nil {    log.Printf("Error: %s\n", err)}if res != nil {    defer res.Body.Close()    // Read/work with body}或者：res, err := http.Get(url)if err != nil {    log.Printf("Error: %s\n", err)}if res == nil {    return}defer res.Body.Close()// Read/work with body即使没有响应数据也不会http.Response保证的文档：Response.Bodynil// The http Client and Transport guarantee that Body is always// non-nil, even on responses without a body or responses with// a zero-length body.但如果错误不是nil，则不必关闭非nil响应主体。