Changeset 35292 in osm for applications/editors/josm/plugins/comfort0/src

Timestamp:

2020-01-13T21:45:38+01:00 (5 years ago)

Author:

simon04

Message:

JOSM/comfort0: fix generated Level0L

File:

• applications/editors/josm/plugins/comfort0/src/net/simon04/comfort0/OsmToLevel0L.java (modified) (4 diffs)

applications/editors/josm/plugins/comfort0/src/net/simon04/comfort0/OsmToLevel0L.java

@@ -22 +22 @@
     private final StringBuilder sb = new StringBuilder();
 
-    public void visit(Collection<OsmPrimitive> primitives) {
+    /**
+     * Visits a collection of primitives
+     * @param primitives The collection of primitives
+     * @return {@code this}
+     */
+    public OsmToLevel0L visit(Collection<OsmPrimitive> primitives) {
         primitives.stream()
                 .sorted(OsmPrimitiveComparator.orderingWaysRelationsNodes().thenComparing(OsmPrimitiveComparator.comparingUniqueId()))
                 .forEachOrdered(p -> p.accept(this));
+        return this;
     }
 
@@ -37 +43 @@
         appendCommon(w);
         for (Node node : w.getNodes()) {
-            sb.append("  n").append(node.getUniqueId());
-            appendDisplayName(node);
+            appendRef(node, "");
         }
     }
@@ -46 +51 @@
         appendCommon(r);
         for (RelationMember member : r.getMembers()) {
-            sb.append("  ");
-            switch (member.getType()) {
-                case NODE:
-                    sb.append("nd ");
-                    break;
-                case WAY:
-                    sb.append("wy ");
-                    break;
-                case RELATION:
-                    sb.append("rel ");
-                    break;
-                default:
-                    break;
-            }
-            sb.append(member.getUniqueId()).append(" ");
-            sb.append(member.getRole());
-            appendDisplayName(member.getMember());
+            appendRef(member.getMember(), " " + member.getRole());
         }
+    }
+
+    private void appendRef(OsmPrimitive primitive, String mixin) {
+        sb.append("  ");
+        switch (primitive.getType()) {
+            case NODE:
+                sb.append("nd ");
+                break;
+            case WAY:
+                sb.append("wy ");
+                break;
+            case RELATION:
+                sb.append("rel ");
+                break;
+            default:
+                break;
+        }
+        sb.append(primitive.getUniqueId());
+        sb.append(mixin);
+        appendDisplayName(primitive);
     }
 
@@ -93 +102 @@
     @Override
     public String toString() {
-        return sb.toString().trim();
+        return sb.toString().replaceFirst("^\\n", "");
     }
 }