使用Go语言json包解码指针值后是否需要添加nil检查？

首先，我们来拆解你的代码逻辑，看看这个检查到底有没有可能生效，以及它是否多余：

核心场景：当JSON请求体是null时，这个检查会触发 #

你的request是一个*models.Owner类型的指针，初始值为nil。当你调用json.Decode(&request)时，你传递的是指针的指针（**models.Owner）。根据Go标准库的encoding/json规则：

• 如果请求体是合法的null值，Decode会将request设置为nil，并且返回err == nil（因为解析null到指针类型是完全合法的操作）。
• 这时候，代码会跳过err != nil的分支，走到request == nil的检查，触发对应的错误处理。

而其他场景下，这个检查确实不会被执行：

• 当JSON解析成功（包括空对象{}）：Decode会自动分配一个models.Owner实例，把request指向它，此时request非nil。
• 当JSON解析失败（格式错误、类型不匹配等）：err会被设置为非nil值，代码直接在第一个分支返回，不会走到后面的nil检查。

是否应该移除这个检查？ #

这取决于你的业务需求：

• 如果你的API不允许客户端发送null作为请求体（比如要求必须是一个合法的Owner对象），那这个检查是有意义的，可以拦截这种不符合预期的请求。
• 如果你的API允许null作为合法输入（比如用于表示“清空”或“重置”操作），那你应该移除这个检查，或者调整逻辑来处理request == nil的合法场景。

优化小建议 #

如果你希望彻底避免request为nil的情况，可以直接声明值类型的变量，而非指针：

var request models.Owner
err := json.NewDecoder(r.Body).Decode(&request)
if err != nil {
    render.Render(w, r, response.ErrInvalidRequest(err))
    return
}
// 此时request永远不会是nil，只会是零值实例，无需nil检查

这种方式更简洁，适合不需要区分“未传入数据”和“传入零值数据”的场景。

内容的提问来源于stack exchange，提问作者Cheemz