Changeset 13670 in josm for trunk/src/org/openstreetmap

Timestamp:

2018-04-23T23:13:03+02:00 (7 years ago)

Author:

Don-vip

Message:

fix #16189 - Add "almost square check" for buildings (patch by marxin, modified)

Location:

trunk/src/org/openstreetmap/josm

Files:

• actions/OrthogonalizeAction.java (modified) (2 diffs)
• data/osm/Way.java (modified) (2 diffs)
• data/validation/OsmValidator.java (modified) (2 diffs)
• data/validation/tests/RightAngleBuildingTest.java (added)
• tools/Geometry.java (modified) (1 diff)

trunk/src/org/openstreetmap/josm/actions/OrthogonalizeAction.java

@@ -188 +188 @@
      * @return a rectifying command
      * @throws InvalidUserInputException if the selection is invalid
-     */
-    static SequenceCommand orthogonalize(Iterable<OsmPrimitive> selection) throws InvalidUserInputException {
+     * @since 13670
+     */
+    public static SequenceCommand orthogonalize(Iterable<OsmPrimitive> selection) throws InvalidUserInputException {
         final List<Node> nodeList = new ArrayList<>();
         final List<WayData> wayDataList = new ArrayList<>();
@@ -619 +620 @@
     /**
      * Exception: unsuited user input
-     */
-    protected static class InvalidUserInputException extends Exception {
+     * @since 13670
+     */
+    public static final class InvalidUserInputException extends Exception {
         InvalidUserInputException(String message) {
             super(message);

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

@@ -16 +16 @@
 import org.openstreetmap.josm.spi.preferences.Config;
 import org.openstreetmap.josm.tools.CopyList;
+import org.openstreetmap.josm.tools.Geometry;
 import org.openstreetmap.josm.tools.Pair;
 import org.openstreetmap.josm.tools.Utils;
@@ -780 +781 @@
         }
     }
+
+    /**
+     * Returns angles of vertices.
+     * @return angles of the way
+     * @since 13670
+     */
+    public synchronized List<Pair<Double, Node>> getAngles() {
+        List<Pair<Double, Node>> angles = new ArrayList<>();
+
+        for (int i = 1; i < nodes.length - 1; i++) {
+            Node n0 = nodes[i - 1];
+            Node n1 = nodes[i];
+            Node n2 = nodes[i + 1];
+
+            double angle = Geometry.getNormalizedAngleInDegrees(Geometry.getCornerAngle(
+                    n0.getEastNorth(), n1.getEastNorth(), n2.getEastNorth()));
+            angles.add(new Pair<>(angle, n1));
+        }
+
+        angles.add(new Pair<>(Geometry.getNormalizedAngleInDegrees(Geometry.getCornerAngle(
+                nodes[nodes.length - 2].getEastNorth(),
+                nodes[0].getEastNorth(),
+                nodes[1].getEastNorth())), nodes[0]));
+
+        return angles;
+    }
 }

trunk/src/org/openstreetmap/josm/data/validation/OsmValidator.java

@@ -53 +53 @@
 import org.openstreetmap.josm.data.validation.tests.PublicTransportRouteTest;
 import org.openstreetmap.josm.data.validation.tests.RelationChecker;
+import org.openstreetmap.josm.data.validation.tests.RightAngleBuildingTest;
 import org.openstreetmap.josm.data.validation.tests.SelfIntersectingWay;
 import org.openstreetmap.josm.data.validation.tests.SimilarNamedWays;
@@ -141 +142 @@
         LongSegment.class, // 3500 .. 3599
         PublicTransportRouteTest.class, // 3600 .. 3699
+        RightAngleBuildingTest.class, // 3700 .. 3799
     };
 

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

@@ -794 +794 @@
 
         return result;
+    }
+
+    /** 
+     * Get angles in radians and return it's value in range [0, 180].
+     *
+     * @param angle the angle in radians
+     * @return normalized angle in degrees
+     * @since 13670
+     */
+    public static double getNormalizedAngleInDegrees(double angle) {
+        return Math.abs(180 * angle / Math.PI);
     }