Implement json.Marshaller interface about gods HOT 8 CLOSED

@saheljalal

I think I had somewhere the idea of making serialization and deserialization (marshalling/unmarshalling) of these structures. My idea was to have that done in a binary form at first, but JSON could also be another option due to readability. Let me meditate on this and find some time, last few months have been crazy for me and I hardly found any free time.

What data structures are you using and would you like this feature for? What is the use case for JSON marshalling?

from gods.

I just discovered this library recently and am using the hashset to store identifiers to start. That list is just a []string but working with the set API is so much easier and faster at call sites and from a performance perspective I'm sure. On the outgoing interface as json, however, I am just marshaling my own struct to handle this for now but was wondering if it was something that could be done in a generic fashion in the data structure itself.

from gods.

@saheljalal

In most cases, the marshalling would have to be handled individually for most structures. They would share a common interface, but the logic would be custom for each. I think that would be a cool feature and will try to include it in this lib, once I clean up the other pull requests.

From the performance perspective, there is a penalty in this lib, because it uses interface{} types everywhere, which gets hit hard type assertion implementation in Go. I am not criticizing this lib, but one should be aware of that fact when using this lib and it's mainly due to the lack of generics (won't go there, I'll let others be opinionated on this) and, thus, having to type assert everywhere. My suggestion is to use native Go's maps with concrete types if the problem you are working on is simple. The HashSet is really a simple implementation that uses Go's native map, where key is the value as to be unique in the set and the value is empty struct{}{}.

from gods.

@emirpasic will keep that in mind going forward and hope to see this materialize in the future. I'd like to make more use of this library given its simple and intuitive interfaces compared to go's limited type offerings. If performance becomes an issue, will evaluate my options at that time. Thanks.

from gods.

I was about to ask for something like unmarshaling, it'd be very very useful

from gods.

@LucasZanella sure, marshaling (serialization) and unmarshaling (deserialization) will be implemented together

from gods.

@saheljalal @LucasZanella JSON marshalling and unmarshalling has been implemented in #53 for all data structures.

Please make note of the ToJSON and FromJSON functions from the examples and let me know if this works out for you.

from gods.

closed by #53

from gods.