Java: IllegalArgumentException

Question

Я смотрю какой-то унаследованный код, и я нашел раздел, который вызывает у меня ошибку «Сравнительный метод нарушает общий контракт!». Я понимаю, что эта ошибка является результатом того, что код не является транзитивным, но я не совсем понимаю, как правильно его исправить.

Вот код, который отвечает за ошибки.

private void sortHistories(List<History> histories) { 
     Collections.sort(histories, new Comparator<History>() { 

      @Override 
      public int compare(History o1, History o2) { 

       return o1 == o2 ? 0 
         : o1 == null ? -1 
         : o2 == null ? 1 
         : o1.getFamilyMembers().equals(o2.getFamilyMembers()) ? 0 //getFamilyMembers() returns a string 
         : o1.getFamilyMembers() == null ? -1 
         : o2.getFamilyMembers() == null ? 1 
         : o2.getFamilyMembers().compareTo(o2.getFamilyMembers()) != 0 ? 
           o2.getFamilyMembers().compareTo(o2.getFamilyMembers()) 
         : o1.getDisease().equals(o2.getDisease()) ? 0 //getDisease() also returns a string 
         : o1.getDisease() == null ? -1 
         : o2.getDisease() == null ? 1 
         : o1.getDisease().compareTo(o2.getDisease()); 
      } 
     }); 
    } 

Первоначально код использует ==, а не equals() при сравнении строк getDisease() и getFamilyMembers(). Я думал, что внесение изменений с == в equals() устранит проблему, но это не тот случай.

Answer 1

Внимательно прочитайте документацию. Завершение этого исключения - это новая функция Java7.

Подробнее здесь :: https://stackoverflow.com/a/8327575/3080158

Старое поведение может быть сконфигурирован с новым свойством системы: java.util.Arrays.useLegacyMergeSort.

http://www.oracle.com/technetwork/java/javase/compatibility-417013.html#source

http://docs.oracle.com/javase/7/docs/api/java/util/Comparator.html

Answer 2

Раствор, благодаря HaifengZhang и YoungHobbit, является:

public int compare(History o1, History o2) { 
       return o1 == o2 ? 0 
         : o1 == null ? -1 
         : o2 == null ? 1 
         : o1.getFamilyMembers() == null ? -1 
         : o2.getFamilyMembers() == null ? 1 
         : o1.getFamilyMembers() == o2.getFamilyMembers() ? 0 
         : o2.getFamilyMembers().compareTo(o1.getFamilyMembers()) != 0 ? 
           o2.getFamilyMembers().compareTo(o1.getFamilyMembers()) 
         : o1.getDisease() == null ? -1 
         : o2.getDisease() == null ? 1 
         : o1.getDisease() == o2.getDisease() ? 0 
         : o1.getDisease().compareTo(o2.getDisease()); 
      }