How to do a generic decode with golang

Question

I've seen some go code that looks like this :

type userForm struct {
    Name     string `json:"name" validate:"min=2"`
    Surname  string `json:"surname" validate:"min=2"`
    Phone    string `json:"phone" validate:"min=10"`
    Email    string `json:"email,omitempty" validate:"omitempty,email"`
}

type emailForm struct {
    Email string `json:"email" validate:"email"`
}

func decodeUserForm(r *http.Request) (userForm, error) {
    var body userForm
    d := json.NewDecoder(r.Body)
    if err := d.Decode(&body); err != nil {
        return body, NewHTTPError(err, 400, "unable to decode user form")
    }
    return body, validateStruct(body)
}

func decodeEmailForm(r *http.Request) (emailForm, error) {
    var body emailForm
    d := json.NewDecoder(r.Body)
    if err := d.Decode(&body); err != nil {
        return body, NewHTTPError(err, 400, "unable to decode email form")
    }
    return body, validateStruct(body)
}

I find two functions redundant. Is there a simpler way to merge those two into a more generic function? Is it good practice in Go?

Jonathan Hall · Accepted Answer · 2020-05-22 14:12:51Z

3

func decodeForm(r *http.Request, dst interface{}) error {
    if err := json.NewDecoder(r.Body).Decode(dst); err != nil {
        return NewHTTPError(err, 400, "unable to decode email form")
    }
    return validateStruct(dst)
}

Then use it as so:

var body emailForm
if err := decodeForm(r, &body); err != nil {
    panic(err)
}

edited May 22, 2020 at 14:12

answered May 22, 2020 at 14:08

Jonathan Hall

80.4k19 gold badges162 silver badges206 bronze badges

Sign up to request clarification or add additional context in comments.

2 Comments

Ado Ren Over a year ago

thanks, that seems way cleaner than what I coded. Is it good practice to go through interface like that ?

Jonathan Hall Over a year ago

@AdoRen: It's the way json works, so it's not only good practice, it's essentially the only way :)

Collectives™ on Stack Overflow

How to do a generic decode with golang

1 Answer 1

2 Comments

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

2 Comments

Related