Changeset 15671 in josm for trunk/src/org

Timestamp:

2020-01-10T00:07:20+01:00 (5 years ago)

Author:

simon04

Message:

Refactoring: OsmUtils.splitMultipleValues

Location:

trunk/src/org/openstreetmap/josm

Files:

• data/osm/OsmUtils.java (modified) (2 diffs)
• gui/mappaint/mapcss/ConditionFactory.java (modified) (2 diffs)

trunk/src/org/openstreetmap/josm/data/osm/OsmUtils.java

@@ -8 +8 @@
 import java.util.Map;
 import java.util.Set;
+import java.util.regex.Pattern;
+import java.util.stream.Stream;
 
 import org.openstreetmap.josm.data.coor.LatLon;
@@ -207 +209 @@
         return ds == null || !ds.isLocked();
     }
+
+    /**
+     * Splits a tag value by <a href="https://wiki.openstreetmap.org/wiki/Semi-colon_value_separator">semi-colon value separator</a>.
+     * Spaces around the ; are ignored.
+     *
+     * @param value the value to separate
+     * @return the separated values as Stream
+     * @since xxx
+     */
+    public static Stream<String> splitMultipleValues(String value) {
+        return Pattern.compile("\\s*;\\s*").splitAsStream(value);
+    }
 }

trunk/src/org/openstreetmap/josm/gui/mappaint/mapcss/ConditionFactory.java

@@ -4 +4 @@
 import java.lang.reflect.Method;
 import java.text.MessageFormat;
-import java.util.Arrays;
 import java.util.EnumSet;
 import java.util.Map;
@@ -173 +172 @@
         /** The reference is treated as a list separated by ';'. Spaces around the ; are ignored.
          *  The value needs to be equal one of the list elements. */
-        ONE_OF((test, prototype) -> Arrays.asList(test.split("\\s*;\\s*")).contains(prototype)),
+        ONE_OF((test, prototype) -> OsmUtils.splitMultipleValues(test).anyMatch(prototype::equals)),
         /** The value needs to begin with the reference string. */
         BEGINS_WITH(String::startsWith),