How to tell Golang Gob encoding that it’s ok to serialize a struct that contains a struct with no exported fields
Asked Answered
M

2

6

I believe this is a legitimate use case for Gob serialization. Yet enc.Encode returns an error because Something has no exported field. Note that I’m not serializing Something directly but only Composed that contains exported fields.

The only workaround I’ve found was to add a Dummy (exported) value to Something. This is ugly. Is there a more elegant solution?

https://play.golang.org/p/0pL6BfBb78m

package main

import (
    "bytes"
    "encoding/gob"
)

type Something struct {
    temp int
}

func (*Something) DoSomething() {}

type Composed struct {
    Something
    DataToSerialize int
}

func main() {
    enc := gob.NewEncoder(&bytes.Buffer{})
    err := enc.Encode(Composed{})
    if err != nil {
        panic(err)
    }
}
Minier answered 16/5, 2018 at 16:10 Comment(3)
fyi: there is an open issue for this in the Go repository github.com/golang/go/issues/5819Circle
@ThunderCat Funnily enough I was experimenting with your second solution exactly at the time you wrote your comment. And believe it or not, I find it less ugly (because it’s limited to the code and does not pollute the stream with dummy data)Minier
@ThunderCat too much boiler plate for the first one, second one is the least ugly one (is the nil array really safe or should it be an empty array?)Minier
V
4

Here are some different workarounds from that proposed in the question.

Don't use embedding.

type Composed struct {
    something       Something
    DataToSerialize int
}

func (c *Composed) DoSomething() { c.something.DoSomething() }

playground example

Implement GobDecoder and GobEncoder

func (*Something) GobDecode([]byte) error     { return nil }
func (Something) GobEncode() ([]byte, error) { return nil, nil }

playground example

Vulvitis answered 16/5, 2018 at 17:6 Comment(3)
First solution is not practical, Something could have dozens of methods.Minier
@FranckJeannin Why would the first solution be not practical? Even if something worth serializing would ever have dozens of methods: Just forward them. This is the exact opposite of rocket-science.Trotyl
@Trotyl it’s just that solution 1 will add exponentially more code, each time you add a method to Something you need to forward it for all the structs that embed Something. If you find solution 2 too rocket-science, I would suggest solution 0 : add a Dummy (exported) member to Something. It’s exactly one line of code regardless of own many times you embed Something.Minier
S
0

As far as I can tell the addition of the functions GobDecode and GobEncode only allow the encoder to avoid the error, but don't allow it to work correctly. If I add a Decode operation it doesn't seem to get the DataToSerialize item back with the value I encoded into it.

Playground example

Siemens answered 17/9, 2020 at 23:7 Comment(0)

© 2022 - 2024 — McMap. All rights reserved.