fastJson遇到的问题
概述
现在的代码开发中,json这种数据类型使用的是越来越多,因为它的存取速度都比较快,而且,使用起来非常的简单,今天工作的时候,我就遇到了一个关于json的生产问题,这个问题我之前确实还没有注意过,这边记录下。
问题再现
再一次远程调用获取相关信息成功之后,需要解析返回的json对象,这个json里面包含了三个元素,一个key为data的jsonObject,表示业务数据,一个Key为success的String,表示是否成功,一个key为resultMessage的失败原因。正常情况下,这个data是不可能为空的,但是今天不知道怎么了,远端返回的结果里面这个data就是空的,所以解析的时候就出现了如下异常:
Exception in thread "main" java.lang.ClassCastException: java.lang.String cannot be cast to com.alibaba.fastjson.JSONObject at com.alibaba.fastjson.JSONObject.getJSONObject(JSONObject.java:109) at com.example.demo.MainTest.main(MainTest.java:10)
当然这里要提前说明下,我这边使用的fastJson的版本是1.2.7,出现上面这个错是不是很奇怪,正常不应该返回一个null吗?怎么返回的是String类型不能转成JSONObject呢?
问题跟踪
问题到了这个地步还有什么好说的呢?只能模仿生产的返回,一步一步的去解析了。
1 public class MainTest { 2 public static void main(String[] args) {
// 写一个字符串,模拟生产的返回,其中data的值为空 3 String str = "{\"data\": \"\",\"success\": \"true\",\"resultMessage\": \"成功\"}"; 4 // 序列化成json 5 JSONObject result = JSONObject.parseObject(str);
// 获取data的值 6 JSONObject data = result.getJSONObject("data"); 7 8 System.out.println(data); 9 } 10 }
通过debug可以得知,第五行代码是可以正常执行的,返回如下:
{"data":"","success":"true","resultMessage":"成功"}
这个没有问题,那问题就在第6行代码上了,我们看一下getJSONObject方法:
1 public JSONObject getJSONObject(String key) { 2 Object value = map.get(key); 3 4 if (value instanceof JSONObject) { 5 return (JSONObject) value; 6 } 7 8 return (JSONObject) toJSON(value); 9 }
这边第2行代码根据key获取的value是"",这个显然不是JSONObject类型,直接走最后一行toJSON方法,
1 public static Object toJSON(Object javaObject, ParserConfig mapping) { 2 if (javaObject == null) { 3 return null; 4 } 5 6 if (javaObject instanceof JSON) { 7 return javaObject; 8 } 9 10 if (javaObject instanceof Map) { 11 Map<Object, Object> map = (Map<Object, Object>) javaObject; 12 13 JSONObject json = new JSONObject(map.size()); 14 15 for (Map.Entry<Object, Object> entry : map.entrySet()) { 16 Object key = entry.getKey(); 17 String jsonKey = TypeUtils.castToString(key); 18 Object jsonValue = toJSON(entry.getValue()); 19 json.put(jsonKey, jsonValue); 20 } 21 22 return json; 23 } 24 25 if (javaObject instanceof Collection) { 26 Collection<Object> collection = (Collection<Object>) javaObject; 27 28 JSONArray array = new JSONArray(collection.size()); 29 30 for (Object item : collection) { 31 Object jsonValue = toJSON(item); 32 array.add(jsonValue); 33 } 34 35 return array; 36 } 37 38 Class<?> clazz = javaObject.getClass(); 39 40 if (clazz.isEnum()) { 41 return ((Enum<?>) javaObject).name(); 42 } 43 44 if (clazz.isArray()) { 45 int len = Array.getLength(javaObject); 46 47 JSONArray array = new JSONArray(len); 48 49 for (int i = 0; i < len; ++i) { 50 Object item = Array.get(javaObject, i); 51 Object jsonValue = toJSON(item); 52 array.add(jsonValue); 53 } 54 55 return array; 56 } 57 58 if (mapping.isPrimitive(clazz)) { 59 return javaObject; 60 } 61 62 try { 63 List<FieldInfo> getters = TypeUtils.computeGetters(clazz, null); 64 65 JSONObject json = new JSONObject(getters.size()); 66 67 for (FieldInfo field : getters) { 68 Object value = field.get(javaObject); 69 Object jsonValue = toJSON(value); 70 71 json.put(field.getName(), jsonValue); 72 } 73 74 return json; 75 } catch (IllegalAccessException e) { 76 throw new JSONException("toJSON error", e); 77 } catch (InvocationTargetException e) { 78 throw new JSONException("toJSON error", e); 79 } 80 }
这个方法有点长,但其实,真正执行了的也就是38行,获取value对应的class对象,这边返回的java.lang.String,再就是58到60行,判断这个对象是不是基础类型,是就直接返回,那么这边就返回的"",后面强制类型转换的时候当然就会报错。
上面这个问题似乎还不好解决啊,后来看了一下fastJson的几个高版本,已经解决了这个问题,我们来看一下1.2.51的getJSONObject方法:
1 public JSONObject getJSONObject(String key) { 2 Object value = map.get(key); 3 4 if (value instanceof JSONObject) { 5 return (JSONObject) value; 6 } 7 8 if (value instanceof String) { 9 return JSON.parseObject((String) value); 10 } 11 12 return (JSONObject) toJSON(value); 13 }
多了8-10行代码,针对String做了下处理,直接返回String解析之后的结果,这样就可以返回理想中的null了。
总结
平时代码开发过程中,一定要进行异常情况的测试,不能够想当然,要尽量覆盖每个分支,要让代码跑不崩溃。