一位同事实现了一个函数,该函数进行 HTTP 调用并将响应主体作为字符串返回。为简洁起见稍微简化一下(不,我们并没有真正忽略所有错误):
func getStuff(id string) string {
response, _ := http.Get(fmt.Sprintf("/some/url/%s", id))
body, _ := ioutil.ReadAll(response.Body)
return string(body)
}
响应通常相当大,所以我想避免不必要的复制。据我了解,正如所写的那样,我们正在制作响应数据的三份副本:
io.ReadAll将传入 HTTP 连接中的数据复制到字节片。
string(body)将字节切片复制到字符串中。
return制作字符串的新副本以供调用函数使用。
那么,首先,我是否正确理解了当前状态?
简单的第一步是返回一个指针:
response, _ := http.Get(fmt.Sprintf("/some/url/%s", id))
body, _ := ioutil.ReadAll(response.Body)
result := string(body)
return &result
这避免了第三个副本。凉爽的。但我仍在制作两份数据副本,我只想制作一份。
我可以让他将返回类型更改为*[]byte,然后我们就可以了return &body。但是所有调用者都需要自己将结果转换为字符串,然后我所做的就是将制作第二个副本的逻辑传播到其他多个地方,而不是将其合并在这里。
我可以使用strings.Builder和io.Copy:
builder := new(strings.Builder)
_, _ := io.Copy(buf, response.Body)
result buf.String()
return &result
这可能会更有效一点(我真的不知道;是吗?),但我最终还是得到了两个数据副本。
是否可以只用一个数据副本来做到这一点?
我认为不是;只是想知道我是否错了!
月关宝盒
相关分类