IfElseまたはregex

Question

非常に長いIfElseを使用しているときに、これを行う最善の方法はどれですか？

 if (text.contains("text")) 
     { 
      // do the thing 
     } 
     else if (text.contains("foo")) 
     { 
      // do the thing 
     } 
     else if (text.contains("bar")) 
     { 
      // do the thing 
     }else ... 

それとも

 if (text.contains("text") || text.contains("foo") || ...) 
     { 
      // do the thing 
     } 

それとも

 Pattern pattern = Pattern.compile("(text)|(foo)|(bar)|..."); 
     Matcher matcher = pattern.matcher(text); 
     if(matcher.find()) 
     { 
      // do the thing 
     } 

そして私はあなたがこれらの多くをチェックする必要があるときにだけ意味。ありがとう！

Answer 1

通常elseIf長い文がcase文に置き換えられ、これは常に可能なわけではありません。もし私がお勧めするなら、私は2番目のオプションに行くだろう。オプション1は、同じことをするIf else if elseステートメントの束を与えるだろう.3番目のケースでは、正規表現はかなりかなり大きく成長する傾向がある。

また、alotの量によっては、データ構造内のすべての文字列をスローし、繰り返してその要素がその中にあるかどうかを確認することができます。

Answer 2

String[] storage = { 
    "text", 
    "foo", 
    "bar", 
    "more text" 
}; 

for(int i=0; i < storage.length(); i++){ 
    //Do Something 
} 

これは役に立ちますか？あなたは、コンパクトな、それが好きなら

Set<String> keywords = new HashSet<String>(); 
keywords.add("text"); 
keywords.add("foo"); 
keywords.add("bar"); 

if(keywords.contains(text)) { 
    //do your thing 
} 

をそして、あなたも書くことができます：

Answer 3

（1）私は個人的に私は読みやすいと思いとしてセットを使用するとcontainsはOで効率的になります

を

Set<String> keywords = new HashSet<String>(Arrays.asList("text", "foo", "bar")); 

if(keywords.contains(text)) { 
    //do your thing 
} 

最後に、同じリストを常に使用する場合は、メソッドを実行するたびにキーワードを再作成する代わりに、private static finalを作成することができます。コメント続き

EDIT
、どのような上記ではtext.equals("xxx")、ないtext.contains("xxx")と条件を使用することと等価であることは事実です。あなたが本当に含まれている使用することを意図している場合、あなたはセットを反復処理し、各文字列をテストする必要がありますが、それはO（n）の動作となります：

は

for (String key : keywords) { 
    if (text.contains(key)) { 
     //do your stuff 
     break; 
    } 
}