Ignore:
Timestamp:
2014-04-27T15:35:47+02:00 (6 years ago)
Author:
Don-vip
Message:

see #8465 - use String switch/case where applicable

Location:
trunk/src/org/openstreetmap/josm/io
Files:
17 edited

Legend:

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

    r6362 r7012  
    183183   
    184184    protected final boolean doStartElement(String qName, Attributes atts) throws SAXException {
    185         if (qName.equals("node")) {
     185        switch (qName) {
     186        case "node":
    186187            startNode(atts);
    187188            return true;
    188         } else if (qName.equals("way")) {
     189        case "way":
    189190            startWay(atts);
    190191            return true;
    191         } else if (qName.equals("relation")) {
     192        case "relation":
    192193            startRelation(atts);
    193194            return true;
    194         } else if (qName.equals("tag")) {
     195        case "tag":
    195196            handleTag(atts);
    196197            return true;
    197         } else if (qName.equals("nd")) {
     198        case "nd":
    198199            handleNodeReference(atts);
    199200            return true;
    200         } else if (qName.equals("member")) {
     201        case "member":
    201202            handleMember(atts);
    202203            return true;
    203         } else {
     204        default:
    204205            return false;
    205206        }
  • trunk/src/org/openstreetmap/josm/io/Capabilities.java

    r7005 r7012  
    8989
    9090    public void put(String element, String attribute, String value) {
    91         if (element.equals("blacklist")) {
    92             if (attribute.equals("regex")) {
     91        if ("blacklist".equals(element)) {
     92            if ("regex".equals(attribute)) {
    9393                imageryBlacklist.add(value);
    9494            }
  • trunk/src/org/openstreetmap/josm/io/ChangesetQuery.java

    r7005 r7012  
    2828
    2929    /**
    30      * Replies a changeset query object from the query part of a OSM API URL for querying
    31      * changesets.
     30     * Replies a changeset query object from the query part of a OSM API URL for querying changesets.
    3231     *
    3332     * @param query the query part
     
    348347        }
    349348
    350         protected boolean parseOpen(String value) throws ChangesetQueryUrlException {
    351             if (value == null || value.trim().isEmpty())
    352                 throw new ChangesetQueryUrlException(tr("Unexpected value for ''{0}'' in changeset query url, got {1}", "open",value));
    353             if (value.equals("true"))
    354                 return true;
    355             else if (value.equals("false"))
    356                 return false;
    357             else
    358                 throw new ChangesetQueryUrlException(tr("Unexpected value for ''{0}'' in changeset query url, got {1}", "open",value));
    359         }
    360 
    361349        protected boolean parseBoolean(String value, String parameter) throws ChangesetQueryUrlException {
    362350            if (value == null || value.trim().isEmpty())
    363351                throw new ChangesetQueryUrlException(tr("Unexpected value for ''{0}'' in changeset query url, got {1}", parameter,value));
    364             if (value.equals("true"))
     352            switch (value) {
     353            case "true":
    365354                return true;
    366             else if (value.equals("false"))
     355            case "false":
    367356                return false;
    368             else
     357            default:
    369358                throw new ChangesetQueryUrlException(tr("Unexpected value for ''{0}'' in changeset query url, got {1}", parameter,value));
     359            }
    370360        }
    371361
     
    414404            for (Entry<String, String> entry: queryParams.entrySet()) {
    415405                String k = entry.getKey();
    416                 if (k.equals("uid")) {
     406                switch(k) {
     407                case "uid":
    417408                    if (queryParams.containsKey("display_name"))
    418409                        throw new ChangesetQueryUrlException(tr("Cannot create a changeset query including both the query parameters ''uid'' and ''display_name''"));
    419410                    csQuery.forUser(parseUid(queryParams.get("uid")));
    420                 } else if (k.equals("display_name")) {
     411                    break;
     412                case "display_name":
    421413                    if (queryParams.containsKey("uid"))
    422414                        throw new ChangesetQueryUrlException(tr("Cannot create a changeset query including both the query parameters ''uid'' and ''display_name''"));
    423415                    csQuery.forUser(queryParams.get("display_name"));
    424                 } else if (k.equals("open")) {
    425                     boolean b = parseBoolean(entry.getValue(), "open");
    426                     csQuery.beingOpen(b);
    427                 } else if (k.equals("closed")) {
    428                     boolean b = parseBoolean(entry.getValue(), "closed");
    429                     csQuery.beingClosed(b);
    430                 } else if (k.equals("time")) {
     416                    break;
     417                case "open":
     418                    csQuery.beingOpen(parseBoolean(entry.getValue(), "open"));
     419                    break;
     420                case "closed":
     421                    csQuery.beingClosed(parseBoolean(entry.getValue(), "closed"));
     422                    break;
     423                case "time":
    431424                    Date[] dates = parseTime(entry.getValue());
    432425                    switch(dates.length) {
     
    438431                        break;
    439432                    }
    440                 } else if (k.equals("bbox")) {
     433                    break;
     434                case "bbox":
    441435                    try {
    442436                        csQuery.inBbox(new Bounds(entry.getValue(), ","));
     
    444438                        throw new ChangesetQueryUrlException(e);
    445439                    }
    446                 } else if (k.equals("changesets")) {
     440                    break;
     441                case "changesets":
    447442                    try {
    448443                        csQuery.forChangesetIds(parseLongs(entry.getValue()));
     
    450445                        throw new ChangesetQueryUrlException(e);
    451446                    }
    452                 } else
     447                    break;
     448                default:
    453449                    throw new ChangesetQueryUrlException(tr("Unsupported parameter ''{0}'' in changeset query string", k));
     450                }
    454451            }
    455452            return csQuery;
  • trunk/src/org/openstreetmap/josm/io/DiffResultProcessor.java

    r7005 r7012  
    148148        }
    149149
    150         @Override public void startElement(String namespaceURI, String localName, String qName, Attributes atts) throws SAXException {
     150        @Override
     151        public void startElement(String namespaceURI, String localName, String qName, Attributes atts) throws SAXException {
    151152            try {
    152                 if (qName.equals("diffResult")) {
     153                switch (qName) {
     154                case "diffResult":
    153155                    // the root element, ignore
    154                 } else if (qName.equals("node") || qName.equals("way") || qName.equals("relation")) {
    155 
     156                    break;
     157                case "node":
     158                case "way":
     159                case "relation":
    156160                    PrimitiveId id  = new SimplePrimitiveId(
    157161                            Long.parseLong(atts.getValue("old_id")),
     
    166170                    }
    167171                    diffResults.put(id, entry);
    168                 } else {
     172                    break;
     173                default:
    169174                    throwException(tr("Unexpected XML element with name ''{0}''", qName));
    170175                }
  • trunk/src/org/openstreetmap/josm/io/GpxReader.java

    r7005 r7012  
    9393        }
    9494
    95         @Override public void startElement(String namespaceURI, String localName, String qName, Attributes atts) throws SAXException {
     95        @Override
     96        public void startElement(String namespaceURI, String localName, String qName, Attributes atts) throws SAXException {
    9697            elements.push(localName);
    9798            switch(currentState) {
     
    109110                break;
    110111            case gpx:
    111                 if (localName.equals("metadata")) {
     112                switch (localName) {
     113                case "metadata":
    112114                    states.push(currentState);
    113115                    currentState = State.metadata;
    114                 } else if (localName.equals("wpt")) {
     116                    break;
     117                case "wpt":
    115118                    states.push(currentState);
    116119                    currentState = State.wpt;
    117120                    currentWayPoint = new WayPoint(parseLatLon(atts));
    118                 } else if (localName.equals("rte")) {
     121                    break;
     122                case "rte":
    119123                    states.push(currentState);
    120124                    currentState = State.rte;
    121125                    currentRoute = new GpxRoute();
    122                 } else if (localName.equals("trk")) {
     126                    break;
     127                case "trk":
    123128                    states.push(currentState);
    124129                    currentState = State.trk;
    125130                    currentTrack = new ArrayList<>();
    126131                    currentTrackAttr = new HashMap<>();
    127                 } else if (localName.equals("extensions")) {
     132                    break;
     133                case "extensions":
    128134                    states.push(currentState);
    129135                    currentState = State.ext;
    130136                    currentExtensions = new Extensions();
    131                 } else if (localName.equals("gpx") && atts.getValue("creator") != null && atts.getValue("creator").startsWith("Nokia Sports Tracker")) {
    132                     nokiaSportsTrackerBug = true;
     137                    break;
     138                case "gpx":
     139                    if (atts.getValue("creator") != null && atts.getValue("creator").startsWith("Nokia Sports Tracker")) {
     140                        nokiaSportsTrackerBug = true;
     141                    }
    133142                }
    134143                break;
    135144            case metadata:
    136                 if (localName.equals("author")) {
     145                switch (localName) {
     146                case "author":
    137147                    states.push(currentState);
    138148                    currentState = State.author;
    139                 } else if (localName.equals("extensions")) {
     149                    break;
     150                case "extensions":
    140151                    states.push(currentState);
    141152                    currentState = State.ext;
    142153                    currentExtensions = new Extensions();
    143                 } else if (localName.equals("copyright")) {
     154                    break;
     155                case "copyright":
    144156                    states.push(currentState);
    145157                    currentState = State.copyright;
    146158                    data.attr.put(META_COPYRIGHT_AUTHOR, atts.getValue("author"));
    147                 } else if (localName.equals("link")) {
     159                    break;
     160                case "link":
    148161                    states.push(currentState);
    149162                    currentState = State.link;
     
    152165                break;
    153166            case author:
    154                 if (localName.equals("link")) {
     167                switch (localName) {
     168                case "link":
    155169                    states.push(currentState);
    156170                    currentState = State.link;
    157171                    currentLink = new GpxLink(atts.getValue("href"));
    158                 } else if (localName.equals("email")) {
     172                    break;
     173                case "email":
    159174                    data.attr.put(META_AUTHOR_EMAIL, atts.getValue("id") + "@" + atts.getValue("domain"));
    160175                }
    161176                break;
    162177            case trk:
    163                 if (localName.equals("trkseg")) {
     178                switch (localName) {
     179                case "trkseg":
    164180                    states.push(currentState);
    165181                    currentState = State.trkseg;
    166182                    currentTrackSeg = new ArrayList<>();
    167                 } else if (localName.equals("link")) {
     183                    break;
     184                case "link":
    168185                    states.push(currentState);
    169186                    currentState = State.link;
    170187                    currentLink = new GpxLink(atts.getValue("href"));
    171                 } else if (localName.equals("extensions")) {
     188                    break;
     189                case "extensions":
    172190                    states.push(currentState);
    173191                    currentState = State.ext;
     
    176194                break;
    177195            case trkseg:
    178                 if (localName.equals("trkpt")) {
     196                if ("trkpt".equals(localName)) {
    179197                    states.push(currentState);
    180198                    currentState = State.wpt;
     
    183201                break;
    184202            case wpt:
    185                 if (localName.equals("link")) {
     203                switch (localName) {
     204                case "link":
    186205                    states.push(currentState);
    187206                    currentState = State.link;
    188207                    currentLink = new GpxLink(atts.getValue("href"));
    189                 } else if (localName.equals("extensions")) {
     208                    break;
     209                case "extensions":
    190210                    states.push(currentState);
    191211                    currentState = State.ext;
    192212                    currentExtensions = new Extensions();
     213                    break;
    193214                }
    194215                break;
    195216            case rte:
    196                 if (localName.equals("link")) {
     217                switch (localName) {
     218                case "link":
    197219                    states.push(currentState);
    198220                    currentState = State.link;
    199221                    currentLink = new GpxLink(atts.getValue("href"));
    200                 } else if (localName.equals("rtept")) {
     222                    break;
     223                case "rtept":
    201224                    states.push(currentState);
    202225                    currentState = State.wpt;
    203226                    currentWayPoint = new WayPoint(parseLatLon(atts));
    204                 } else if (localName.equals("extensions")) {
     227                    break;
     228                case "extensions":
    205229                    states.push(currentState);
    206230                    currentState = State.ext;
    207231                    currentExtensions = new Extensions();
     232                    break;
    208233                }
    209234                break;
     
    212237        }
    213238
    214         @Override public void characters(char[] ch, int start, int length) {
     239        @Override
     240        public void characters(char[] ch, int start, int length) {
    215241            /**
    216242             * Remove illegal characters generated by the Nokia Sports Tracker device.
     
    241267
    242268        @SuppressWarnings("unchecked")
    243         @Override public void endElement(String namespaceURI, String localName, String qName) {
     269        @Override
     270        public void endElement(String namespaceURI, String localName, String qName) {
    244271            elements.pop();
    245272            switch (currentState) {
    246273            case gpx:       // GPX 1.0
    247274            case metadata:  // GPX 1.1
    248                 if (localName.equals("name")) {
     275                switch (localName) {
     276                case "name":
    249277                    data.attr.put(META_NAME, accumulator.toString());
    250                 } else if (localName.equals("desc")) {
     278                    break;
     279                case "desc":
    251280                    data.attr.put(META_DESC, accumulator.toString());
    252                 } else if (localName.equals("time")) {
     281                    break;
     282                case "time":
    253283                    data.attr.put(META_TIME, accumulator.toString());
    254                 } else if (localName.equals("keywords")) {
     284                    break;
     285                case "keywords":
    255286                    data.attr.put(META_KEYWORDS, accumulator.toString());
    256                 } else if (version.equals("1.0") && localName.equals("author")) {
    257                     // author is a string in 1.0, but complex element in 1.1
     287                    break;
     288                case "author":
     289                    if ("1.0".equals(version)) {
     290                        // author is a string in 1.0, but complex element in 1.1
     291                        data.attr.put(META_AUTHOR_NAME, accumulator.toString());
     292                    }
     293                    break;
     294                case "email":
     295                    if ("1.0".equals(version)) {
     296                        data.attr.put(META_AUTHOR_EMAIL, accumulator.toString());
     297                    }
     298                    break;
     299                case "url":
     300                case "urlname":
     301                    data.attr.put(localName, accumulator.toString());
     302                    break;
     303                case "metadata":
     304                case "gpx":
     305                    if ((currentState == State.metadata && "metadata".equals(localName)) ||
     306                        (currentState == State.gpx && "gpx".equals(localName))) {
     307                        convertUrlToLink(data.attr);
     308                        if (currentExtensions != null && !currentExtensions.isEmpty()) {
     309                            data.attr.put(META_EXTENSIONS, currentExtensions);
     310                        }
     311                        currentState = states.pop();
     312                        break;
     313                    }
     314                default:
     315                    //TODO: parse bounds, extensions
     316                }
     317                break;
     318            case author:
     319                switch (localName) {
     320                case "author":
     321                    currentState = states.pop();
     322                    break;
     323                case "name":
    258324                    data.attr.put(META_AUTHOR_NAME, accumulator.toString());
    259                 } else if (version.equals("1.0") && localName.equals("email")) {
    260                     data.attr.put(META_AUTHOR_EMAIL, accumulator.toString());
    261                 } else if (localName.equals("url") || localName.equals("urlname")) {
    262                     data.attr.put(localName, accumulator.toString());
    263                 } else if ((currentState == State.metadata && localName.equals("metadata")) ||
    264                         (currentState == State.gpx && localName.equals("gpx"))) {
    265                     convertUrlToLink(data.attr);
    266                     if (currentExtensions != null && !currentExtensions.isEmpty()) {
    267                         data.attr.put(META_EXTENSIONS, currentExtensions);
    268                     }
    269                     currentState = states.pop();
    270                 }
    271                 //TODO: parse bounds, extensions
    272                 break;
    273             case author:
    274                 if (localName.equals("author")) {
    275                     currentState = states.pop();
    276                 } else if (localName.equals("name")) {
    277                     data.attr.put(META_AUTHOR_NAME, accumulator.toString());
    278                 } else if (localName.equals("email")) {
     325                    break;
     326                case "email":
    279327                    // do nothing, has been parsed on startElement
    280                 } else if (localName.equals("link")) {
     328                    break;
     329                case "link":
    281330                    data.attr.put(META_AUTHOR_LINK, currentLink);
     331                    break;
    282332                }
    283333                break;
    284334            case copyright:
    285                 if (localName.equals("copyright")) {
    286                     currentState = states.pop();
    287                 } else if (localName.equals("year")) {
     335                switch (localName) {
     336                case "copyright":
     337                    currentState = states.pop();
     338                    break;
     339                case "year":
    288340                    data.attr.put(META_COPYRIGHT_YEAR, accumulator.toString());
    289                 } else if (localName.equals("license")) {
     341                    break;
     342                case "license":
    290343                    data.attr.put(META_COPYRIGHT_LICENSE, accumulator.toString());
     344                    break;
    291345                }
    292346                break;
    293347            case link:
    294                 if (localName.equals("text")) {
     348                switch (localName) {
     349                case "text":
    295350                    currentLink.text = accumulator.toString();
    296                 } else if (localName.equals("type")) {
     351                    break;
     352                case "type":
    297353                    currentLink.type = accumulator.toString();
    298                 } else if (localName.equals("link")) {
     354                    break;
     355                case "link":
    299356                    if (currentLink.uri == null && accumulator != null && accumulator.toString().length() != 0) {
    300357                        currentLink = new GpxLink(accumulator.toString());
    301358                    }
    302359                    currentState = states.pop();
     360                    break;
    303361                }
    304362                if (currentState == State.author) {
     
    313371                break;
    314372            case wpt:
    315                 if (   localName.equals("ele")  || localName.equals("magvar")
    316                         || localName.equals("name") || localName.equals("src")
    317                         || localName.equals("geoidheight") || localName.equals("type")
    318                         || localName.equals("sym") || localName.equals("url")
    319                         || localName.equals("urlname")) {
     373                switch (localName) {
     374                case "ele":
     375                case "magvar":
     376                case "name":
     377                case "src":
     378                case "geoidheight":
     379                case "type":
     380                case "sym":
     381                case "url":
     382                case "urlname":
    320383                    currentWayPoint.attr.put(localName, accumulator.toString());
    321                 } else if(localName.equals("hdop") || localName.equals("vdop") ||
    322                         localName.equals("pdop")) {
     384                    break;
     385                case "hdop":
     386                case "vdop":
     387                case "pdop":
    323388                    try {
    324389                        currentWayPoint.attr.put(localName, Float.parseFloat(accumulator.toString()));
     
    326391                        currentWayPoint.attr.put(localName, new Float(0));
    327392                    }
    328                 } else if (localName.equals("time")) {
     393                    break;
     394                case "time":
    329395                    currentWayPoint.attr.put(localName, accumulator.toString());
    330396                    currentWayPoint.setTime();
    331                 } else if (localName.equals("cmt") || localName.equals("desc")) {
     397                    break;
     398                case "cmt":
     399                case "desc":
    332400                    currentWayPoint.attr.put(localName, accumulator.toString());
    333401                    currentWayPoint.setTime();
    334                 } else if (localName.equals("rtept")) {
     402                    break;
     403                case "rtept":
    335404                    currentState = states.pop();
    336405                    convertUrlToLink(currentWayPoint.attr);
    337406                    currentRoute.routePoints.add(currentWayPoint);
    338                 } else if (localName.equals("trkpt")) {
     407                    break;
     408                case "trkpt":
    339409                    currentState = states.pop();
    340410                    convertUrlToLink(currentWayPoint.attr);
    341411                    currentTrackSeg.add(currentWayPoint);
    342                 } else if (localName.equals("wpt")) {
     412                    break;
     413                case "wpt":
    343414                    currentState = states.pop();
    344415                    convertUrlToLink(currentWayPoint.attr);
     
    347418                    }
    348419                    data.waypoints.add(currentWayPoint);
     420                    break;
    349421                }
    350422                break;
    351423            case trkseg:
    352                 if (localName.equals("trkseg")) {
     424                if ("trkseg".equals(localName)) {
    353425                    currentState = states.pop();
    354426                    currentTrack.add(currentTrackSeg);
     
    356428                break;
    357429            case trk:
    358                 if (localName.equals("trk")) {
     430                switch (localName) {
     431                case "trk":
    359432                    currentState = states.pop();
    360433                    convertUrlToLink(currentTrackAttr);
    361434                    data.tracks.add(new ImmutableGpxTrack(currentTrack, currentTrackAttr));
    362                 } else if (localName.equals("name") || localName.equals("cmt")
    363                         || localName.equals("desc") || localName.equals("src")
    364                         || localName.equals("type") || localName.equals("number")
    365                         || localName.equals("url") || localName.equals("urlname")) {
     435                    break;
     436                case "name":
     437                case "cmt":
     438                case "desc":
     439                case "src":
     440                case "type":
     441                case "number":
     442                case "url":
     443                case "urlname":
    366444                    currentTrackAttr.put(localName, accumulator.toString());
     445                    break;
    367446                }
    368447                break;
    369448            case ext:
    370                 if (localName.equals("extensions")) {
    371                     currentState = states.pop();
    372                 // only interested in extensions written by JOSM
     449                if ("extensions".equals(localName)) {
     450                    currentState = states.pop();
    373451                } else if (JOSM_EXTENSIONS_NAMESPACE_URI.equals(namespaceURI)) {
     452                    // only interested in extensions written by JOSM
    374453                    currentExtensions.put(localName, accumulator.toString());
    375454                }
    376455                break;
    377456            default:
    378                 if (localName.equals("wpt")) {
    379                     currentState = states.pop();
    380                 } else if (localName.equals("rte")) {
     457                switch (localName) {
     458                case "wpt":
     459                    currentState = states.pop();
     460                    break;
     461                case "rte":
    381462                    currentState = states.pop();
    382463                    convertUrlToLink(currentRoute.attr);
    383464                    data.routes.add(currentRoute);
    384                 }
    385             }
    386         }
    387 
    388         @Override public void endDocument() throws SAXException  {
     465                    break;
     466                }
     467            }
     468        }
     469
     470        @Override
     471        public void endDocument() throws SAXException  {
    389472            if (!states.empty())
    390473                throw new SAXException(tr("Parse error: invalid document structure for GPX document."));
  • trunk/src/org/openstreetmap/josm/io/MirroredInputStream.java

    r7005 r7012  
    131131        try {
    132132            url = new URL(name);
    133             if (url.getProtocol().equals("file")) {
     133            if ("file".equals(url.getProtocol())) {
    134134                file = new File(name.substring("file:/".length()));
    135135                if (!file.exists()) {
     
    234234        try {
    235235            url = new URL(name);
    236             if (!url.getProtocol().equals("file")) {
     236            if (!"file".equals(url.getProtocol())) {
    237237                String prefKey = getPrefKey(url, destDir);
    238238                List<String> localPath = new ArrayList<>(Main.pref.getCollection(prefKey));
  • trunk/src/org/openstreetmap/josm/io/NmeaReader.java

    r7005 r7012  
    264264
    265265            // handle the packet content
    266             if(e[0].equals("$GPGGA") || e[0].equals("$GNGGA")) {
     266            if("$GPGGA".equals(e[0]) || "$GNGGA".equals(e[0])) {
    267267                // Position
    268268                LatLon latLon = parseLatLon(
     
    299299                // elevation
    300300                accu=e[GPGGA.HEIGHT_UNTIS.position];
    301                 if(accu.equals("M")) {
     301                if("M".equals(accu)) {
    302302                    // Ignore heights that are not in meters for now
    303303                    accu=e[GPGGA.HEIGHT.position];
     
    345345                    }
    346346                }
    347             } else if(e[0].equals("$GPVTG") || e[0].equals("$GNVTG")) {
     347            } else if("$GPVTG".equals(e[0]) || "$GNVTG".equals(e[0])) {
    348348                // COURSE
    349349                accu = e[GPVTG.COURSE_REF.position];
    350                 if(accu.equals("T")) {
     350                if("T".equals(accu)) {
    351351                    // other values than (T)rue are ignored
    352352                    accu = e[GPVTG.COURSE.position];
     
    366366                    }
    367367                }
    368             } else if(e[0].equals("$GPGSA") || e[0].equals("$GNGSA")) {
     368            } else if("$GPGSA".equals(e[0]) || "$GNGSA".equals(e[0])) {
    369369                // vdop
    370370                accu=e[GPGSA.VDOP.position];
     
    383383                }
    384384            }
    385             else if(e[0].equals("$GPRMC") || e[0].equals("$GNRMC")) {
     385            else if("$GPRMC".equals(e[0]) || "$GNRMC".equals(e[0])) {
    386386                // coordinates
    387387                LatLon latLon = parseLatLon(
  • trunk/src/org/openstreetmap/josm/io/OsmApi.java

    r7005 r7012  
    620620                }
    621621
    622                 if (requestMethod.equals("PUT") || requestMethod.equals("POST") || requestMethod.equals("DELETE")) {
     622                if ("PUT".equals(requestMethod) || "POST".equals(requestMethod) || "DELETE".equals(requestMethod)) {
    623623                    activeConnection.setDoOutput(true);
    624624                    activeConnection.setRequestProperty("Content-type", "text/xml");
  • trunk/src/org/openstreetmap/josm/io/OsmChangeReader.java

    r6881 r7012  
    3434    @Override
    3535    protected void parseRoot() throws XMLStreamException {
    36         if (parser.getLocalName().equals("osmChange")) {
     36        if ("osmChange".equals(parser.getLocalName())) {
    3737            parseOsmChange();
    3838        } else {
     
    6969            if (event == XMLStreamConstants.START_ELEMENT) {
    7070                OsmPrimitive p = null;
    71                 if (parser.getLocalName().equals("node")) {
     71                switch (parser.getLocalName()) {
     72                case "node":
    7273                    p = parseNode();
    73                 } else if (parser.getLocalName().equals("way")) {
     74                    break;
     75                case "way":
    7476                    p = parseWay();
    75                 } else if (parser.getLocalName().equals("relation")) {
     77                    break;
     78                case "relation":
    7679                    p = parseRelation();
    77                 } else {
     80                    break;
     81                default:
    7882                    parseUnknown();
    7983                }
    8084                if (p != null && action != null) {
    81                     if (action.equals("modify")) {
     85                    if ("modify".equals(action)) {
    8286                        p.setModified(true);
    83                     } else if (action.equals("delete")) {
     87                    } else if ("delete".equals(action)) {
    8488                        p.setDeleted(true);
    8589                    }
  • trunk/src/org/openstreetmap/josm/io/OsmChangesetContentParser.java

    r7004 r7012  
    4747        }
    4848
    49         @Override public void startElement(String namespaceURI, String localName, String qName, Attributes atts) throws SAXException {
     49        @Override
     50        public void startElement(String namespaceURI, String localName, String qName, Attributes atts) throws SAXException {
    5051            if (super.doStartElement(qName, atts)) {
    5152                // done
    52             } else if (qName.equals("osmChange")) {
     53                return;
     54            }
     55            switch (qName) {
     56            case "osmChange":
    5357                // do nothing
    54             } else if (qName.equals("create")) {
     58                break;
     59            case "create":
    5560                currentModificationType = ChangesetModificationType.CREATED;
    56             } else if (qName.equals("modify")) {
     61                break;
     62            case "modify":
    5763                currentModificationType = ChangesetModificationType.UPDATED;
    58             } else if (qName.equals("delete")) {
     64                break;
     65            case "delete":
    5966                currentModificationType = ChangesetModificationType.DELETED;
    60             } else {
    61                 Main.warn(tr("Unsupported start element ''{0}'' in changeset content at position ({1},{2}). Skipping.", qName, locator.getLineNumber(), locator.getColumnNumber()));
     67                break;
     68            default:
     69                Main.warn(tr("Unsupported start element ''{0}'' in changeset content at position ({1},{2}). Skipping.",
     70                        qName, locator.getLineNumber(), locator.getColumnNumber()));
    6271            }
    6372        }
     
    6574        @Override
    6675        public void endElement(String uri, String localName, String qName) throws SAXException {
    67             if (qName.equals("node")
    68                     || qName.equals("way")
    69                     || qName.equals("relation")) {
     76            switch (qName) {
     77            case "node":
     78            case "way":
     79            case "relation":
    7080                if (currentModificationType == null) {
    7181                    throwException(tr("Illegal document structure. Found node, way, or relation outside of ''create'', ''modify'', or ''delete''."));
    7282                }
    7383                data.put(currentPrimitive, currentModificationType);
    74             } else if (qName.equals("osmChange")) {
     84                break;
     85            case "osmChange":
    7586                // do nothing
    76             } else if (qName.equals("create")) {
     87                break;
     88            case "create":
    7789                currentModificationType = null;
    78             } else if (qName.equals("modify")) {
     90                break;
     91            case "modify":
    7992                currentModificationType = null;
    80             } else if (qName.equals("delete")) {
     93                break;
     94            case "delete":
    8195                currentModificationType = null;
    82             } else if (qName.equals("tag")) {
     96                break;
     97            case "tag":
     98            case "nd":
     99            case "member":
    83100                // do nothing
    84             } else if (qName.equals("nd")) {
    85                 // do nothing
    86             } else if (qName.equals("member")) {
    87                 // do nothing
    88             } else {
    89                 Main.warn(tr("Unsupported end element ''{0}'' in changeset content at position ({1},{2}). Skipping.", qName, locator.getLineNumber(), locator.getColumnNumber()));
     101                break;
     102            default:
     103                Main.warn(tr("Unsupported end element ''{0}'' in changeset content at position ({1},{2}). Skipping.",
     104                        qName, locator.getLineNumber(), locator.getColumnNumber()));
    90105            }
    91106        }
  • trunk/src/org/openstreetmap/josm/io/OsmChangesetParser.java

    r7005 r7012  
    112112            if (value == null) {
    113113                throwException(tr("Missing mandatory attribute ''{0}''.", "open"));
    114             } else if (value.equals("true")) {
     114            } else if ("true".equals(value)) {
    115115                current.setOpen(true);
    116             } else if (value.equals("false")) {
     116            } else if ("false".equals(value)) {
    117117                current.setOpen(false);
    118118            } else {
     
    160160        @Override
    161161        public void startElement(String namespaceURI, String localName, String qName, Attributes atts) throws SAXException {
    162             if (qName.equals("osm")) {
     162            switch (qName) {
     163            case "osm":
    163164                if (atts == null) {
    164165                    throwException(tr("Missing mandatory attribute ''{0}'' of XML element {1}.", "version", "osm"));
     
    168169                    throwException(tr("Missing mandatory attribute ''{0}''.", "version"));
    169170                }
    170                 if (!(v.equals("0.6"))) {
     171                if (!("0.6".equals(v))) {
    171172                    throwException(tr("Unsupported version: {0}", v));
    172173                }
    173             } else if (qName.equals("changeset")) {
     174                break;
     175            case "changeset":
    174176                current = new Changeset();
    175177                parseChangesetAttributes(current, atts);
    176             } else if (qName.equals("tag")) {
     178                break;
     179            case "tag":
    177180                String key = atts.getValue("k");
    178181                String value = atts.getValue("v");
    179182                current.put(key, value);
    180             } else {
     183                break;
     184            default:
    181185                throwException(tr("Undefined element ''{0}'' found in input stream. Aborting.", qName));
    182186            }
     
    185189        @Override
    186190        public void endElement(String uri, String localName, String qName) throws SAXException {
    187             if (qName.equals("changeset")) {
     191            if ("changeset".equals(qName)) {
    188192                changesets.add(current);
    189193            }
  • trunk/src/org/openstreetmap/josm/io/OsmConnection.java

    r6643 r7012  
    131131    protected void addAuth(HttpURLConnection connection) throws OsmTransferException {
    132132        String authMethod = Main.pref.get("osm-server.auth-method", "basic");
    133         if (authMethod.equals("basic")) {
     133        if ("basic".equals(authMethod)) {
    134134            addBasicAuthorizationHeader(connection);
    135         } else if (authMethod.equals("oauth")) {
     135        } else if ("oauth".equals(authMethod)) {
    136136            addOAuthAuthorizationHeader(connection);
    137137        } else {
  • trunk/src/org/openstreetmap/josm/io/OsmHistoryReader.java

    r6643 r7012  
    5555        @Override
    5656        public void endElement(String uri, String localName, String qName) throws SAXException {
    57             if (qName.equals("node")
    58                     || qName.equals("way")
    59                     || qName.equals("relation")) {
     57            if ("node".equals(qName)
     58                    || "way".equals(qName)
     59                    || "relation".equals(qName)) {
    6060                data.put(currentPrimitive);
    6161            }
  • trunk/src/org/openstreetmap/josm/io/OsmReader.java

    r7005 r7012  
    109109
    110110    protected void parseRoot() throws XMLStreamException {
    111         if (parser.getLocalName().equals("osm")) {
     111        if ("osm".equals(parser.getLocalName())) {
    112112            parseOsm();
    113113        } else {
     
    121121            throwException(tr("Missing mandatory attribute ''{0}''.", "version"));
    122122        }
    123         if (!(v.equals("0.5") || v.equals("0.6"))) {
     123        if (!("0.5".equals(v) || "0.6".equals(v))) {
    124124            throwException(tr("Unsupported version: {0}", v));
    125125        }
     
    143143
    144144            if (event == XMLStreamConstants.START_ELEMENT) {
    145                 if (parser.getLocalName().equals("bounds")) {
     145                switch (parser.getLocalName()) {
     146                case "bounds":
    146147                    parseBounds(generator);
    147                 } else if (parser.getLocalName().equals("node")) {
     148                    break;
     149                case "node":
    148150                    parseNode();
    149                 } else if (parser.getLocalName().equals("way")) {
     151                    break;
     152                case "way":
    150153                    parseWay();
    151                 } else if (parser.getLocalName().equals("relation")) {
     154                    break;
     155                case "relation":
    152156                    parseRelation();
    153                 } else if (parser.getLocalName().equals("changeset")) {
     157                    break;
     158                case "changeset":
    154159                    parseChangeset(uploadChangesetId);
    155                 } else {
     160                    break;
     161                default:
    156162                    parseUnknown();
    157163                }
     
    205211            int event = parser.next();
    206212            if (event == XMLStreamConstants.START_ELEMENT) {
    207                 if (parser.getLocalName().equals("tag")) {
     213                if ("tag".equals(parser.getLocalName())) {
    208214                    parseTag(n);
    209215                } else {
     
    227233            int event = parser.next();
    228234            if (event == XMLStreamConstants.START_ELEMENT) {
    229                 if (parser.getLocalName().equals("nd")) {
     235                switch (parser.getLocalName()) {
     236                case "nd":
    230237                    nodeIds.add(parseWayNode(w));
    231                 } else if (parser.getLocalName().equals("tag")) {
     238                    break;
     239                case "tag":
    232240                    parseTag(w);
    233                 } else {
     241                    break;
     242                default:
    234243                    parseUnknown();
    235244                }
     
    274283            int event = parser.next();
    275284            if (event == XMLStreamConstants.START_ELEMENT) {
    276                 if (parser.getLocalName().equals("member")) {
     285                switch (parser.getLocalName()) {
     286                case "member":
    277287                    members.add(parseRelationMember(r));
    278                 } else if (parser.getLocalName().equals("tag")) {
     288                    break;
     289                case "tag":
    279290                    parseTag(r);
    280                 } else {
     291                    break;
     292                default:
    281293                    parseUnknown();
    282294                }
     
    337349                int event = parser.next();
    338350                if (event == XMLStreamConstants.START_ELEMENT) {
    339                     if (parser.getLocalName().equals("tag")) {
     351                    if ("tag".equals(parser.getLocalName())) {
    340352                        parseTag(uploadChangeset);
    341353                    } else {
     
    447459                throwException(tr("Illegal value for attribute ''version'' on OSM primitive with ID {0}. Got {1}.", Long.toString(current.getUniqueId()), versionString), e);
    448460            }
    449             if (ds.getVersion().equals("0.6")){
     461            switch (ds.getVersion()) {
     462            case "0.6":
    450463                if (version <= 0 && !current.isNew()) {
    451464                    throwException(tr("Illegal value for attribute ''version'' on OSM primitive with ID {0}. Got {1}.", Long.toString(current.getUniqueId()), versionString));
     
    454467                    version = 0;
    455468                }
    456             } else if (ds.getVersion().equals("0.5")) {
     469                break;
     470            case "0.5":
    457471                if (version <= 0 && !current.isNew()) {
    458472                    Main.warn(tr("Normalizing value of attribute ''version'' of element {0} to {2}, API version is ''{3}''. Got {1}.", current.getUniqueId(), version, 1, "0.5"));
     
    462476                    version = 0;
    463477                }
    464             } else {
     478                break;
     479            default:
    465480                // should not happen. API version has been checked before
    466481                throwException(tr("Unknown or unsupported API version. Got {0}.", ds.getVersion()));
     
    469484            // version expected for OSM primitives with an id assigned by the server (id > 0), since API 0.6
    470485            //
    471             if (!current.isNew() && ds.getVersion() != null && ds.getVersion().equals("0.6")) {
     486            if (!current.isNew() && ds.getVersion() != null && "0.6".equals(ds.getVersion())) {
    472487                throwException(tr("Missing attribute ''version'' on OSM primitive with ID {0}.", Long.toString(current.getUniqueId())));
    473             } else if (!current.isNew() && ds.getVersion() != null && ds.getVersion().equals("0.5")) {
     488            } else if (!current.isNew() && ds.getVersion() != null && "0.5".equals(ds.getVersion())) {
    474489                // default version in 0.5 files for existing primitives
    475490                Main.warn(tr("Normalizing value of attribute ''version'' of element {0} to {2}, API version is ''{3}''. Got {1}.", current.getUniqueId(), version, 1, "0.5"));
    476491                version= 1;
    477             } else if (current.isNew() && ds.getVersion() != null && ds.getVersion().equals("0.5")) {
    478                 // default version in 0.5 files for new primitives, no warning necessary. This is
    479                 // (was) legal in API 0.5
     492            } else if (current.isNew() && ds.getVersion() != null && "0.5".equals(ds.getVersion())) {
     493                // default version in 0.5 files for new primitives, no warning necessary. This is (was) legal in API 0.5
    480494                version= 0;
    481495            }
     
    486500        if (action == null) {
    487501            // do nothing
    488         } else if (action.equals("delete")) {
     502        } else if ("delete".equals(action)) {
    489503            current.setDeleted(true);
    490504            current.setModified(current.isVisible());
    491         } else if (action.equals("modify")) {
     505        } else if ("modify".equals(action)) {
    492506            current.setModified(true);
    493507        }
  • trunk/src/org/openstreetmap/josm/io/imagery/ImageryReader.java

    r7005 r7012  
    9898            switch (states.peek()) {
    9999            case INIT:
    100                 if (qName.equals("imagery")) {
     100                if ("imagery".equals(qName)) {
    101101                    newState = State.IMAGERY;
    102102                }
    103103                break;
    104104            case IMAGERY:
    105                 if (qName.equals("entry")) {
     105                if ("entry".equals(qName)) {
    106106                    entry = new ImageryInfo();
    107107                    skipEntry = false;
     
    128128                }).contains(qName)) {
    129129                    newState = State.ENTRY_ATTRIBUTE;
    130                 } else if (qName.equals("bounds")) {
     130                } else if ("bounds".equals(qName)) {
    131131                    try {
    132132                        bounds = new ImageryBounds(
     
    139139                    }
    140140                    newState = State.BOUNDS;
    141                 } else if (qName.equals("projections")) {
     141                } else if ("projections".equals(qName)) {
    142142                    projections = new ArrayList<>();
    143143                    newState = State.PROJECTIONS;
     
    145145                break;
    146146            case BOUNDS:
    147                 if (qName.equals("shape")) {
     147                if ("shape".equals(qName)) {
    148148                    shape = new Shape();
    149149                    newState = State.SHAPE;
     
    151151                break;
    152152            case SHAPE:
    153                 if (qName.equals("point")) {
     153                if ("point".equals(qName)) {
    154154                    try {
    155155                        shape.addPoint(atts.getValue("lat"), atts.getValue("lon"));
     
    160160                break;
    161161            case PROJECTIONS:
    162                 if (qName.equals("code")) {
     162                if ("code".equals(qName)) {
    163163                    newState = State.CODE;
    164164                }
     
    192192                throw new RuntimeException("parsing error: more closing than opening elements");
    193193            case ENTRY:
    194                 if (qName.equals("entry")) {
     194                if ("entry".equals(qName)) {
    195195                    if (!skipEntry) {
    196196                        entries.add(entry);
     
    200200                break;
    201201            case ENTRY_ATTRIBUTE:
    202                 if (qName.equals("name")) {
     202                switch(qName) {
     203                case "name":
    203204                    entry.setTranslatedName(accumulator.toString());
    204                 } else if (qName.equals("type")) {
     205                    break;
     206                case "type":
    205207                    boolean found = false;
    206208                    for (ImageryType type : ImageryType.values()) {
     
    214216                        skipEntry = true;
    215217                    }
    216                 } else if (qName.equals("default")) {
    217                     if (accumulator.toString().equals("true")) {
     218                    break;
     219                case "default":
     220                    switch (accumulator.toString()) {
     221                    case "true":
    218222                        entry.setDefaultEntry(true);
    219                     } else if (accumulator.toString().equals("false")) {
     223                        break;
     224                    case "false":
    220225                        entry.setDefaultEntry(false);
    221                     } else {
     226                        break;
     227                    default:
    222228                        skipEntry = true;
    223229                    }
    224                 } else if (qName.equals("url")) {
     230                    break;
     231                case "url":
    225232                    entry.setUrl(accumulator.toString());
    226                 } else if (qName.equals("eula")) {
     233                    break;
     234                case "eula":
    227235                    entry.setEulaAcceptanceRequired(accumulator.toString());
    228                 } else if (qName.equals("min-zoom") || qName.equals("max-zoom")) {
     236                    break;
     237                case "min-zoom":
     238                case "max-zoom":
    229239                    Integer val = null;
    230240                    try {
     
    236246                        skipEntry = true;
    237247                    } else {
    238                         if (qName.equals("min-zoom")) {
     248                        if ("min-zoom".equals(qName)) {
    239249                            entry.setDefaultMinZoom(val);
    240250                        } else {
     
    242252                        }
    243253                    }
    244                 } else if (qName.equals("attribution-text")) {
     254                    break;
     255                case "attribution-text":
    245256                    entry.setAttributionText(accumulator.toString());
    246                 } else if (qName.equals("attribution-url")) {
     257                    break;
     258                case "attribution-url":
    247259                    entry.setAttributionLinkURL(accumulator.toString());
    248                 } else if (qName.equals("logo-image")) {
     260                    break;
     261                case "logo-image":
    249262                    entry.setAttributionImage(accumulator.toString());
    250                 } else if (qName.equals("logo-url")) {
     263                    break;
     264                case "logo-url":
    251265                    entry.setAttributionImageURL(accumulator.toString());
    252                 } else if (qName.equals("terms-of-use-text")) {
     266                    break;
     267                case "terms-of-use-text":
    253268                    entry.setTermsOfUseText(accumulator.toString());
    254                 } else if (qName.equals("terms-of-use-url")) {
     269                    break;
     270                case "terms-of-use-url":
    255271                    entry.setTermsOfUseURL(accumulator.toString());
    256                 } else if (qName.equals("country-code")) {
     272                    break;
     273                case "country-code":
    257274                    entry.setCountryCode(accumulator.toString());
    258                 } else if (qName.equals("icon")) {
     275                    break;
     276                case "icon":
    259277                    entry.setIcon(accumulator.toString());
     278                    break;
    260279                }
    261280                break;
  • trunk/src/org/openstreetmap/josm/io/imagery/WMSGrabber.java

    r7005 r7012  
    9292            e = ne.lon();
    9393        }
    94         if (myProj.equals("EPSG:4326") && !info.getServerProjections().contains(myProj) && info.getServerProjections().contains("CRS:84")) {
     94        if ("EPSG:4326".equals(myProj) && !info.getServerProjections().contains(myProj) && info.getServerProjections().contains("CRS:84")) {
    9595            myProj = "CRS:84";
    9696        }
     
    114114        if (baseURL.toLowerCase().contains("crs=epsg:4326")) {
    115115            switchLatLon = true;
    116         } else if (baseURL.toLowerCase().contains("crs=") && myProj.equals("EPSG:4326")) {
     116        } else if (baseURL.toLowerCase().contains("crs=") && "EPSG:4326".equals(myProj)) {
    117117            switchLatLon = true;
    118118        }
  • trunk/src/org/openstreetmap/josm/io/session/GeoImageSessionImporter.java

    r7005 r7012  
    4545                            Element attrElem = (Element) attrNode;
    4646                            try {
    47                                 String attrElemName = attrElem.getTagName();
    48                                 if ("file".equals(attrElemName)) {
     47                                switch(attrElem.getTagName()) {
     48                                case "file":
    4949                                    entry.setFile(new File(attrElem.getTextContent()));
    50                                 } else if ("position".equals(attrElemName)) {
     50                                    break;
     51                                case "position":
    5152                                    double lat = Double.parseDouble(attrElem.getAttribute("lat"));
    5253                                    double lon = Double.parseDouble(attrElem.getAttribute("lon"));
    5354                                    entry.setPos(new LatLon(lat, lon));
    54                                 } else if ("speed".equals(attrElemName)) {
     55                                    break;
     56                                case "speed":
    5557                                    entry.setSpeed(Double.parseDouble(attrElem.getTextContent()));
    56                                 } else if ("elevation".equals(attrElemName)) {
     58                                    break;
     59                                case "elevation":
    5760                                    entry.setElevation(Double.parseDouble(attrElem.getTextContent()));
    58                                 } else if ("gps-time".equals(attrElemName)) {
     61                                    break;
     62                                case "gps-time":
    5963                                    entry.setGpsTime(new Date(Long.parseLong(attrElem.getTextContent())));
    60                                 } else if ("exif-orientation".equals(attrElemName)) {
     64                                    break;
     65                                case "exif-orientation":
    6166                                    entry.setExifOrientation(Integer.parseInt(attrElem.getTextContent()));
    62                                 } else if ("exif-time".equals(attrElemName)) {
     67                                    break;
     68                                case "exif-time":
    6369                                    entry.setExifTime(new Date(Long.parseLong(attrElem.getTextContent())));
    64                                 } else if ("exif-gps-time".equals(attrElemName)) {
     70                                    break;
     71                                case "exif-gps-time":
    6572                                    entry.setExifGpsTime(new Date(Long.parseLong(attrElem.getTextContent())));
    66                                 } else if ("exif-coordinates".equals(attrElemName)) {
    67                                     double lat = Double.parseDouble(attrElem.getAttribute("lat"));
    68                                     double lon = Double.parseDouble(attrElem.getAttribute("lon"));
    69                                     entry.setExifCoor(new LatLon(lat, lon));
    70                                 } else if ("exif-image-direction".equals(attrElemName)) {
     73                                    break;
     74                                case "exif-coordinates":
     75                                    entry.setExifCoor(new LatLon(
     76                                            Double.parseDouble(attrElem.getAttribute("lat")),
     77                                            Double.parseDouble(attrElem.getAttribute("lon"))));
     78                                    break;
     79                                case "exif-image-direction":
    7180                                    entry.setExifImgDir(Double.parseDouble(attrElem.getTextContent()));
    72                                 } else if ("is-new-gps-data".equals(attrElemName) && Boolean.parseBoolean(attrElem.getTextContent())) {
    73                                     entry.flagNewGpsData();
     81                                    break;
     82                                case "is-new-gps-data":
     83                                    if (Boolean.parseBoolean(attrElem.getTextContent())) {
     84                                        entry.flagNewGpsData();
     85                                    }
    7486                                }
    7587                                // TODO: handle thumbnail loading
Note: See TracChangeset for help on using the changeset viewer.