即使在检查null之后，比较方法也违反了它的一般约定

Question

我得到以下错误

无法恢复activity { com . XXX . yyy . zzz . home activity }:Java . lang . illegalargumentexception:比较法违反了其通用契约！

即使出现此错误，我仍在处理字符串的空情况。任何关于可能出错的提示。下面的代码

public class ConversationComparer implements Comparator<Conversation> {
@Override
public int compare(Conversation x, Conversation y) {

    if (x.getLastMessageDate() == null) {
        return 1;
    }

    if (y.getLastMessageDate() == null) {
        return -1;
    }

    return y.getLastMessageDate().compareTo(x.getLastMessageDate());

}}


public java.util.Date getLastMessageDate() {
        return lastMessageDate;
    }

这就是我使用比较器的方法

if (conversationListAdapter != null) {
            Collections.sort(this.list,new ConversationComparer());
            conversationListAdapter.notifyDataSetChanged();
        }

Answer 1

我唯一能看到的错误是你没有正确处理x.getLastMessageDate（）和y.getLastMessageDate（）都是空的情况。

我认为如果你在方法的开头加上这一行，它就符合合同。

if (x.getLastMessageDate() == null && y.getLastMessageDate() == null)
    return 0; 

Answer 2

您还应该再次检查 x 和 y 为 null，如果两者都为 null，则返回 0。

@Override
public int compare(Conversation x, Conversation y) {

   if (x == y || (x != null && y != null && x.getLastMessageDate() == y.getLastMessageDate()))
       return 0;    

   if (x == null)
       return 1;

   if (y == null)
       return -1;

   if (x.getLastMessageDate() == null)
       return 1;

   if (y.getLastMessageDate() == null)
       return -1;

   return y.getLastMessageDate().compareTo(x.getLastMessageDate());
}

此外，了解“getLastMessageDate()”返回什么数据类型也很有趣。