Changeset 8496 in josm for trunk/src/org/openstreetmap/josm/data

Timestamp:

2015-06-19T20:23:18+02:00 (10 years ago)

Author:

simon04

Message:

fix #11505 - Download objects: load a range of OSM objects (modified patch by windu.2b)

File:

• trunk/src/org/openstreetmap/josm/data/osm/SimplePrimitiveId.java (modified) (4 diffs)

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

@@ -5 +5 @@
 import java.util.ArrayList;
 import java.util.List;
+import java.util.regex.MatchResult;
 import java.util.regex.Matcher;
 import java.util.regex.Pattern;
@@ -15 +16 @@
     private final OsmPrimitiveType type;
 
-    public static final Pattern ID_PATTERN = Pattern.compile("((n(ode)?|w(ay)?|r(el(ation)?)?)[ /]?)(\\d+)");
+    public static final Pattern ID_PATTERN = Pattern.compile("(n|node|w|way|r|rel|relation)[ /]?(\\d+)");
+
+    public static final Pattern MULTIPLE_IDS_PATTERN = Pattern.compile(ID_PATTERN.pattern() + "(-(\\d+))?");
 
     public SimplePrimitiveId(long id, OsmPrimitiveType type) {
@@ -80 +83 @@
         final Matcher m = ID_PATTERN.matcher(s);
         if (m.matches()) {
-            return new SimplePrimitiveId(Long.parseLong(m.group(m.groupCount())),
-                    s.charAt(0) == 'n'
-                            ? OsmPrimitiveType.NODE
-                            : s.charAt(0) == 'w'
-                            ? OsmPrimitiveType.WAY
-                            : OsmPrimitiveType.RELATION);
+            return new SimplePrimitiveId(Long.parseLong(m.group(m.groupCount())), getOsmPrimitiveType(s.charAt(0)));
         } else {
             throw new IllegalArgumentException("The string " + s + " does not match the pattern " + ID_PATTERN);
@@ -92 +90 @@
 
     /**
+     * Parses a range {@code SimplePrimitiveId} from the string {@code s}.
+     * @param s the string to be parsed, e.g., {@code node1}, {@code node1-7}, {@code node70-7}.
+     * @return the parsed {@code SimplePrimitiveId}s
+     * @throws IllegalArgumentException if the string does not match the pattern
+     */
+    public static List<SimplePrimitiveId> multipleFromString(String s) {
+        final Matcher m = MULTIPLE_IDS_PATTERN.matcher(s);
+        if (m.matches()) {
+            return extractIdsInto(m, new ArrayList<SimplePrimitiveId>());
+        } else {
+            throw new IllegalArgumentException("The string " + s + " does not match the pattern " + MULTIPLE_IDS_PATTERN);
+        }
+    }
+
+    /**
      * Attempts to parse extract any primitive id from the string {@code s}.
-     * @param s the string to be parsed, e.g., {@code n1, w1}, {@code node1 and rel2}.
+     * @param s the string to be parsed, e.g., {@code "n1, w1"}, {@code "node1 and rel2"}, {@code "node 123-29"}.
      * @return the parsed list of {@code OsmPrimitiveType}s.
      */
     public static List<SimplePrimitiveId> fuzzyParse(String s) {
         final List<SimplePrimitiveId> ids = new ArrayList<>();
-        final Matcher m = ID_PATTERN.matcher(s);
+        final Matcher m = MULTIPLE_IDS_PATTERN.matcher(s);
         while (m.find()) {
-            final char firstChar = s.charAt(m.start());
-            ids.add(new SimplePrimitiveId(Long.parseLong(m.group(m.groupCount())),
-                    firstChar == 'n'
-                            ? OsmPrimitiveType.NODE
-                            : firstChar == 'w'
-                            ? OsmPrimitiveType.WAY
-                            : OsmPrimitiveType.RELATION));
+            extractIdsInto(m, ids);
         }
         return ids;
     }
+
+    private static List<SimplePrimitiveId> extractIdsInto(MatchResult m, List<SimplePrimitiveId> ids) {
+        final OsmPrimitiveType type = getOsmPrimitiveType(m.group(1).charAt(0));
+        final String firstId = m.group(2);
+        final String lastId = m.group(4);
+        if (lastId != null) {
+            final long lastIdParsed;
+            if (lastId.length() < firstId.length()) {
+                // parse ranges such as 123-25 or 123-5
+                lastIdParsed = Long.parseLong(firstId.substring(0, firstId.length() - lastId.length()) + lastId);
+            } else {
+                // parse ranges such as 123-125 or 998-1001
+                lastIdParsed = Long.parseLong(lastId);
+            }
+            for (long i = Long.parseLong(firstId); i <= lastIdParsed; i++) {
+                ids.add(new SimplePrimitiveId(i, type));
+            }
+        } else {
+            ids.add(new SimplePrimitiveId(Long.parseLong(firstId), type));
+        }
+        return ids;
+    }
+
+    private static OsmPrimitiveType getOsmPrimitiveType(char firstChar) {
+        return firstChar == 'n' ? OsmPrimitiveType.NODE : firstChar == 'w' ? OsmPrimitiveType.WAY : OsmPrimitiveType.RELATION;
+    }
 }