当你使用标准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
}