Ignore:
Timestamp:
2017-04-15T01:38:46+02:00 (7 years ago)
Author:
Don-vip
Message:

sonar - squid:S1192 - String literals should not be duplicated

File:
1 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/org/openstreetmap/josm/io/imagery/ImageryReader.java

    r11884 r11913  
    9898
    9999    private static class Parser extends DefaultHandler {
     100        private static final String MAX_ZOOM = "max-zoom";
     101        private static final String MIN_ZOOM = "min-zoom";
     102        private static final String TILE_SIZE = "tile-size";
     103        private static final String TRUE = "true";
     104
    100105        private StringBuilder accumulator = new StringBuilder();
    101106
    … …  
    155160                    metadataHeaders = new HashMap<>();
    156161                    String best = atts.getValue("eli-best");
    157                     if ("true".equals(best)) {
     162                    if (TRUE.equals(best)) {
    158163                        entry.setBestMarked(true);
    159164                    }
    … …  
    164169                        "type",
    165170                        "url",
    166                         "min-zoom",
    167                         "max-zoom",
    168                         "tile-size",
     171                        MIN_ZOOM,
     172                        MAX_ZOOM,
     173                        TILE_SIZE,
    169174                }).contains(qName)) {
    170175                    newState = State.MIRROR_ATTRIBUTE;
    … …  
    184189                        "url",
    185190                        "eula",
    186                         "min-zoom",
    187                         "max-zoom",
     191                        MIN_ZOOM,
     192                        MAX_ZOOM,
    188193                        "attribution-text",
    189194                        "attribution-url",
    … …  
    195200                        "icon",
    196201                        "date",
    197                         "tile-size",
     202                        TILE_SIZE,
    198203                        "valid-georeference",
    199204                }).contains(qName)) {
    … …  
    264269            }
    265270            states.push(newState);
    266             if (newState == State.UNKNOWN && "true".equals(atts.getValue("mandatory"))) {
     271            if (newState == State.UNKNOWN && TRUE.equals(atts.getValue("mandatory"))) {
    267272                skipEntry = true;
    268273            }
    … …  
    319324                        mirrorEntry.setUrl(accumulator.toString());
    320325                        break;
    321                     case "min-zoom":
    322                     case "max-zoom":
     326                    case MIN_ZOOM:
     327                    case MAX_ZOOM:
    323328                        Integer val = null;
    324329                        try {
    … …  
    330335                            mirrorEntry = null;
    331336                        } else {
    332                             if ("min-zoom".equals(qName)) {
     337                            if (MIN_ZOOM.equals(qName)) {
    333338                                mirrorEntry.setDefaultMinZoom(val);
    334339                            } else {
    … …  
    337342                        }
    338343                        break;
    339                     case "tile-size":
     344                    case TILE_SIZE:
    340345                        Integer tileSize = null;
    341346                        try {
    … …  
    383388                case "default":
    384389                    switch (accumulator.toString()) {
    385                     case "true":
     390                    case TRUE:
    386391                        entry.setDefaultEntry(true);
    387392                        break;
    … …  
    399404                    entry.setEulaAcceptanceRequired(accumulator.toString());
    400405                    break;
    401                 case "min-zoom":
    402                 case "max-zoom":
     406                case MIN_ZOOM:
     407                case MAX_ZOOM:
    403408                    Integer val = null;
    404409                    try {
    … …  
    410415                        skipEntry = true;
    411416                    } else {
    412                         if ("min-zoom".equals(qName)) {
     417                        if (MIN_ZOOM.equals(qName)) {
    413418                            entry.setDefaultMinZoom(val);
    414419                        } else {
    … …  
    441446                    entry.setIcon(accumulator.toString());
    442447                    break;
    443                 case "tile-size":
     448                case TILE_SIZE:
    444449                    Integer tileSize = null;
    445450                    try {
Note: See TracChangeset for help on using the changeset viewer.