Use a switch-case instead of if-spaghettis

5 years ago · e1a8a3b752
parent 6b54086590
commit e1a8a3b752
1 changed files with 9 additions and 10 deletions
--- a/airsonic-main/src/main/java/org/airsonic/player/util/StringUtil.java
+++ b/airsonic-main/src/main/java/org/airsonic/player/util/StringUtil.java
@ -267,18 +267,17 @@ public final class StringUtil {
        }

        String[] elements = s.split("_");
-
-        if (elements.length == 0) {
+        switch (elements.length) {
+            case 0:
                return new Locale(s, "", "");
-        }
-        if (elements.length == 1) {
+            case 1:
                return new Locale(elements[0], "", "");
-        }
-        if (elements.length == 2) {
+            case 2:
                return new Locale(elements[0], elements[1], "");
-        }
+            default:
                return new Locale(elements[0], elements[1], elements[2]);
        }
+    }

    /**
     * URL-encodes the input value using UTF-8.