は、Javaでの単語内の単語を比較

Question

基本的に私はしました：あなたは、私はその後、私は,(space)で各単語を分割していた後、私はif()を使用して、Action, Animation, Comdeyを含む文字列genreを持って見ることができるように

public static void main(String[] args) 
{ 
String genre = "Action, Animation, Comdey"; 
String[] genres = genre.split(", "); 
for(int i=0;i<(genres.length)-1;i++){ 
    System.out.println(genres[i]); 
    if(!genres[i].equals("Action")    || !genres[i].equals("Adventure") 
      || !genres[i].equals("Animation") || !genres[i].equals("Biography") 
      || !genres[i].equals("Comedy")  || !genres[i].equals("Crime") 
      || !genres[i].equals("Documentary") || !genres[i].equals("Drama") 
      || !genres[i].equals("Family")  || !genres[i].equals("Fantasy") 
      || !genres[i].equals("Film-Noir") || !genres[i].equals("Game-Show") 
      || !genres[i].equals("History")  || !genres[i].equals("Horror") 
      || !genres[i].equals("Music")  || !genres[i].equals("Musical") 
      || !genres[i].equals("Mystery")  || !genres[i].equals("News") 
      || !genres[i].equals("Reality-TV") || !genres[i].equals("Sci-Fi") 
      || !genres[i].equals("Sport")  || !genres[i].equals("Talk-Show") 
      || !genres[i].equals("Thriller") || !genres[i].equals("War") 
      || !genres[i].equals("Western") 
      ){ 
     System.out.println("Selected genres could not be identified"); 
     return; 
    }else { 
     System.out.println("Success"); 
    } 
} 
} 

各アレイを比較する。あなたはAction、AnimationとComedy私のif()単語リストに存在する私の配列を知っているかもしれません。だから、System.out.println("Selected genres could not be identified");を実行するとは思わないが、私はなぜそれが本当に私を驚かせているのかわからない。

https://www.compilejava.net/に自分のコードをコピーして貼り付けて、結果を自分で見ることができます。

私が間違っていることを教えていただければ、本当に感謝しています。

Answer 1

あなたは現在のジャンルは例えば、利用可能なジャンルのnoneと一致していることを確認しようとしている場合は、論理積（&&）に、すべての論理和（||）を変更する必要があります。

public static void main(String[] args) { 
    String genre = "Action, Animation, Comdey"; 
    String[] genres = genre.split(", "); 
    for (int i = 0; i < (genres.length) - 1; i++) { 
     System.out.println(genres[i]); 
     if (!genres[i].equals("Action") && !genres[i].equals("Adventure") && !genres[i].equals("Animation") 
       && !genres[i].equals("Biography") && !genres[i].equals("Comedy") && !genres[i].equals("Crime") 
       && !genres[i].equals("Documentary") && !genres[i].equals("Drama") && !genres[i].equals("Family") 
       && !genres[i].equals("Fantasy") && !genres[i].equals("Film-Noir") && !genres[i].equals("Game-Show") 
       && !genres[i].equals("History") && !genres[i].equals("Horror") && !genres[i].equals("Music") 
       && !genres[i].equals("Musical") && !genres[i].equals("Mystery") && !genres[i].equals("News") 
       && !genres[i].equals("Reality-TV") && !genres[i].equals("Sci-Fi") && !genres[i].equals("Sport") 
       && !genres[i].equals("Talk-Show") && !genres[i].equals("Thriller") && !genres[i].equals("War") 
       && !genres[i].equals("Western")) { 
      System.out.println("Selected genres could not be identified"); 
      return; 
     } else { 
      System.out.println("Success"); 
     } 
    } 
} 

Here「SAもう少し論理OR演算子です。あなたのシナリオでは、!genres[i].equals("Action")は「行動」の場合にはfalseと評価されますが、!genres[i].equals("Adventure")のような他の条件はtrueと評価され、「選択されたジャンルは特定できませんでした。

Answer 2

文字列のジャンル=「アクション、アニメーション、Comdeyは」文字列のジャンルに変更する必要があります=「アクション、アニメーション、コメディ」上記の書き込みが

Answer 3

クリーナーで効率的な方法は、コードを言及したTreeSetのを使用することになります。

public static void main(String[] args) 
    { 
    String genre = "Action, Animation, Comdey"; 
    String[] genres = genre.split(", "); 
    Set<String> genreSet = new TreeSet<String>(); 
    genreSet.add("Action"); 
    genreSet.add("Adventure"); 
    genreSet.add("Animation"); 
    genreSet.add("Documentary"); 
    genreSet.add("Comedy"); 
    //more genres follows here 

    for (int i = 0; i < (genres.length) - 1; i++) { 
     System.out.println(genres[i]); 
     if (!genreSet.contains(genres[i])) { 
      System.out.println("Selected genres could not be identified"); 
      return; 
     } else { 
      System.out.println("Success"); 
     } 
    } 
    } 

TreeSetのジャンルストリングを検索する時間の複雑さが（log n）のオーダーになるため、効率的です。