Enum deserialization with Newtonsoft.Json

Question

Hello I want to deserialize a class and in this class is a enum value:

[JsonConverter(typeof(StringEnumConverter))] public enum MessageType { Verify, Disconnect, } [Serializable] public class SocketMessage { public Dictionary<String, String> Header { get; private set; } public MessageType MessageType { get; private set; } public SocketMessage(MessageType type) { Header = new Dictionary<String, String>(); MessageType = type; } public void AddHeaderData(String key, String data) { Header.Add(key, data); } public byte[] ToJSONBytes() { String json = JsonConvert.SerializeObject(this); return Encoding.UTF8.GetBytes(json); } public static SocketMessage FromJSONBytes(byte[] json) { String s = Encoding.UTF8.GetString(json); return JsonConvert.DeserializeObject<SocketMessage>(s); } }

The dictionary will be correctly deserialized but the enum always get his default value >verify< the json looks like this: {"Header":{"Test":"Test"},"MessageType":"Disconnect"}

I really don't understand why it happens

I appreciate any help!

steve16351 · Accepted Answer · 2021-01-22 21:38:09Z

It does not work as you have it because of the private set on the MessageType property. So the getter is public and so it serializes fine, but when it comes to deserialize, it is ignored. There are a few possible solutions to this.

Use a public setter for MessageType. There is probably a reason why you wanted to use a private setter in the first place though, so this maybe not for you.
Apply the JsonProperty attribute to the MessageType property:

[JsonProperty] public MessageType MessageType { get; private set; }

Change the constructor parameter name to messageType instead of type, so it matches the serialized name:

public SocketMessage(MessageType messageType) { Header = new Dictionary<String, String>(); MessageType = messageType; }

Why was it not a problem for Header? This is because it is new'ed up in the constructor. The constructor is called in deserialization, so this gets set, and then added to as the dictionary is deserialized for each entry. If you removed it from the constructor, you would see it has the same problem and would be null after deserialization.

Thanks it worked! It seems the JsonProperty attribute isn't that issue but the name type => messageType fixed it, should I use the attribute anyway? I'm new to Json.net and json in general
@Traijan any one of those three things on their own should fix the problem. You don't have to use the JsonProperty if you have changed the constructor, though there are lots of other things you could use it for, further reading here.

Collectives™ on Stack Overflow

Enum deserialization with Newtonsoft.Json

1 Answer 1

2 Comments

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

2 Comments

Related