文字列操作に対するクリーナーコードの提案

Question

私のコードの6行目のif文のロジックには、たくさんの考えがあります。しかし、私は経験豊かな開発者からのフィードバックを得ることを望んでいました。皆さんは私のコードが複雑すぎると思っていますか？もしそうなら、あなたはこれをより簡潔に書いていますか？個人的に

Answer 1

を好むでしょうか？はい、あなたは本当に私はずっと簡単に使用されるだろう:-)

です：

public String seeColor (String color) { 
    if (color.startsWith("red")) return "red"; 
    if (color.startsWith("blue")) return "blue"; 
    return ""; 
} 

次完全なプログラムを行動でそれを示しています

public class Test 
{ 
    public static String seeColor (String color) { 
     if (color.startsWith("red")) return "red"; 
     if (color.startsWith("blue")) return "blue"; 
     return ""; 
    } 

    public static void main(String[] args) { 
     String[] testData = { "redxx", "xxred", "blueTimes", "NoColor", 
      "red", "re", "blu", "blue", "a", "", "xyzred" }; 
     for (String s: testData) 
      System.out.println("[" + s + "] -> [" + seeColor(s) + "]"); 
    } 
} 

そのプログラムの存在の出力、予想通り：

[redxx] -> [red] 
[xxred] -> [] 
[blueTimes] -> [blue] 
[NoColor] -> [] 
[red] -> [red] 
[re] -> [] 
[blu] -> [] 
[blue] -> [blue] 
[a] -> [] 
[] -> [] 
[xyzred] -> [] 

---

あなたが将来的にそれが容易に拡張したい場合は、あなたのようなものを選ぶことができます：

その後

public static String seeColor (String color) { 
    String[] allow = {"red", "blue"}; 
    for (String s: allow) 
     if (color.startsWith(s)) 
      return s; 
    return ""; 
} 

、allow配列に色を追加すると、あなたがそれを認識しているために取る必要がある唯一のステップです。

Answer 2

、私はあなたが

if (str != null) { 
    if (str.equals("red")) return "red"; 
    else if (str.equals("blue")) return "blue"; 
} 
return ""; 

または少し複雑switch

if (str != null) { 
    switch (str) { 
    case "red": 
     return "red"; 
    case "blue"; 
     return "blue"; 
    } 
} 
return ""; 

テストlengthようなもので、それを簡素化し、その後にsubstring()が表示されますを実行するかもしれないと思います比較的高価な多くを実行するString私に無意味な操作。

編集

あなただけの文字を開始し、テストする必要がある場合は、私は以上に複雑

if (str != null) { 
    if (str.startsWith("red")) return "red"; 
    else if (str.startsWith("blue")) return "blue"; 
} 
return ""; 

Answer 3

簡単に枝場合は、新規に追加するのではなく拡張することができるように、ストリームであなたの色の名前をカプセル化したほうが良いだろうと私には思える：

return Stream.of("red", "blue", "green") 
    .filter(colourName::startsWith).findAny().orElse(""); 

注このソリューションが依存していることJava 8ストリームとメソッド参照について

Answer 4

正規表現溶液。 ^は文字列の一致開始を意味します。

public static String GetColor(String input) { 
    String[] colors = {"red", "blue"}; 
    for(String color : colors) { 
     if(input.matches(String.format("^%s.*", color))) 
      return color; 
    } 
    return ""; 
}