Changeset 12865 in josm for trunk/src/org/openstreetmap/josm/tools/MultiMap.java

Timestamp:

2017-09-15T01:20:41+02:00 (7 years ago)

Author:

Don-vip

Message:

see #11390 - SonarQube - squid:S3824 - "Map.get" and value test should be replaced with single method call

File:

• trunk/src/org/openstreetmap/josm/tools/MultiMap.java (modified) (2 diffs)

trunk/src/org/openstreetmap/josm/tools/MultiMap.java

@@ -66 +66 @@
      */
     public void put(A key, B value) {
-        Set<B> vals = map.get(key);
-        if (vals == null) {
-            vals = new LinkedHashSet<>();
-            map.put(key, vals);
-        }
-        vals.add(value);
+        map.computeIfAbsent(key, k -> new LinkedHashSet<>()).add(value);
     }
 
@@ -95 +90 @@
      */
     public void putAll(A key, Collection<B> values) {
-        Set<B> vals = map.get(key);
-        if (vals == null) {
-            vals = new LinkedHashSet<>(values);
-            map.put(key, vals);
-        }
-        vals.addAll(values);
+        map.computeIfAbsent(key, k -> new LinkedHashSet<>(values)).addAll(values);
     }