Changeset 7864 in josm for trunk/src/org/openstreetmap/josm/gui/mappaint/Range.java

Timestamp:

2014-12-20T22:43:20+01:00 (9 years ago)

Author:

Don-vip

Message:

global cleanup of IllegalArgumentExceptions thrown by JOSM

File:

• trunk/src/org/openstreetmap/josm/gui/mappaint/Range.java (modified) (3 diffs)

trunk/src/org/openstreetmap/josm/gui/mappaint/Range.java

@@ -13 +13 @@
     public static final Range ZERO_TO_INFINITY = new Range(0.0, Double.POSITIVE_INFINITY);
 
+    /**
+     * Constructs a new {@code Range}.
+     * @param lower Lower bound. Must be positive or zero
+     * @param upper Upper bound
+     * @throws IllegalArgumentException if the range is invalid ({@code lower < 0 || lower >= upper})
+     */
     public Range(double lower, double upper) {
         if (lower < 0 || lower >= upper)
-            throw new IllegalArgumentException();
+            throw new IllegalArgumentException("Invalid range: "+lower+"-"+upper);
         this.lower = lower;
         this.upper = upper;
@@ -29 +35 @@
     public static Range cut(Range a, Range b) {
         if (b.lower >= a.upper || b.upper <= a.lower)
-            throw new IllegalArgumentException();
+            throw new IllegalArgumentException("Ranges do not overlap: "+a+" - "+b);
         return new Range(Math.max(a.lower, b.lower), Math.min(a.upper, b.upper));
     }
@@ -49 +55 @@
     public Range reduceAround(double x, Range other) {
         if (!contains(x))
-            throw new IllegalArgumentException();
+            throw new IllegalArgumentException(x+" is not inside "+this);
         if (other.contains(x))
-            throw new IllegalArgumentException();
+            throw new IllegalArgumentException(x+" is inside "+other);
 
         if (x < other.lower && other.lower < upper)