java – 我应该关注这个compareTo / equals / hashCode实现吗?

我正在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并从这些对象获得异常行为.任何想法都赞赏.谢谢!

解决方法:

我怀疑这是“第一次使用胜利”的情况 – 有人需要对这些对象的集合进行排序或将它们放在哈希映射中,他们只关心日期.最简单的实现方法是重写equals / hashCode并实现Comparable< T>就像你说的那样.

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

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

上一篇:c – 在单独文件中的功能实现


下一篇:java中的图形实现