Changeset 15950 in josm for trunk/src/org


Ignore:
Timestamp:
2020-02-28T00:24:17+01:00 (5 years ago)
Author:
simon04
Message:

Simplify OsmUtils.isTrue/isFalse/isReversed

Just use a plain switch statement instead of a HashMap

File:
1 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/org/openstreetmap/josm/data/osm/OsmUtils.java

    r15692 r15950  
    22package org.openstreetmap.josm.data.osm;
    33
    4 import java.util.Arrays;
    54import java.util.Collection;
    6 import java.util.HashSet;
    75import java.util.Locale;
    86import java.util.Map;
    9 import java.util.Set;
    107import java.util.regex.Pattern;
    118import java.util.stream.Stream;
    … …  
    1916 */
    2017public final class OsmUtils {
    21 
    22     private static final Set<String> TRUE_VALUES = new HashSet<>(Arrays
    23             .asList("true", "yes", "1", "on"));
    24     private static final Set<String> FALSE_VALUES = new HashSet<>(Arrays
    25             .asList("false", "no", "0", "off"));
    26     private static final Set<String> REVERSE_VALUES = new HashSet<>(Arrays
    27             .asList("reverse", "-1"));
    2818
    2919    /**
    … …  
    7060        if (value == null) return null;
    7161        String lowerValue = value.toLowerCase(Locale.ENGLISH);
    72         if (TRUE_VALUES.contains(lowerValue)) return Boolean.TRUE;
    73         if (FALSE_VALUES.contains(lowerValue)) return Boolean.FALSE;
     62        if (isTrue(lowerValue)) return Boolean.TRUE;
     63        if (isFalse(lowerValue)) return Boolean.FALSE;
    7464        return null;
    7565    }
    … …  
    9383     */
    9484    public static boolean isReversed(String value) {
    95         return REVERSE_VALUES.contains(value);
     85        switch (value) {
     86            case "reverse":
     87            case "-1":
     88                return true;
     89            default:
     90                return false;
     91        }
    9692    }
    9793
    … …  
    10298     */
    10399    public static boolean isTrue(String value) {
    104         return TRUE_VALUES.contains(value);
     100        switch (value) {
     101            case "true":
     102            case "yes":
     103            case "1":
     104            case "on":
     105                return true;
     106            default:
     107                return false;
     108        }
    105109    }
    106110
    … …  
    111115     */
    112116    public static boolean isFalse(String value) {
    113         return FALSE_VALUES.contains(value);
     117        switch (value) {
     118            case "false":
     119            case "no":
     120            case "0":
     121            case "off":
     122                return true;
     123            default:
     124                return false;
     125        }
    114126    }
    115127
Note: See TracChangeset for help on using the changeset viewer.