当你使用标准http库发起请求时,你得到一个http的响应变量。如果你不读取响应主体,你依旧需要关闭它。对于新的Go开发者而言,这个很容易就会忘掉。

错误的做法

resp可能为nil

resp, err := http.Get(url)
defer resp.Body.Close()//may panic
if err != nil {
    return err
}

err不为nil,resp也不为nil

有些同学可能以为这样子修改代码就可以了

resp, err := http.Get(url)
if err != nil {
    return err
}
defer resp.Body.Close()

这对于绝大多数情况是可以的,因为err不为nil往往意味着resp为nil,但是当遇到一个重定向错误时,err不为nil,resp也不为nil,这种做法将导致内存泄漏!

正确的做法

resp, err := http.Get(url)
if nil != resp {
    defer resp.Body.Close()
}

if err != nil {
    return err
}

crazy
102 声望2 粉丝