英文:
How to call Body.Close() when using require()?
问题
API测试中有以下样板代码。
我想在require.Equal(ru.ts.T(), tp.expectedStatus, resp.StatusCode)失败时调用resp.Body.Close()。
目前我有以下脚本。有更好的编写方式吗？
我想避免条件if tp.expectedStatus != resp.StatusCode，并在require.Equal(ru.ts.T(), tp.expectedStatus, resp.StatusCode)失败时调用resp.Body.Close()。
func Invoke(ip RestParams, es int) *http.Response {
client := &http.Client{}
// 使用ip初始化req
resp, err := client.Do(req)
...
if tp.expectedStatus != resp.StatusCode {
resp.Body.Close()
require.Equal(ru.ts.T(), tp.expectedStatus, resp.StatusCode)
return nil
}
return resp
}
英文:
API test has below boiler plate code.
I want to call resp.Body.Close() even when require.Equal(ru.ts.T(), tp.expectedStatus, resp.StatusCode) fails.
Currently i have script like below. Are there better ways to write this?
I want to avoid condition if tp.expectedStatus != resp.StatusCode and call resp.Body.Close() when require.Equal(ru.ts.T(), tp.expectedStatus, resp.StatusCode) fails.
func Invoke(ip RestParams, es int) *http.Response {
client := &http.Client{}
// Initialize req using ip
resp, err := client.Do(req)
...
if tp.expectedStatus != resp.StatusCode {
resp.Body.Close()
require.Equal(ru.ts.T(), tp.expectedStatus, resp.StatusCode)
return nil
}
return resp
}
答案1
得分: 1
你几乎总是应该执行以下操作：
resp, err := client.Do(req)
if err != nil {...} // 或者使用 require.NoError(err)
defer resp.Body.Close()
http包保证了一个非nil的响应体，在错误为nil时应立即关闭。
另外，我认为你应该避免返回http.Response。在这里对其进行解组，并返回一个结构体模型，这样你就可以在这个函数中处理所有的技术http层。
英文:
You should almost always do
resp, err := client.Do(req)
if err != nil {...} // Or require.NoError(err)
defer resp.Body.Close()
http package garanties a non nil body that should be closed as soon as the error is nil.
As a side note, I think you should avoid returning the http.Response. Unmarshal it here and return a struct model so you can handle all your technical http layer in this functio.
通过集体智慧和协作来改善编程学习和解决问题的方式。致力于成为全球开发者共同参与的知识库，让每个人都能够通过互相帮助和分享经验来进步。
评论