Ignore:
Timestamp:
2014-04-19T00:36:43+02:00 (6 years ago)
Author:
Don-vip
Message:

sonar - place Strings literals on the left side when checking for equality (can avoid NPEs)

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

Legend:

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

    r6907 r6990  
    165165            String url = st.nextToken();
    166166
    167             if (!method.equals("GET")) {
     167            if (!"GET".equals(method)) {
    168168                sendNotImplemented(out);
    169169                return;
  • trunk/src/org/openstreetmap/josm/io/session/GeoImageSessionImporter.java

    r6450 r6990  
    3737            if (imgNode.getNodeType() == Node.ELEMENT_NODE) {
    3838                Element imgElem = (Element) imgNode;
    39                 if (imgElem.getTagName().equals("geoimage")) {
     39                if ("geoimage".equals(imgElem.getTagName())) {
    4040                    ImageEntry entry = new ImageEntry();
    4141                    NodeList attrNodes = imgElem.getChildNodes();
     
    4545                            Element attrElem = (Element) attrNode;
    4646                            try {
    47                                 if (attrElem.getTagName().equals("file")) {
     47                                String attrElemName = attrElem.getTagName();
     48                                if ("file".equals(attrElemName)) {
    4849                                    entry.setFile(new File(attrElem.getTextContent()));
    49                                 } else if (attrElem.getTagName().equals("position")) {
     50                                } else if ("position".equals(attrElemName)) {
    5051                                    double lat = Double.parseDouble(attrElem.getAttribute("lat"));
    5152                                    double lon = Double.parseDouble(attrElem.getAttribute("lon"));
    5253                                    entry.setPos(new LatLon(lat, lon));
    53                                 } else if (attrElem.getTagName().equals("speed")) {
     54                                } else if ("speed".equals(attrElemName)) {
    5455                                    entry.setSpeed(Double.parseDouble(attrElem.getTextContent()));
    55                                 } else if (attrElem.getTagName().equals("elevation")) {
     56                                } else if ("elevation".equals(attrElemName)) {
    5657                                    entry.setElevation(Double.parseDouble(attrElem.getTextContent()));
    57                                 } else if (attrElem.getTagName().equals("gps-time")) {
     58                                } else if ("gps-time".equals(attrElemName)) {
    5859                                    entry.setGpsTime(new Date(Long.parseLong(attrElem.getTextContent())));
    59                                 } else if (attrElem.getTagName().equals("exif-orientation")) {
     60                                } else if ("exif-orientation".equals(attrElemName)) {
    6061                                    entry.setExifOrientation(Integer.parseInt(attrElem.getTextContent()));
    61                                 } else if (attrElem.getTagName().equals("exif-time")) {
     62                                } else if ("exif-time".equals(attrElemName)) {
    6263                                    entry.setExifTime(new Date(Long.parseLong(attrElem.getTextContent())));
    63                                 } else if (attrElem.getTagName().equals("exif-gps-time")) {
     64                                } else if ("exif-gps-time".equals(attrElemName)) {
    6465                                    entry.setExifGpsTime(new Date(Long.parseLong(attrElem.getTextContent())));
    65                                 } else if (attrElem.getTagName().equals("exif-coordinates")) {
     66                                } else if ("exif-coordinates".equals(attrElemName)) {
    6667                                    double lat = Double.parseDouble(attrElem.getAttribute("lat"));
    6768                                    double lon = Double.parseDouble(attrElem.getAttribute("lon"));
    6869                                    entry.setExifCoor(new LatLon(lat, lon));
    69                                 } else if (attrElem.getTagName().equals("exif-image-direction")) {
     70                                } else if ("exif-image-direction".equals(attrElemName)) {
    7071                                    entry.setExifImgDir(Double.parseDouble(attrElem.getTextContent()));
    71                                 } else if (attrElem.getTagName().equals("is-new-gps-data") && Boolean.parseBoolean(attrElem.getTextContent())) {
     72                                } else if ("is-new-gps-data".equals(attrElemName) && Boolean.parseBoolean(attrElem.getTextContent())) {
    7273                                    entry.flagNewGpsData();
    7374                                }
     
    7980                    }
    8081                    entries.add(entry);
    81                 } else if (imgElem.getTagName().equals("show-thumbnails")) {
     82                } else if ("show-thumbnails".equals(imgElem.getTagName())) {
    8283                    useThumbs = Boolean.parseBoolean(imgElem.getTextContent());
    8384                }
Note: See TracChangeset for help on using the changeset viewer.