Skip to content

Commit f820045

Browse files
committed
fix Utils.getType and resolveValue.
1 parent ece6cbd commit f820045

File tree

2 files changed

+8
-18
lines changed

2 files changed

+8
-18
lines changed

README.md

+1-1
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,6 @@ To install the library add:
1515
maven { url "https://jitpack.io" }
1616
}
1717
dependencies {
18-
compile 'com.github.webee:java-json-api:v1.3.0'
18+
compile 'com.github.webee:java-json-api:v1.3.1'
1919
}
2020
```

src/main/java/com/github/webee/json/Utils.java

+7-17
Original file line numberDiff line numberDiff line change
@@ -23,10 +23,6 @@ public static JSONType getType(Object value) {
2323
return JSONType.Number;
2424
} else if (value instanceof Boolean) {
2525
return JSONType.Boolean;
26-
} else if (value instanceof Map) {
27-
return JSONType.Object;
28-
} else if (value instanceof Object[]) {
29-
return JSONType.Array;
3026
} else if (value instanceof JSONObject) {
3127
return JSONType.Object;
3228
} else if (value instanceof JSONArray) {
@@ -104,29 +100,23 @@ public static Object resolveValue(Object value) {
104100

105101
public static Object resolveValue(Object value, JSONType t) {
106102
if (t == null) {
103+
if (value instanceof Map) {
104+
return Utils.objectToMap((Map<String, Object>) value);
105+
} else if (value instanceof Object[]) {
106+
return Utils.arrayToObjects((Object[]) value);
107+
}
107108
return null;
108109
}
109110

110111
switch (t) {
111112
case Object:
112-
if (value instanceof JSONObject) {
113-
return Utils.objectToMap((JSONObject) value);
114-
} else if (value instanceof Map) {
115-
return Utils.objectToMap((Map<String, Object>) value);
116-
}
117-
break;
113+
return Utils.objectToMap((JSONObject) value);
118114
case Array:
119-
if (value instanceof JSONArray) {
120-
return Utils.arrayToObjects((JSONArray) value);
121-
} else if (value instanceof Object[]) {
122-
return Utils.arrayToObjects((Object[]) value);
123-
}
124-
break;
115+
return Utils.arrayToObjects((JSONArray) value);
125116
case Null:
126117
return null;
127118
default:
128119
return value;
129120
}
130-
return null;
131121
}
132122
}

0 commit comments

Comments
 (0)