Changeset 5235 in josm


Ignore:
Timestamp:
2012-05-12T19:11:58+02:00 (8 years ago)
Author:
bastiK
Message:

no rounding for projection bounds, to avoid 42 being dispayed as 41.99999999999

Location:
trunk/src/org/openstreetmap/josm
Files:
19 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/org/openstreetmap/josm/data/Bounds.java

    r4580 r5235  
    4343    }
    4444
     45    public Bounds(LatLon min, LatLon max, boolean roundToOsmPrecision) {
     46        this(min.lat(), min.lon(), max.lat(), max.lon(), roundToOsmPrecision);
     47    }
     48
    4549    public Bounds(LatLon b) {
     50        this(b, true);
     51    }
     52   
     53    public Bounds(LatLon b, boolean roundToOsmPrecision) {
    4654        // Do not call this(b, b) to avoid GPX performance issue (see #7028) until roundToOsmPrecision() is improved
    47         this.minLat = LatLon.roundToOsmPrecision(b.lat());
    48         this.minLon = LatLon.roundToOsmPrecision(b.lon());
     55        if (roundToOsmPrecision) {
     56            this.minLat = LatLon.roundToOsmPrecision(b.lat());
     57            this.minLon = LatLon.roundToOsmPrecision(b.lon());
     58        } else {
     59            this.minLat = b.lat();
     60            this.minLon = b.lon();
     61        }
    4962        this.maxLat = this.minLat;
    5063        this.maxLon = this.minLon;
     
    5265
    5366    public Bounds(double minlat, double minlon, double maxlat, double maxlon) {
    54         this.minLat = LatLon.roundToOsmPrecision(minlat);
    55         this.minLon = LatLon.roundToOsmPrecision(minlon);
    56         this.maxLat = LatLon.roundToOsmPrecision(maxlat);
    57         this.maxLon = LatLon.roundToOsmPrecision(maxlon);
     67        this(minlat, minlon, maxlat, maxlon, true);
     68    }
     69
     70    public Bounds(double minlat, double minlon, double maxlat, double maxlon, boolean roundToOsmPrecision) {
     71        if (roundToOsmPrecision) {
     72            this.minLat = LatLon.roundToOsmPrecision(minlat);
     73            this.minLon = LatLon.roundToOsmPrecision(minlon);
     74            this.maxLat = LatLon.roundToOsmPrecision(maxlat);
     75            this.maxLon = LatLon.roundToOsmPrecision(maxlon);
     76        } else {
     77            this.minLat = minlat;
     78            this.minLon = minlon;
     79            this.maxLat = maxlat;
     80            this.maxLon = maxlon;
     81        }
    5882    }
    5983
    6084    public Bounds(double [] coords) {
     85        this(coords, true);
     86    }
     87
     88    public Bounds(double [] coords, boolean roundToOsmPrecision) {
    6189        CheckParameterUtil.ensureParameterNotNull(coords, "coords");
    6290        if (coords.length != 4)
    6391            throw new IllegalArgumentException(MessageFormat.format("Expected array of length 4, got {0}", coords.length));
    64         this.minLat = LatLon.roundToOsmPrecision(coords[0]);
    65         this.minLon = LatLon.roundToOsmPrecision(coords[1]);
    66         this.maxLat = LatLon.roundToOsmPrecision(coords[2]);
    67         this.maxLon = LatLon.roundToOsmPrecision(coords[3]);
     92        if (roundToOsmPrecision) {
     93            this.minLat = LatLon.roundToOsmPrecision(coords[0]);
     94            this.minLon = LatLon.roundToOsmPrecision(coords[1]);
     95            this.maxLat = LatLon.roundToOsmPrecision(coords[2]);
     96            this.maxLon = LatLon.roundToOsmPrecision(coords[3]);
     97        } else {
     98            this.minLat = coords[0];
     99            this.minLon = coords[1];
     100            this.maxLat = coords[2];
     101            this.maxLon = coords[3];
     102        }
    68103    }
    69104
     
    73108
    74109    public Bounds(String asString, String separator, ParseMethod parseMethod) throws IllegalArgumentException {
     110        this(asString, separator, parseMethod, true);
     111    }
     112
     113    public Bounds(String asString, String separator, ParseMethod parseMethod, boolean roundToOsmPrecision) throws IllegalArgumentException {
    75114        CheckParameterUtil.ensureParameterNotNull(asString, "asString");
    76115        String[] components = asString.split(separator);
     
    88127        switch (parseMethod) {
    89128            case LEFT_BOTTOM_RIGHT_TOP:
    90                 this.minLat = initLat(values[1]);
    91                 this.minLon = initLon(values[0]);
    92                 this.maxLat = initLat(values[3]);
    93                 this.maxLon = initLon(values[2]);
     129                this.minLat = initLat(values[1], roundToOsmPrecision);
     130                this.minLon = initLon(values[0], roundToOsmPrecision);
     131                this.maxLat = initLat(values[3], roundToOsmPrecision);
     132                this.maxLon = initLon(values[2], roundToOsmPrecision);
    94133                break;
    95134            case MINLAT_MINLON_MAXLAT_MAXLON:
    96135            default:
    97                 this.minLat = initLat(values[0]);
    98                 this.minLon = initLon(values[1]);
    99                 this.maxLat = initLat(values[2]);
    100                 this.maxLon = initLon(values[3]);
    101         }
    102     }
    103    
    104     protected static double initLat(double value) {
     136                this.minLat = initLat(values[0], roundToOsmPrecision);
     137                this.minLon = initLon(values[1], roundToOsmPrecision);
     138                this.maxLat = initLat(values[2], roundToOsmPrecision);
     139                this.maxLon = initLon(values[3], roundToOsmPrecision);
     140        }
     141    }
     142   
     143    protected static double initLat(double value, boolean roundToOsmPrecision) {
    105144        if (!LatLon.isValidLat(value))
    106145            throw new IllegalArgumentException(tr("Illegal latitude value ''{0}''", value));
    107         return LatLon.roundToOsmPrecision(value);
    108     }
    109 
    110     protected static double initLon(double value) {
     146        return roundToOsmPrecision ? LatLon.roundToOsmPrecision(value) : value;
     147    }
     148
     149    protected static double initLon(double value, boolean roundToOsmPrecision) {
    111150        if (!LatLon.isValidLon(value))
    112151            throw new IllegalArgumentException(tr("Illegal longitude value ''{0}''", value));
    113         return LatLon.roundToOsmPrecision(value);
     152        return roundToOsmPrecision ? LatLon.roundToOsmPrecision(value) : value;
    114153    }
    115154
  • trunk/src/org/openstreetmap/josm/data/coor/LatLon.java

    r4869 r5235  
    4545        // a comma separated list of coordinates.
    4646        cDdFormatter = (DecimalFormat) NumberFormat.getInstance(Locale.UK);
    47         cDdFormatter.applyPattern("###0.0000000");
     47        cDdFormatter.applyPattern("###0.0######");
    4848    }
    4949
  • trunk/src/org/openstreetmap/josm/data/projection/BelgianLambert1972.java

    r5072 r5235  
    4747        return new Bounds(
    4848                new LatLon(49.51, 2.54),
    49                 new LatLon(51.50, 6.40));
     49                new LatLon(51.50, 6.40), false);
    5050    }
    5151
  • trunk/src/org/openstreetmap/josm/data/projection/BelgianLambert2008.java

    r5072 r5235  
    4646        return new Bounds(
    4747                new LatLon(49.51, 2.54),
    48                 new LatLon(51.50, 6.40));
     48                new LatLon(51.50, 6.40), false);
    4949    }
    5050
  • trunk/src/org/openstreetmap/josm/data/projection/CustomProjection.java

    r5234 r5235  
    343343                parseAngle(numStr[0], "minlon (+bounds)"),
    344344                parseAngle(numStr[3], "maxlat (+bounds)"),
    345                 parseAngle(numStr[2], "maxlon (+bounds)"));
     345                parseAngle(numStr[2], "maxlon (+bounds)"), false);
    346346    }
    347347
  • trunk/src/org/openstreetmap/josm/data/projection/Epsg3008.java

    r5066 r5235  
    5454    public Bounds getWorldBoundsLatLon() {
    5555        return new Bounds(
    56                 new LatLon(55.2, 12.1),     // new LatLon(-90.0, -180.0),
    57                 new LatLon(62.26, 14.65));  // new LatLon(90.0, 180.0));
     56                new LatLon(55.2, 12.1),
     57                new LatLon(62.26, 14.65), false);
    5858    }
    5959
  • trunk/src/org/openstreetmap/josm/data/projection/Epsg4326.java

    r2516 r5235  
    4444        return new Bounds(
    4545                new LatLon(-90.0, -180.0),
    46                 new LatLon(90.0, 180.0));
     46                new LatLon(90.0, 180.0), false);
    4747    }
    4848
  • trunk/src/org/openstreetmap/josm/data/projection/GaussKrueger.java

    r5234 r5235  
    1717
    1818    private static Bounds[] bounds = {
    19         new Bounds(new LatLon(-5, 3.5), new LatLon(85, 8.5)),
    20         new Bounds(new LatLon(-5, 6.5), new LatLon(85, 11.5)),
    21         new Bounds(new LatLon(-5, 9.5), new LatLon(85, 14.5)),
    22         new Bounds(new LatLon(-5, 12.5), new LatLon(85, 17.5)),
     19        new Bounds(new LatLon(-5, 3.5), new LatLon(85, 8.5), false),
     20        new Bounds(new LatLon(-5, 6.5), new LatLon(85, 11.5), false),
     21        new Bounds(new LatLon(-5, 9.5), new LatLon(85, 14.5), false),
     22        new Bounds(new LatLon(-5, 12.5), new LatLon(85, 17.5), false),
    2323    };
    2424
  • trunk/src/org/openstreetmap/josm/data/projection/Lambert.java

    r5234 r5235  
    143143        Bounds b= new Bounds(
    144144                new LatLon(Math.max(zoneLimitsDegree[layoutZone][1] - cMaxOverlappingZonesDegree, Math.toDegrees(cMinLatZone1Radian)), Math.toDegrees(cMinLonZonesRadian)),
    145                 new LatLon(Math.min(zoneLimitsDegree[layoutZone][0] + cMaxOverlappingZonesDegree, Math.toDegrees(cMaxLatZone1Radian)), Math.toDegrees(cMaxLonZonesRadian)));
     145                new LatLon(Math.min(zoneLimitsDegree[layoutZone][0] + cMaxOverlappingZonesDegree, Math.toDegrees(cMaxLatZone1Radian)), Math.toDegrees(cMaxLonZonesRadian)),
     146                false);
    146147        return b;
    147148    }
  • trunk/src/org/openstreetmap/josm/data/projection/Lambert93.java

    r5066 r5235  
    4545        return new Bounds(
    4646                new LatLon(41.0, -5.5),
    47                 new LatLon(51.0, 10.2));
     47                new LatLon(51.0, 10.2), false);
    4848    }
    4949
  • trunk/src/org/openstreetmap/josm/data/projection/LambertCC9Zones.java

    r5234 r5235  
    9090        return new Bounds(
    9191                new LatLon(Math.max(medLatZone - 1.0 - cMaxOverlappingZones, cMinLatZonesDegree), -5.5),
    92                 new LatLon(Math.min(medLatZone + 1.0 + cMaxOverlappingZones, Math.toDegrees(cMaxLatZonesRadian)), 10.2));
     92                new LatLon(Math.min(medLatZone + 1.0 + cMaxOverlappingZones, Math.toDegrees(cMaxLatZonesRadian)), 10.2),
     93                false);
    9394    }
    9495
  • trunk/src/org/openstreetmap/josm/data/projection/LambertEST.java

    r5066 r5235  
    6363        return new Bounds(
    6464                new LatLon(56.05, 21.64),
    65                 new LatLon(61.13, 28.58));
     65                new LatLon(61.13, 28.58), false);
    6666    }
    6767
  • trunk/src/org/openstreetmap/josm/data/projection/Mercator.java

    r5066 r5235  
    6060        return new Bounds(
    6161                new LatLon(-85.05112877980659, -180.0),
    62                 new LatLon(85.05112877980659, 180.0));
     62                new LatLon(85.05112877980659, 180.0), false);
    6363    }
    6464
  • trunk/src/org/openstreetmap/josm/data/projection/Puwg.java

    r5234 r5235  
    121121            return new Bounds(
    122122                    new LatLon(49.00, 14.12),
    123                     new LatLon(54.84, 24.15));
     123                    new LatLon(54.84, 24.15), false);
    124124        }
    125125
     
    164164            return new Bounds(
    165165                    new LatLon(49.00, (3 * getZone()) - 1.5),
    166                     new LatLon(54.84, (3 * getZone()) + 1.5));
     166                    new LatLon(54.84, (3 * getZone()) + 1.5), false);
    167167        }
    168168
  • trunk/src/org/openstreetmap/josm/data/projection/SwissGrid.java

    r5234 r5235  
    6363    @Override
    6464    public Bounds getWorldBoundsLatLon() {
    65         return new Bounds(new LatLon(45.7, 5.7), new LatLon(47.9, 10.6));
     65        return new Bounds(new LatLon(45.7, 5.7), new LatLon(47.9, 10.6), false);
    6666    }
    6767
  • trunk/src/org/openstreetmap/josm/data/projection/TransverseMercatorLV.java

    r5066 r5235  
    5656        return new Bounds(
    5757                new LatLon(-90.0, -180.0),
    58                 new LatLon(90.0, 180.0));
     58                new LatLon(90.0, 180.0), false);
    5959    }
    6060}
  • trunk/src/org/openstreetmap/josm/data/projection/UTM.java

    r5234 r5235  
    100100            return new Bounds(
    101101                    new LatLon(-5.0, getUtmCentralMeridianDeg(getzone())-5.0),
    102                     new LatLon(85.0, getUtmCentralMeridianDeg(getzone())+5.0));
     102                    new LatLon(85.0, getUtmCentralMeridianDeg(getzone())+5.0), false);
    103103        else
    104104            return new Bounds(
    105105                    new LatLon(-85.0, getUtmCentralMeridianDeg(getzone())-5.0),
    106                     new LatLon(5.0, getUtmCentralMeridianDeg(getzone())+5.0));
     106                    new LatLon(5.0, getUtmCentralMeridianDeg(getzone())+5.0), false);
    107107    }
    108108
  • trunk/src/org/openstreetmap/josm/data/projection/UTM_France_DOM.java

    r5234 r5235  
    1919public class UTM_France_DOM extends AbstractProjection {
    2020
    21     private final static Bounds FortMarigotBounds = new Bounds( new LatLon(17.6,-63.25), new LatLon(18.5,-62.5));
    22     private final static Bounds SainteAnneBounds = new Bounds( new LatLon(15.8,-61.9), new LatLon(16.6,-60.9));
    23     private final static Bounds MartiniqueBounds = new Bounds( new LatLon(14.25,-61.25), new LatLon(15.025,-60.725));
    24     private final static Bounds ReunionBounds = new Bounds( new LatLon(-25.92,37.58), new LatLon(-10.6, 58.27));
    25     private final static Bounds GuyaneBounds = new Bounds( new LatLon(2.16 , -54.0), new LatLon(9.06 , -49.62));
     21    private final static Bounds FortMarigotBounds = new Bounds( new LatLon(17.6,-63.25), new LatLon(18.5,-62.5), false);
     22    private final static Bounds SainteAnneBounds = new Bounds( new LatLon(15.8,-61.9), new LatLon(16.6,-60.9), false);
     23    private final static Bounds MartiniqueBounds = new Bounds( new LatLon(14.25,-61.25), new LatLon(15.025,-60.725), false);
     24    private final static Bounds ReunionBounds = new Bounds( new LatLon(-25.92,37.58), new LatLon(-10.6, 58.27), false);
     25    private final static Bounds GuyaneBounds = new Bounds( new LatLon(2.16 , -54.0), new LatLon(9.06 , -49.62), false);
    2626    private final static Bounds[] utmBounds = { FortMarigotBounds, SainteAnneBounds, MartiniqueBounds, ReunionBounds, GuyaneBounds };
    2727
  • trunk/src/org/openstreetmap/josm/gui/preferences/projection/ProjectionPreference.java

    r5234 r5235  
    198198        Bounds b = proj.getWorldBoundsLatLon();
    199199        CoordinateFormat cf = CoordinateFormat.getDefaultFormat();
    200         bounds.setText(b.getMin().latToString(cf)+"; "+b.getMin().lonToString(cf)+" : "+b.getMax().latToString(cf)+"; "+b.getMax().lonToString(cf));
     200        bounds.setText(b.getMin().lonToString(cf)+", "+b.getMin().latToString(cf)+" : "+b.getMax().lonToString(cf)+", "+b.getMax().latToString(cf));
    201201        boolean showCode = true;
    202202        if (pc instanceof SubPrefsOptions) {
Note: See TracChangeset for help on using the changeset viewer.