Ignore:
Timestamp:
2015-05-18T23:34:11+02:00 (5 years ago)
Author:
Don-vip
Message:
  • global use of String.isEmpty()
  • Correctness - Method throws alternative exception from catch block without history
Location:
trunk/src/org/openstreetmap/josm/io
Files:
12 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/org/openstreetmap/josm/io/ChangesetQuery.java

    r8390 r8394  
    315315
    316316        /**
     317         * Constructs a new {@code ChangesetQueryUrlException} with the specified cause and detail message.
     318         *
     319         * @param message the detail message. The detail message is saved for later retrieval by the {@link #getMessage()} method.
     320         * @param  cause the cause (which is saved for later retrieval by the {@link #getCause()} method).
     321         *         (A <tt>null</tt> value is permitted, and indicates that the cause is nonexistent or unknown.)
     322         */
     323        public ChangesetQueryUrlException(String message, Throwable cause) {
     324            super(message, cause);
     325        }
     326
     327        /**
    317328         * Constructs a new {@code ChangesetQueryUrlException} with the specified cause and a detail message of
    318329         * <tt>(cause==null ? null : cause.toString())</tt> (which typically contains the class and detail message of <tt>cause</tt>).
     
    329340        protected int parseUid(String value) throws ChangesetQueryUrlException {
    330341            if (value == null || value.trim().isEmpty())
    331                 throw new ChangesetQueryUrlException(tr("Unexpected value for ''{0}'' in changeset query url, got {1}", "uid", value));
     342                throw new ChangesetQueryUrlException(
     343                        tr("Unexpected value for ''{0}'' in changeset query url, got {1}", "uid", value));
    332344            int id;
    333345            try {
    334346                id = Integer.parseInt(value);
    335347                if (id <= 0)
    336                     throw new ChangesetQueryUrlException(tr("Unexpected value for ''{0}'' in changeset query url, got {1}", "uid", value));
     348                    throw new ChangesetQueryUrlException(
     349                            tr("Unexpected value for ''{0}'' in changeset query url, got {1}", "uid", value));
    337350            } catch(NumberFormatException e) {
    338                 throw new ChangesetQueryUrlException(tr("Unexpected value for ''{0}'' in changeset query url, got {1}", "uid", value));
     351                throw new ChangesetQueryUrlException(
     352                        tr("Unexpected value for ''{0}'' in changeset query url, got {1}", "uid", value), e);
    339353            }
    340354            return id;
     
    343357        protected boolean parseBoolean(String value, String parameter) throws ChangesetQueryUrlException {
    344358            if (value == null || value.trim().isEmpty())
    345                 throw new ChangesetQueryUrlException(tr("Unexpected value for ''{0}'' in changeset query url, got {1}", parameter, value));
     359                throw new ChangesetQueryUrlException(
     360                        tr("Unexpected value for ''{0}'' in changeset query url, got {1}", parameter, value));
    346361            switch (value) {
    347362            case "true":
     
    350365                return false;
    351366            default:
    352                 throw new ChangesetQueryUrlException(tr("Unexpected value for ''{0}'' in changeset query url, got {1}", parameter, value));
     367                throw new ChangesetQueryUrlException(
     368                        tr("Unexpected value for ''{0}'' in changeset query url, got {1}", parameter, value));
    353369            }
    354370        }
     
    356372        protected Date parseDate(String value, String parameter) throws ChangesetQueryUrlException {
    357373            if (value == null || value.trim().isEmpty())
    358                 throw new ChangesetQueryUrlException(tr("Unexpected value for ''{0}'' in changeset query url, got {1}", parameter, value));
     374                throw new ChangesetQueryUrlException(
     375                        tr("Unexpected value for ''{0}'' in changeset query url, got {1}", parameter, value));
    359376            DateFormat formatter = DateUtils.newIsoDateTimeFormat();
    360377            try {
    361378                return formatter.parse(value);
    362379            } catch(ParseException e) {
    363                 throw new ChangesetQueryUrlException(tr("Unexpected value for ''{0}'' in changeset query url, got {1}", parameter, value));
     380                throw new ChangesetQueryUrlException(
     381                        tr("Unexpected value for ''{0}'' in changeset query url, got {1}", parameter, value), e);
    364382            }
    365383        }
     
    368386            String[] dates = value.split(",");
    369387            if (dates == null || dates.length == 0 || dates.length > 2)
    370                 throw new ChangesetQueryUrlException(tr("Unexpected value for ''{0}'' in changeset query url, got {1}", "time", value));
     388                throw new ChangesetQueryUrlException(
     389                        tr("Unexpected value for ''{0}'' in changeset query url, got {1}", "time", value));
    371390            if (dates.length == 1)
    372391                return new Date[]{parseDate(dates[0], "time")};
     
    395414                case "uid":
    396415                    if (queryParams.containsKey("display_name"))
    397                         throw new ChangesetQueryUrlException(tr("Cannot create a changeset query including both the query parameters ''uid'' and ''display_name''"));
     416                        throw new ChangesetQueryUrlException(
     417                                tr("Cannot create a changeset query including both the query parameters ''uid'' and ''display_name''"));
    398418                    csQuery.forUser(parseUid(queryParams.get("uid")));
    399419                    break;
    400420                case "display_name":
    401421                    if (queryParams.containsKey("uid"))
    402                         throw new ChangesetQueryUrlException(tr("Cannot create a changeset query including both the query parameters ''uid'' and ''display_name''"));
     422                        throw new ChangesetQueryUrlException(
     423                                tr("Cannot create a changeset query including both the query parameters ''uid'' and ''display_name''"));
    403424                    csQuery.forUser(queryParams.get("display_name"));
    404425                    break;
     
    435456                    break;
    436457                default:
    437                     throw new ChangesetQueryUrlException(tr("Unsupported parameter ''{0}'' in changeset query string", k));
     458                    throw new ChangesetQueryUrlException(
     459                            tr("Unsupported parameter ''{0}'' in changeset query string", k));
    438460                }
    439461            }
  • trunk/src/org/openstreetmap/josm/io/DefaultProxySelector.java

    r7187 r8394  
    111111    public final void initFromPreferences() {
    112112        String value = Main.pref.get(ProxyPreferencesPanel.PROXY_POLICY);
    113         if (value.length() == 0) {
     113        if (value.isEmpty()) {
    114114            proxyPolicy = ProxyPolicy.NO_PROXY;
    115115        } else {
  • trunk/src/org/openstreetmap/josm/io/OsmApi.java

    r8390 r8394  
    361361            osm.setChangesetId(getChangeset().getId());
    362362        } catch(NumberFormatException e){
    363             throw new OsmTransferException(tr("Unexpected format of ID replied by the server. Got ''{0}''.", ret));
     363            throw new OsmTransferException(tr("Unexpected format of ID replied by the server. Got ''{0}''.", ret), e);
    364364        }
    365365    }
     
    384384        } catch(NumberFormatException e) {
    385385            throw new OsmTransferException(tr("Unexpected format of new version of modified primitive ''{0}''. Got ''{1}''.",
    386                     osm.getId(), ret));
     386                    osm.getId(), ret), e);
    387387        }
    388388    }
     
    427427                changeset.setOpen(true);
    428428            } catch(NumberFormatException e){
    429                 throw new OsmTransferException(tr("Unexpected format of ID replied by the server. Got ''{0}''.", ret));
     429                throw new OsmTransferException(tr("Unexpected format of ID replied by the server. Got ''{0}''.", ret), e);
    430430            }
    431431            progressMonitor.setCustomText(tr("Successfully opened changeset {0}",changeset.getId()));
  • trunk/src/org/openstreetmap/josm/io/OsmChangeImporter.java

    r6716 r8394  
    3535    }
    3636
    37     @Override public void importData(File file, ProgressMonitor progressMonitor) throws IOException, IllegalDataException {
     37    @Override
     38    public void importData(File file, ProgressMonitor progressMonitor) throws IOException, IllegalDataException {
    3839        try {
    3940            importData(Compression.getUncompressedFileInputStream(file), file, progressMonitor);
    4041        } catch (FileNotFoundException e) {
    4142            Main.error(e);
    42             throw new IOException(tr("File ''{0}'' does not exist.", file.getName()));
     43            throw new IOException(tr("File ''{0}'' does not exist.", file.getName()), e);
    4344        }
    4445    }
  • trunk/src/org/openstreetmap/josm/io/OsmServerUserInfoReader.java

    r7509 r8394  
    5252                userInfo.setId(Integer.parseInt(v));
    5353            } catch(NumberFormatException e) {
    54                 throw new XmlParsingException(tr("Illegal value for attribute ''{0}'' on XML tag ''{1}''. Got {2}.", "id", "user", v));
     54                throw new XmlParsingException(tr("Illegal value for attribute ''{0}'' on XML tag ''{1}''. Got {2}.", "id", "user", v), e);
    5555            }
    5656            // -- display name
     
    7777                    lat = Double.parseDouble(v);
    7878                } catch(NumberFormatException e) {
    79                     throw new XmlParsingException(tr("Illegal value for attribute ''{0}'' on XML tag ''{1}''. Got {2}.", "lat", "home", v));
     79                    throw new XmlParsingException(tr("Illegal value for attribute ''{0}'' on XML tag ''{1}''. Got {2}.",
     80                            "lat", "home", v), e);
    8081                }
    8182
     
    8788                    lon = Double.parseDouble(v);
    8889                } catch(NumberFormatException e) {
    89                     throw new XmlParsingException(tr("Illegal value for attribute ''{0}'' on XML tag ''{1}''. Got {2}.", "lon", "home", v));
     90                    throw new XmlParsingException(tr("Illegal value for attribute ''{0}'' on XML tag ''{1}''. Got {2}.",
     91                            "lon", "home", v), e);
    9092                }
    9193
     
    9799                    zoom = Integer.parseInt(v);
    98100                } catch(NumberFormatException e) {
    99                     throw new XmlParsingException(tr("Illegal value for attribute ''{0}'' on XML tag ''{1}''. Got {2}.", "zoom", "home", v));
     101                    throw new XmlParsingException(tr("Illegal value for attribute ''{0}'' on XML tag ''{1}''. Got {2}.",
     102                            "zoom", "home", v), e);
    100103                }
    101104                userInfo.setHome(new LatLon(lat,lon));
  • trunk/src/org/openstreetmap/josm/io/ProgressInputStream.java

    r6380 r8394  
    3838            progressMonitor.finishTask();
    3939            if (con.getHeaderField("Error") != null)
    40                 throw new OsmTransferException(tr(con.getHeaderField("Error")));
     40                throw new OsmTransferException(tr(con.getHeaderField("Error")), e);
    4141            throw new OsmTransferException(e);
    4242        }
  • trunk/src/org/openstreetmap/josm/io/remotecontrol/DNSName.java

    r8291 r8394  
    7171     */
    7272    public DNSName(String name) throws IOException {
    73         if (name == null || name.length() == 0)
     73        if (name == null || name.isEmpty())
    7474            throw new IOException("DNS name must not be null");
    7575        if (name.indexOf(' ') != -1)
  • trunk/src/org/openstreetmap/josm/io/remotecontrol/handler/AddNodeHandler.java

    r7937 r8394  
    3030    private double lat;
    3131    private double lon;
    32    
     32
    3333    private Node node;
    3434
     
    4747        return new String[] { "lat", "lon" };
    4848    }
    49    
     49
    5050    @Override
    5151    public String[] getOptionalParams()
     
    6363        return new String[] {
    6464            "/add_node?lat=11&lon=22",
    65             "/add_node?lon=13.3&lat=53.2&addtags=natural=tree|name=%20%20%20==Great%20Oak==" 
     65            "/add_node?lon=13.3&lat=53.2&addtags=natural=tree|name=%20%20%20==Great%20Oak=="
    6666        };
    6767    }
    68    
     68
    6969    @Override
    7070    public String getPermissionMessage() {
     
    122122            lon = Double.parseDouble(args.get("lon"));
    123123        } catch (NumberFormatException e) {
    124             throw new RequestHandlerBadRequestException("NumberFormatException ("+e.getMessage()+")");
     124            throw new RequestHandlerBadRequestException("NumberFormatException ("+e.getMessage()+")", e);
    125125        }
    126126        if (!Main.main.hasEditLayer()) {
  • trunk/src/org/openstreetmap/josm/io/remotecontrol/handler/AddWayHandler.java

    r8332 r8394  
    104104                allCoordinates.add(new LatLon(lat, lon));
    105105            } catch (NumberFormatException e) {
    106                 throw new RequestHandlerBadRequestException("NumberFormatException ("+e.getMessage()+")");
     106                throw new RequestHandlerBadRequestException("NumberFormatException ("+e.getMessage()+")", e);
    107107            }
    108108        }
  • trunk/src/org/openstreetmap/josm/io/remotecontrol/handler/LoadAndZoomHandler.java

    r8318 r8394  
    272272            maxlon = LatLon.roundToOsmPrecision(Double.parseDouble(args.get("right")));
    273273        } catch (NumberFormatException e) {
    274             throw new RequestHandlerBadRequestException("NumberFormatException ("+e.getMessage()+")");
     274            throw new RequestHandlerBadRequestException("NumberFormatException ("+e.getMessage()+")", e);
    275275        }
    276276
  • trunk/src/org/openstreetmap/josm/io/remotecontrol/handler/RequestHandler.java

    r8338 r8394  
    223223        if(mandatory != null) for (String key : mandatory) {
    224224            String value = args.get(key);
    225             if ((value == null) || (value.length() == 0)) {
     225            if (value == null || value.isEmpty()) {
    226226                error = true;
    227227                Main.warn("'" + myCommand + "' remote control request must have '" + key + "' parameter");
  • trunk/src/org/openstreetmap/josm/io/session/SessionReader.java

    r8390 r8394  
    179179                    return new BufferedInputStream(new FileInputStream(file));
    180180                } catch (FileNotFoundException e) {
    181                     throw new IOException(tr("File ''{0}'' does not exist.", file.getPath()));
     181                    throw new IOException(tr("File ''{0}'' does not exist.", file.getPath()), e);
    182182                }
    183183            } else if (inZipPath != null) {
Note: See TracChangeset for help on using the changeset viewer.