promoted Unmarshal method on embedded field caused confusion 后续

encoding/json: promoted Unmarshal method on embedded field caused confusion

接上一篇，官方在早上给了回复：

简单解释下就是嵌入字段 Nested 的方法被提升了，导致 Object 的方法不会被执行，所以 Num 字段不会被 Unmarshal 。跟上一篇中的解释差不多意思。但是官方给了两种更加优雅的解决这个问题的方式，让我们来欣赏下大佬的代码。

方法 a

代码中只需添加下面一行即可：

var _ json.Unmarshaler = (*Object)(nil)
复制代码

package main

import (
 "encoding/json"
 "fmt"
 "time"
)

var testJSON = `{"num":5,"duration":"5s"}`

type Nested struct {
 Dur time.Duration `json:"duration"`
}

func (obj *Object) UnmarshalJSON(data []byte) error {
 tmp := struct {
  Dur string `json:"duration"`
  Num int    `json:"num"`
 }{}
 if err := json.Unmarshal(data, &tmp); err != nil {
  return err
 }

 dur, err := time.ParseDuration(tmp.Dur)
 if err != nil {
  return err
 }
 obj.Dur = dur
 obj.Num = tmp.Num
 return nil
}

type Object struct {
 Nested
 Num int `json:"num"`
}

var _ json.Unmarshaler = (*Object)(nil)

func main() {
 obj := Object{}
 _ = json.Unmarshal([]byte(testJSON), &obj)
 fmt.Printf("result: %+v \n", obj)
}
复制代码

随后这位老哥补充到，在嵌入字段都实现了接口方法的情况下， The type assertion will be a nice guide , 添加该类型的断言是一个好的实践，可以帮助你快速捕捉到潜在的 bug 。

方法 b

实现 custom time unmarshaller 。

package main

import (
 "encoding/json"
 "fmt"
 "time"
)

var testJSON = `{"num":5,"duration":"5s"}`

type customTimeDuration time.Duration

type Nested struct {
 Dur customTimeDuration `json:"duration"`
}

func (ctd *customTimeDuration) UnmarshalJSON(b []byte) error {
 var durStr string
 if err := json.Unmarshal(b, &durStr); err != nil {
  return err
 }
 dur, err := time.ParseDuration(durStr)
 if err == nil {
  *ctd = customTimeDuration(dur)
 }
 return err
}

type Object struct {
 Nested
 Num int `json:"num"`
}

func main() {
 obj := Object{}
 _ = json.Unmarshal([]byte(testJSON), &obj)
 fmt.Printf("result: %+v \n", obj)
}
复制代码

这种方式其实就是跟以上一篇分开解析的思路比较像，他重新声明了别名类型，然后为这个别名类型实现 UnmarshalJson 接口。个人倾向于第一种添加类型断言的方式，简洁又容易理解 ，对代码侵入比较小。

官方对这个问题的回复还是很热情的，他说他自己的团队在几年前也遇到了一模一样的问题，很能理解开发者的心情，他当时还针对这个问题写了一篇类似的文章，https://medium.com/@odeke_et/compile-type-assertions-to-the-rescue-6ddab4b8398b。我说啥来着，这是一个前人踩坑，后人踩坑，未来还会踩的坑。

u1s1, 这位大佬给出的方案和代码还是很赏心悦目的，值得学习（抄一下）。

------------------------------ END ----------------------------------