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

see #8465 - use String switch/case where applicable

File:
1 edited

Legend:

Unmodified
Added
Removed

  • 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        }
Note: See TracChangeset for help on using the changeset viewer.