我应该关注这个compareTo/equals/hashCode实现吗？

Question

我正在QA中编写一堆代码并找到了几个实例,其中开发人员有一个实现Comparable的DTO.这个DTO里面有7个或8个字段.compareTo方法仅在一个字段上实现:

private DateMidnight field1;  //from Joda date/time library

public int compareTo(SomeObject o) {
   if (o == null) {
      return -1;
   }
   return field1.compareTo(o.getField1());
}

类似地,equals方法被覆盖,基本归结为:

return field1.equals(o.getField1());

最后哈希码方法的实现是:

return field1.hashCode;

field1永远不应该为null,并且在这些对象中是唯一的(即我们不应该得到两个具有相同对象的对象field1).

因此,实现是一致的,这是好的,但我应该担心只使用一个字段？这不寻常吗？是否可能导致问题或混淆其他开发人员？我正在考虑这样的场景,其中传递了这些对象的列表,而另一个开发人员使用Map或Set of somesort并从这些对象获得异常行为.任何想法都赞赏.谢谢!

Answer 1

我怀疑这是"第一次使用胜利"的情况 - 有人需要对这些对象的集合进行排序或将它们放在哈希映射中,他们只关心日期.最简单的实现方式是以你所说的方式覆盖equals/ hashCode实现Comparable<T>.

对于专家排序,更好的方法是Comparator<T>在不同的类中实现......但遗憾的是,Java没有任何等效的等级测试类.老实说,我认为它是Java集合中的一个主要弱点.

假设这真的不是 "一个自然而明显的比较",它在设计方面肯定闻起来......而且应该非常仔细地记录.