Gson can't deserialize inherited class?

Question

I have a simple Json structure like:

{"MessageType":"TimeData","TimeData":{"hh":12,"mm":13,"ms":15,"ss":14}}

and I devised the following classes to deserialize it:

public class JsonMessage
{
    public enum MessageTypes{
        WhoAreYou,
        TimeData
    }
    JsonMessage(){
    }
    public MessageTypes MessageType;
}

class TimeData extends JsonMessage{
    int hh;
    int mm;
    int ss;
    int ms;

    TimeData() {
    }    
}

I need to split deserialization into tow phases:

1- deserialize to read the MessageType.

2- proceed with the rest of deserialization based on the MessageType

The code is straightforward:

public void dispatch(Object message, IoSession session)
    {

            Gson gson = new Gson();
            JsonMessage result = gson.fromJson(message.toString(), JsonMessage.class);
            System.out.println(result.MessageType.toString());
            switch (result.MessageType)
                {
                    case WhoAreYou:{
                    //.....
                    break;
                    }
                    case TimeUpdate:
                        TimeData res = new Gson().fromJson(message.toString(), TimeData.class);
                        System.out.println(res.hh);
                        break;
                    default:break;
        }
    }

My Program can enter the correct switch-case(which is TimeUpdate) but it doesn't parse it correctly (The println prints 0 instead of 12)

where do you think I have done something wrong? thank you

You can take a look to [http://stackoverflow.com/a/22081826/3315914](http://stackoverflow.com/a/22081826/3315914). I've answered a similar question there. — rpax, Feb 28 '14 at 08:05

score 8 · Accepted Answer · answered Apr 26 '13 at 04:16

The issue is that your JSON represents an Object that contains another object you're interested in while your Java is just a single object.

You can actually just write deserializers for each type and use them once you determine the MessageType:

public static void main(String[] args)
{
    Gson gson = new GsonBuilder().registerTypeAdapter(TimeData.class, new TimeDataDeserializer()).create();
    String json = "{\"MessageType\":\"TimeData\",\"TimeData\":{\"hh\":12,\"mm\":13,\"ms\":15,\"ss\":14}}";
    JsonMessage message = gson.fromJson(json, JsonMessage.class);

    switch(message.MessageType)
    {
        case TimeData:
            TimeData td = new GsonBuilder()
                            .registerTypeAdapter(TimeData.class, new TimeDataDeserializer())
                            .create()
                            .fromJson(json, TimeData.class);
            td.MessageType = message.MessageType
            System.out.println(td.hh);
            break;
        default:
            break;
    }
}

class TimeDataDeserializer implements JsonDeserializer<TimeData>
{
    @Override
    public TimeData deserialize(JsonElement je, Type type, JsonDeserializationContext jdc)  
        throws JsonParseException
    {
        JsonObject jo = je.getAsJsonObject().getAsJsonObject("TimeData");
        Gson g = new Gson();
        return g.fromJson(jo, TimeData.class);
    }
}

It still doesn't cast the whole response into TimeData object as MessageTypes is accessed from parent class and not child. Any answer that could solve this problem? — akshat tailang, Nov 28 '18 at 08:33

Jack · Answer 2 · 2013-04-25T17:30:56.380

I managed to solve this similar problem by implementing a custom JsonDeserializer in the following way.

First you attach to your enum the subclasses based on the type and a method to retrieve the correct Class<?> according to the enum name:

enum MessageType {
  WHO_ARE_YOU(WhoAreYou.class),
  TIME_UPDATE(TimeUpdate.class);

  public final Class<?> clazz;

  MessageType(Class<?> clazz) { this.clazz = clazz; }

  public static MessageType forName(String name) {
    for (MessageType t : values())
      if (name.equals(t.name()))
        return t;

    return NULL;
  }
}

Then in the deserialize method I did the following:

public JsonMessage deserialize(JsonElement json, Type typeOfT, JsonDeserializationContext context) throws JsonParseException
{
   JsonObject object = json.getAsJsonObject();
   String kind = object.get("messageType").getAsString();
   Class<?> clazz = MessageType.forName(kind).clazz;
   JsonMessage result = null;

   try {
     result = (JsonMessage)clazz.newInstance();
     Field[] fs = clazz.getFields();

     for (Field f : fs) {
       Object value = context.deserialize(object.get(f.getName()), f.getType());                 
       if (value != null)
         f.set(result, value);
     }
   } 
   catch (Exception e) {
     e.printStackTrace();
   }
}

Everything is managed by reflection so that a correct object is created and then all fields are deserialized accordingly.

I had a complex hierarchy of objects so I preferred to go this way to let the gson deserializer manage everything. Of course you will need to register the serializer with the gson parser instance.

A NOTE: Your naming of things is quite incorrect according to Java standards. enum constants should be ALL_CAPITALIZED, enum class names should be singular (eg. MessageType). Instance variables should be camelcased (eg. messageType not MessageType)

thanks for the solution. I started Java Programming a couple of days ago, so I need to read the code a couple of times to comprehend. I will start making changes now and will let you know the result in no time. Thanks — rahman, Apr 25 '13 at 16:56
Jack, I am sorry: in you first code snippet, I had to change `for (MessageType t : values)` to `for (MessageType t : MessageType.values())` and even then I got the error:"non-static variable MessageType cannot be referenced from a static context" I tried a bit up&down with no luck. canyou help please? — rahman, Apr 25 '13 at 17:15
make the method `static` and you are done (and yes, replace `values` with `values()`) — Jack, Apr 25 '13 at 17:30

Gson can't deserialize inherited class?

2 Answers2