From e1a8a3b752b82c8ffd3a3bd3412ad7e903217647 Mon Sep 17 00:00:00 2001 From: jvoisin Date: Sun, 13 Oct 2019 22:45:00 +0200 Subject: [PATCH] Use a switch-case instead of if-spaghettis --- .../org/airsonic/player/util/StringUtil.java | 19 +++++++++---------- 1 file changed, 9 insertions(+), 10 deletions(-) diff --git a/airsonic-main/src/main/java/org/airsonic/player/util/StringUtil.java b/airsonic-main/src/main/java/org/airsonic/player/util/StringUtil.java index 987e2b62..ed796630 100644 --- 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,17 +267,16 @@ public final class StringUtil { } String[] elements = s.split("_"); - - if (elements.length == 0) { - return new Locale(s, "", ""); - } - if (elements.length == 1) { - return new Locale(elements[0], "", ""); - } - if (elements.length == 2) { - return new Locale(elements[0], elements[1], ""); + switch (elements.length) { + case 0: + return new Locale(s, "", ""); + case 1: + return new Locale(elements[0], "", ""); + case 2: + return new Locale(elements[0], elements[1], ""); + default: + return new Locale(elements[0], elements[1], elements[2]); } - return new Locale(elements[0], elements[1], elements[2]); } /**