我正在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集合中的一个主要弱点.
假设这真的不是“一个自然而明显的比较”,它在设计方面肯定闻起来……而且应该非常仔细地记录.