Collections.sortを使用してカスタムクラスの配列リストの文字列をソート

Question

私自身の匿名コンパイラを宣言してCollections.sortを使用してカスタムクラスの配列リストをソートしようとしています。しかし、ソートは期待どおりに動作していません。

私のコードは

Collections.sort(arrlstContacts, new Comparator<Contacts>() { 

     public int compare(Contacts lhs, Contacts rhs) { 

      int result = lhs.Name.compareTo(rhs.Name); 

      if(result > 0) 
      { 
       return 1; 

      } 
      else if (result < 0) 
      { 
       return -1; 
      } 
      else 
      { 
       return 0; 
      } 
     } 
    }); 

結果がソートされた順序ではないです。

Answer 1

は、単に実行します。

Collections.sort(
    arrlstContacts, 
    new Comparator<Contacts>() 
    { 
    public int compare(Contacts lhs, Contacts rhs) 
    { 
     return lhs.Name.compareTo(rhs.Name); 
    } 
    } 
); 

あなたの元のコードが否定された辞書式比較を実行するString.compareTo方法。たとえば、文字列number1とnumber123を比較すると、それぞれ-2と2が生成されます。

マージソートのマージ部分がCollections.sortの方法では、リスト内の文字列間で十分に区別できない可能性があります（おそらく起きているようです）。アルファベット順にソートされていません。

Answer 2

アダムによって示されるように、あなたはreturn (lhs.Name.compareTo(rhs.Name)); likeso使用することができます：アダムが言うのように

Collections.sort(arrlstContacts, new Comparator<Contacts>() { 
    public int compare(Contacts lhs, Contacts rhs) { 
     return (lhs.Name.compareTo(rhs.Name)); 
    } 
});