比較メソッドがJava 7の一般契約に違反しています

Question

Java 7でJavaコードをコンパイルして実行した後、「比較メソッドがその一般契約に違反しています。

私はComparison method violates its general contract! Java 7 onlyを読んでおり、以前のバージョンのJavaでは無視されていたコードに問題があることを認識しています。しかし、私は自分のコードに何が間違っているかは分からない。 Collections.sort（）はエラーを生成します。

私のコードは次のとおりです。

public Comparator sortBySmoothDays() { 
    Comparator c = new Comparator() { 
     public int compare(Object arg0, Object arg1) { 
      Date date0 = ((PosObject)arg0).getDate(); 
      Date date1 = ((PosObject)arg1).getDate(); 

      double d1 = MyUtils.calcSmoothDays(date0, new Date()); 
      double d2 = MyUtils.calcSmoothDays(date1, new Date()); 
      if (d1 >= d2) { 
       return 1; 
      } 
      else { 
       return -1; 
      } 
     } 
    }; 
    return c; 
} 


Comparator c = ComparatorUtils.getInstance().sortBySmoothDays(); 
Collections.sort(posList, c); 

誰でも助けることができますか？ありがとうございました！

Answer 1

コンパレータが0を返さなければならないことを意味します。現在の実装では、等しい場合は1を返します。

double d1 = MyUtils.calcSmoothDays(date0, new Date()); 
double d2 = MyUtils.calcSmoothDays(date1, new Date()); 

return Double.compare(d1, d2); 

Answer 2

コンパレータでは、すべてのオブジェクトがそれ自身よりも大きく比較されます。は常に1を返します。

これは以下requirementに違反：

実装は、すべてのxおよびyについて（（Y、X）を比較されたい）== -sgn（（x、y）を比較されたい）そのSGNを保証しなければなりません。

上記の要件は、がゼロを返す必要があることを意味します。

私はcontractを読んで、実装がそれを実現することをお勧めします。

特に、date0.equals(date1)の場合、コンパレータは浮動小数点の変換や比較を行うことなく、直ちにゼロを返すはずです。

Answer 3

2つのオブジェクトが等しいと比較すると、つまり、calcSmoothDaysが同じ値を返した場合、compare（object1、object2）== 1、compare（object2、object1）== 1という状況はありませんでしたか？

だから、値が等しい場合は物体1>オブジェクト2とオブジェクト2>オブジェクト1 ...