ネストされたforループ - 最初の "if"ステートメントの読み取りのみ

Question

私はforループをネストしました。それはまあまあです。最初のif文だけをtrueとして読み込みます。それ以降はすべての可能な真の文章を無視します。

for(int i = 0; i < inGroups.length; i++) 
{ 
    for(int g = 0; g < theGroups.length; g++) 
    { 
     if(inGroups[i].equals(theGroups[g])) 
     { 
      gLV.setItemChecked(g, true); 
     } 
    } 
} 

Answer 1

コードは正常です。 @HotLicksについて言及したように、デバッガを使用したり、printステートメントを追加して、何が起こっているかを知る必要があります。あなたのコードに関する一般的なコメントの

カップル：

• あなたのGroupオブジェクトまたは何がそうでなければ、それは常にfalse implemented an equals methodになりましたあなたの配列であることを確認する必要があります。
• setItemCheckedは複数の値を処理できますか？ setを2回呼び出すと、前の値が上書きされますか？

• あなたのコードは非常に非効率です（O（N^2））。あなたはO（N）である次のようなことをすることを検討することができます。配列が小さいのかどうかは関係ありません。この方法を使用する場合は、implement hashCode() and equals() methodsが必要です。

  Set<Group> inGroupsSet = new HashSet<Group>(); 
  // load inGroups into a set 
  for (Group group : inGroups) 
      inGroupsSet.add(group); 
  // look up each Group in theGroups in the set 
  for (int g = 0; g < theGroups.length; g++) 
      if (inGroupsSet.contains(theGroups[g])) 
       gLV.setItemChecked(g, true); 
  

何かがここにお役に立てば幸いです。