Skip to content

Commit 1bba184

Browse files
javadevjavadev
authored
Added U.objectBuilder().fromMap(map).
1 parent 0082e9a commit 1bba184

File tree

7 files changed

+66
-37
lines changed

7 files changed

+66
-37
lines changed

‎src/main/java/com/github/underscore/Optional.java‎

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -62,7 +62,6 @@ public boolean isPresent() {
6262
return !absent;
6363
}
6464

65-
@SuppressWarnings("unchecked")
6665
public Optional<T> filter(Predicate<? super T> predicate) {
6766
Underscore.checkNotNull(predicate);
6867
if (isPresent()) {

‎src/main/java/com/github/underscore/Underscore.java‎

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1165,7 +1165,6 @@ public static <E> int size(final E... array) {
11651165
return array.length;
11661166
}
11671167

1168-
@SuppressWarnings("unchecked")
11691168
public static <E> List<List<E>> partition(final Iterable<E> iterable, final Predicate<E> pred) {
11701169
final List<E> retVal1 = newArrayList();
11711170
final List<E> retVal2 = newArrayList();

‎src/main/java/com/github/underscore/lodash/U.java‎

Lines changed: 32 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -211,7 +211,6 @@ public Chain<T> compact(final T falsyValue) {
211211
}
212212

213213
@Override
214-
@SuppressWarnings("unchecked")
215214
public Chain flatten() {
216215
return new Chain<>(Underscore.flatten(value()));
217216
}
@@ -966,9 +965,7 @@ public static <T> List<T> fill(List<T> list, T item) {
966965
}
967966

968967
public static <T> T[] fill(T[] array, T item) {
969-
for (int i = 0; i < array.length; i++) {
970-
array[i] = item;
971-
}
968+
Arrays.fill(array, item);
972969
return array;
973970
}
974971

@@ -1763,7 +1760,6 @@ public static <T> T remove(final Map<String, Object> object, final String path)
17631760
return baseGetOrSetOrRemove(object, path, null, OperationType.REMOVE);
17641761
}
17651762

1766-
@SuppressWarnings("unchecked")
17671763
public static Map<String, Object> rename(
17681764
final Map<String, Object> map, final String oldKey, final String newKey) {
17691765
Map<String, Object> outMap = newLinkedHashMap();
@@ -1842,7 +1838,6 @@ private static Object makeObjectForSetValue(
18421838
return result;
18431839
}
18441840

1845-
@SuppressWarnings("unchecked")
18461841
public static Map<String, Object> update(
18471842
final Map<String, Object> map1, final Map<String, Object> map2) {
18481843
Map<String, Object> outMap = newLinkedHashMap();
@@ -2363,7 +2358,6 @@ public static Map<String, Object> fromXmlMap(final String xml) {
23632358
return fromXmlMap(xml, Xml.FromType.FOR_CONVERT);
23642359
}
23652360

2366-
@SuppressWarnings("unchecked")
23672361
public static Map<String, Object> fromXmlMap(final String xml, final Xml.FromType fromType) {
23682362
final Object object = Xml.fromXml(xml, fromType);
23692363
return getStringObjectMap(object);
@@ -2411,7 +2405,6 @@ public Object fromJson() {
24112405
return Json.fromJson(getString().get());
24122406
}
24132407

2414-
@SuppressWarnings("unchecked")
24152408
public static Map<String, Object> fromJsonMap(final String string) {
24162409
final Object object = Json.fromJson(string);
24172410
return getStringObjectMap(object);
@@ -2669,7 +2662,6 @@ private static Object makeObjectSelfClose(Object value, String newValue) {
26692662
return result;
26702663
}
26712664

2672-
@SuppressWarnings("unchecked")
26732665
public static Map<String, Object> replaceEmptyValueWithNull(Map<String, Object> map) {
26742666
if (map == null || map.isEmpty()) {
26752667
return null;
@@ -2698,7 +2690,6 @@ private static Object makeObjectEmptyValue(Object value) {
26982690
return result;
26992691
}
27002692

2701-
@SuppressWarnings("unchecked")
27022693
public static Object replaceEmptyValueWithEmptyString(Map<String, Object> map) {
27032694
if (map.isEmpty()) {
27042695
return "";
@@ -2728,7 +2719,6 @@ private static Object makeObjectEmptyString(Object value) {
27282719
return result;
27292720
}
27302721

2731-
@SuppressWarnings("unchecked")
27322722
public static Map<String, Object> forceAttributeUsage(Map<String, Object> map) {
27332723
Map<String, Object> outMap = newLinkedHashMap();
27342724
for (Map.Entry<String, Object> entry : map.entrySet()) {
@@ -2760,7 +2750,6 @@ private static Object makeAttributeUsage(Object value) {
27602750
return result;
27612751
}
27622752

2763-
@SuppressWarnings("unchecked")
27642753
public static Map<String, Object> replaceNullWithEmptyValue(Map<String, Object> map) {
27652754
Map<String, Object> outMap = newLinkedHashMap();
27662755
for (Map.Entry<String, Object> entry : map.entrySet()) {
@@ -2790,7 +2779,6 @@ private static Object makeReplaceNullValue(Object value) {
27902779
return result;
27912780
}
27922781

2793-
@SuppressWarnings("unchecked")
27942782
public static Map<String, Object> replaceEmptyStringWithEmptyValue(Map<String, Object> map) {
27952783
Map<String, Object> outMap = newLinkedHashMap();
27962784
for (Map.Entry<String, Object> entry : map.entrySet()) {
@@ -2821,7 +2809,6 @@ private static Object makeReplaceEmptyString(Object value) {
28212809
return result;
28222810
}
28232811

2824-
@SuppressWarnings("unchecked")
28252812
public static Map<String, Object> replaceFirstLevel(Map<String, Object> map) {
28262813
return replaceFirstLevel(map, 0);
28272814
}
@@ -2860,6 +2847,31 @@ private static Object makeReplaceFirstLevel(Object value, int level) {
28602847
return result;
28612848
}
28622849

2850+
public static Map<String, Object> deepCopyMap(Map<String, Object> map) {
2851+
Map<String, Object> outMap = newLinkedHashMap();
2852+
for (Map.Entry<String, Object> entry : map.entrySet()) {
2853+
outMap.put(entry.getKey(), makeDeepCopyMap(entry.getValue()));
2854+
}
2855+
return outMap;
2856+
}
2857+
2858+
@SuppressWarnings("unchecked")
2859+
private static Object makeDeepCopyMap(Object value) {
2860+
final Object result;
2861+
if (value instanceof List) {
2862+
List<Object> values = newArrayList();
2863+
for (Object item : (List) value) {
2864+
values.add(item instanceof Map ? deepCopyMap((Map) item) : item);
2865+
}
2866+
result = values;
2867+
} else if (value instanceof Map) {
2868+
result = deepCopyMap((Map) value);
2869+
} else {
2870+
result = value;
2871+
}
2872+
return result;
2873+
}
2874+
28632875
public static long gcd(long value1, long value2) {
28642876
if (value1 == 0) {
28652877
return value2;
@@ -2941,6 +2953,12 @@ public static Builder fromXml(final String xml) {
29412953
return builder;
29422954
}
29432955

2956+
public static Builder fromMap(final Map<String, Object> map) {
2957+
final Builder builder = new Builder();
2958+
deepCopyMap(map).forEach(builder.data::put);
2959+
return builder;
2960+
}
2961+
29442962
public String toJson() {
29452963
return Json.toJson(data);
29462964
}

‎src/main/java/com/github/underscore/lodash/Xml.java‎

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -194,11 +194,6 @@ public static class XmlStringBuilderText extends XmlStringBuilderWithoutHeader {
194194
public XmlStringBuilderText(XmlStringBuilder.Step identStep, int ident) {
195195
super(identStep, ident);
196196
}
197-
198-
@Override
199-
public String toString() {
200-
return super.toString();
201-
}
202197
}
203198

204199
public static class XmlArray {
@@ -237,7 +232,6 @@ public static void writeXml(
237232
}
238233
}
239234

240-
@SuppressWarnings("unchecked")
241235
private static void writeXml(
242236
Collection<?> collection,
243237
XmlStringBuilder builder,

‎src/test/java/com/github/underscore/CollectionsTest.java‎

Lines changed: 8 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,6 @@
3030

3131
import java.util.ArrayDeque;
3232
import java.util.ArrayList;
33-
import java.util.Arrays;
3433
import java.util.Collections;
3534
import java.util.Comparator;
3635
import java.util.Iterator;
@@ -802,14 +801,15 @@ public void invoke() {
802801
Underscore.chain(asList(" foo ", " bar")).invoke("trim").value());
803802
assertEquals(
804803
asList("foo1", "bar1"),
805-
Underscore.invoke(asList("foo", "bar"), "concat", Arrays.<Object>asList("1")));
804+
Underscore.invoke(asList("foo", "bar"), "concat", Collections.singletonList("1")));
806805
assertEquals(
807806
asList("foo1", "bar1"),
808-
new Underscore(asList("foo", "bar")).invoke("concat", Arrays.<Object>asList("1")));
807+
new Underscore(asList("foo", "bar"))
808+
.invoke("concat", Collections.<Object>singletonList("1")));
809809
assertEquals(
810810
asList("foo1", "bar1"),
811811
Underscore.chain(asList("foo", "bar"))
812-
.invoke("concat", Arrays.<Object>asList("1"))
812+
.invoke("concat", Collections.singletonList("1"))
813813
.value());
814814
assertEquals(
815815
asList("[1, 5, 7]", "[1, 2, 3]").toString(),
@@ -824,7 +824,7 @@ public void invoke() {
824824
@Test(expected = IllegalArgumentException.class)
825825
@SuppressWarnings("unchecked")
826826
public void invokeError() {
827-
Underscore.invoke(asList("foo", 123), "concat", Arrays.<Object>asList("1"));
827+
Underscore.invoke(asList("foo", 123), "concat", Collections.<Object>singletonList("1"));
828828
}
829829

830830
@Test(expected = IllegalArgumentException.class)
@@ -866,7 +866,7 @@ public Integer getAge() {
866866
return age;
867867
}
868868
}
869-
assertEquals("[]", Underscore.pluck(asList(), "name").toString());
869+
assertEquals("[]", Underscore.pluck(Collections.emptyList(), "name").toString());
870870
final List<?> result =
871871
Underscore.pluck(
872872
asList(
@@ -900,7 +900,8 @@ public Integer getAge() {
900900
.pluck("name")
901901
.value();
902902
assertEquals("[moe, larry, curly]", resultChain.toString());
903-
final Set<?> resultEmpty2 = Underscore.pluck(new LinkedHashSet(asList()), "name");
903+
final Set<?> resultEmpty2 =
904+
Underscore.pluck(new LinkedHashSet(Collections.emptyList()), "name");
904905
assertEquals("[]", resultEmpty2.toString());
905906
final Set<?> resultSet =
906907
Underscore.pluck(

‎src/test/java/com/github/underscore/lodash/LodashTest.java‎

Lines changed: 26 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -777,10 +777,7 @@ public void renameRoot() {
777777
U.jsonToXml("{\"a\": \"b\", \"c\": \"d\"}", "json"));
778778
assertEquals(
779779
"<?xml version=\"1.0\" encoding=\"UTF-8\"?>\n<json a=\"b\" c=\"d\"></json>",
780-
U.jsonToXml(
781-
"{\"a\": \"b\", \"c\": \"d\"}",
782-
U.Mode.FORCE_ATTRIBUTE_USAGE,
783-
"json"));
780+
U.jsonToXml("{\"a\": \"b\", \"c\": \"d\"}", U.Mode.FORCE_ATTRIBUTE_USAGE, "json"));
784781
}
785782

786783
@Test
@@ -816,7 +813,8 @@ public void forceAddRoot() {
816813
+ " }\n"
817814
+ " ]\n"
818815
+ "}",
819-
U.Mode.FORCE_ADD_ROOT_JSON_TO_XML, "newroot"));
816+
U.Mode.FORCE_ADD_ROOT_JSON_TO_XML,
817+
"newroot"));
820818
assertEquals(
821819
"<?xml version=\"1.0\" encoding=\"UTF-8\"?>\n"
822820
+ "<root>\n"
@@ -1135,6 +1133,28 @@ public void replaceFirstLevel() {
11351133
U.replaceFirstLevel(result4);
11361134
}
11371135

1136+
@Test
1137+
@SuppressWarnings("unchecked")
1138+
public void deepCopyMap() {
1139+
Map<String, Object> result = U.deepCopyMap(U.fromXml("<a/>"));
1140+
assertEquals("{a={-self-closing=true}, #omit-xml-declaration=yes}", result.toString());
1141+
Map<String, Object> result2 = U.deepCopyMap(U.fromXml("<a><b>c</b></a>"));
1142+
assertEquals("{a={b=c}, #omit-xml-declaration=yes}", result2.toString());
1143+
Map<String, Object> map = U.newLinkedHashMap();
1144+
List<Object> list = U.newArrayList();
1145+
list.add(U.newLinkedHashMap());
1146+
map.put("list", list);
1147+
U.deepCopyMap(map);
1148+
Map<String, Object> map2 = U.newLinkedHashMap();
1149+
List<Object> list2 = U.newArrayList();
1150+
list2.add(U.newArrayList());
1151+
map2.put("list", list2);
1152+
U.deepCopyMap(map2);
1153+
Map<String, Object> result3 = U.fromXml("<a/>");
1154+
((Map<String, Object>) result3.get("a")).put("-self-closing", "false");
1155+
U.deepCopyMap(result3);
1156+
}
1157+
11381158
@Test
11391159
public void objectBuilder() {
11401160
U.Builder builder = U.objectBuilder().add("1", "2").add("2");
@@ -1143,6 +1163,7 @@ public void objectBuilder() {
11431163
U.Builder.fromJson("{}");
11441164
builder.toXml();
11451165
U.Builder.fromXml("<a/>");
1166+
U.Builder.fromMap(U.newLinkedHashMap());
11461167
builder.set("1", "3");
11471168
builder.toString();
11481169
assertEquals("{1=3}", builder.build().toString());

‎src/test/java/com/github/underscore/lodash/StringTest.java‎

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -3411,7 +3411,6 @@ public void fromXml() {
34113411
U.fromXml(stringXml);
34123412
}
34133413

3414-
@SuppressWarnings("unchecked")
34153414
@Test
34163415
public void fromXmlMap() {
34173416
String stringXml =
@@ -3424,7 +3423,6 @@ public void fromXmlMap() {
34243423
assertEquals("{}", U.fromXmlMap(stringXml3, Xml.FromType.FOR_CONVERT).toString());
34253424
}
34263425

3427-
@SuppressWarnings("unchecked")
34283426
@Test
34293427
public void fromJsonMap() {
34303428
String stringJson = "[]";
@@ -3901,7 +3899,6 @@ public void testJavaObjectArrayToString() {
39013899
builder.toString());
39023900
}
39033901

3904-
@SuppressWarnings("unchecked")
39053902
@Test
39063903
public void toJsonJavaFromList() {
39073904
final List<String> testList = new ArrayList<>();

0 commit comments

Comments
 (0)