Changeset 30796 in osm for applications/editors/josm/plugins/opendata/src

Timestamp:

2014-11-12T02:37:22+01:00 (10 years ago)

Author:

donvip

Message:

[josm_opendata] fix #josm10473 - detect duplicated ways during SHP import

Location:

applications/editors/josm/plugins/opendata/src/org/openstreetmap/josm/plugins/opendata/core/io/geographic

Files:

• GeographicReader.java (modified) (4 diffs)
• ShpReader.java (modified) (1 diff)

applications/editors/josm/plugins/opendata/src/org/openstreetmap/josm/plugins/opendata/core/io/geographic/GeographicReader.java

@@ -9 +9 @@
 import java.util.ArrayList;
 import java.util.Arrays;
+import java.util.Collections;
 import java.util.HashMap;
 import java.util.List;
@@ -43 +44 @@
 import org.openstreetmap.josm.data.osm.RelationMember;
 import org.openstreetmap.josm.data.osm.Way;
+import org.openstreetmap.josm.data.validation.tests.DuplicateWay;
 import org.openstreetmap.josm.gui.ExtendedDialog;
 import org.openstreetmap.josm.io.AbstractReader;
@@ -116 +118 @@
     }
     
-        protected Node createOrGetEmptyNode(Point p) throws MismatchedDimensionException, TransformException {
-                Point p2 = (Point) JTS.transform(p, transform);
-                LatLon key = new LatLon(p2.getY(), p2.getX());
-                Node n = getNode(p2, key);
-                if(n != null && n.hasKeys()) {
-                        n = null;
-                }
-                if (n == null) {
-                        n = new Node(key);
-                        if (handler == null || handler.useNodeMap()) {
-                                nodes.put(key, n);
-                        }
-                        ds.addPrimitive(n);
-                } else if (n.getDataSet() == null) {
-                    // handler may have removed the node from DataSet (see Paris public light handler for example)
-                    ds.addPrimitive(n);
-                }
-                return n;
-        }
+    protected Node createOrGetEmptyNode(Point p) throws MismatchedDimensionException, TransformException {
+        Point p2 = (Point) JTS.transform(p, transform);
+        LatLon key = new LatLon(p2.getY(), p2.getX());
+        Node n = getNode(p2, key);
+        if (n != null && n.hasKeys()) {
+            n = null;
+        }
+        if (n == null) {
+            n = new Node(key);
+            if (handler == null || handler.useNodeMap()) {
+                nodes.put(key, n);
+            }
+            ds.addPrimitive(n);
+        } else if (n.getDataSet() == null) {
+            // handler may have removed the node from DataSet (see Paris public light handler for example)
+            ds.addPrimitive(n);
+        }
+        return n;
+    }
         
     protected <T extends OsmPrimitive> T addOsmPrimitive(T p) {
@@ -145 +147 @@
     }
 
-    protected final Way createWay(LineString ls) {
-        Way w = createWay();
+    protected final Way createOrGetWay(LineString ls) {
+        Way w = null;
+        Way tempWay = new Way();
         if (ls != null) {
+            // Build list of nodes 
             for (int i=0; i<ls.getNumPoints(); i++) {
                 try {
-                    w.addNode(createOrGetNode(ls.getPointN(i)));
+                    tempWay.addNode(createOrGetNode(ls.getPointN(i)));
                 } catch (Exception e) {
                     Main.error(e.getMessage());
                 }
             }
+            // Find possible duplicated ways
+            if (tempWay.getNodesCount() > 0) {
+                List<Way> candidates = OsmPrimitive.getFilteredList(tempWay.firstNode().getReferrers(), Way.class);
+                candidates.remove(tempWay);
+                List<LatLon> tempNodes = DuplicateWay.getOrderedNodes(tempWay);
+                for (Way candidate : candidates) {
+                    List<LatLon> candNodesA = DuplicateWay.getOrderedNodes(candidate);
+                    List<LatLon> candNodesB = new ArrayList<>(candNodesA);
+                    Collections.reverse(candNodesB);
+                    if (tempNodes.equals(candNodesA) || tempNodes.equals(candNodesB)) {
+                        w = candidate;
+                        break;
+                    }
+                }
+            }
+        }
+        // If no duplicate way found, create new one
+        if (w == null) {
+            w = createWay();
+            w.setNodes(tempWay.getNodes());
         }
         return w;

applications/editors/josm/plugins/opendata/src/org/openstreetmap/josm/plugins/opendata/core/io/geographic/ShpReader.java

@@ -138 +138 @@
                                 r = createMultipolygon();
                             }
-                            w = createWay(p.getExteriorRing());
+                            w = createOrGetWay(p.getExteriorRing());
                             if (r != null) {
                                 addWayToMp(r, "outer", w);
                                 for (int j=0; j<p.getNumInteriorRing(); j++) {
-                                    addWayToMp(r, "inner", createWay(p.getInteriorRingN(j)));
+                                    addWayToMp(r, "inner", createOrGetWay(p.getInteriorRingN(j)));
                                 }
                             }
                         } else if (g instanceof LineString) {
-                            w = createWay((LineString) g);
+                            w = createOrGetWay((LineString) g);
                         } else if (g instanceof Point) {
                             // Some belgian data sets hold points into collections ?!