Effective Java (对象通用方法)八、覆盖equals时请遵守通用约定:?? ?? 对于Object类中提供的equals方法在必
Effective Java (对象通用方法)
八、覆盖equals时请遵守通用约定:
?? ?? 对于Object类中提供的equals方法在必要的时候是必要重载的,然而如果违背了一些通用的重载准则,将会给程序带来一些潜在的运行时错误。如果自定义的class没有重载该方法,那么该类实例之间的相等性的比较将是基于两个对象是否指向同一地址来判定的。因此对于以下几种情况可以考虑不重载该方法:
? ? ? 1.?? ?类的每一个实例本质上都是唯一的。
? ? ? 不同于值对象,需要根据其内容作出一定的判定,然而该类型的类,其实例的自身便具备了一定的唯一性,如Thread、Timer等,他本身并不具备更多逻辑比较的必要性。
?? ?? 2.?? ?不关心类是否提供了“逻辑相等”的测试功能。
? ? ? 如Random类,开发者在使用过程中并不关心两个Random对象是否可以生成同样随机数的值,对于一些工具类亦是如此,如NumberFormat和DateFormat等。
?? ?? 3.?? ?超类已经覆盖了equals,从超类继承过来的行为对于子类也是合适的。
?? ?? 如Set实现都从AbstractSet中继承了equals实现,因此其子类将不在需要重新定义该方法,当然这也是充分利用了继承的一个优势。
?? ?? 4.?? ?类是私有的或是包级别私有的,可以确定它的equals方法永远不会被调用。
?? ?
?? ?? 那么什么时候应该覆盖Object.equals呢?如果类具有自己特有的“逻辑相等”概念,而且超类中没有覆盖equals以实现期望的行为,这是我们就需要覆盖equals方法,如各种值对象,或者像Integer和Date这种表示某个值的对象。在重载之后,当对象插入Map和Set等容器中时,可以得到预期的行为。枚举也可以被视为值对象,然而却是这种情形的一个例外,对于枚举是没有必要重载equals方法,直接比较对象地址即可,而且效率也更高。
? ? ? 在覆盖equals是,该条目给出了通用的重载原则:
?? ?? 1.?? ?自反性:对于非null的引用值x,x.equals(x)返回true。
? ? ? 如果违反了该原则,当x对象实例被存入集合之后,下次希望从该集合中取出该对象时,集合的contains方法将直接无法找到之前存入的对象实例。
? ? ? 2.?? ?对称性:对于任何非null的引用值x和y,如果y.equals(x)为true,那么x.equals(y)也为true。
1???? public final class CaseInsensitiveString {
2???????? private final String s;
3???????? public CaseInsensitiveString(String s) {
4???????????? this.s = s;
5???????? }
6???????? @Override public boolean equals(Object o) {
7???????????? if (o instanceof CaseInsensitiveString)
8???????????????? return s.equalsIgnoreCase((CaseInsensitiveString)o).s);
9???????????? if (o instanceof String) //One-way interoperability
10???????????????? return s.equalsIgnoreCase((String)o);
11???????????? return false;
12???????? }
13???? }
14???? public static void main(String[] args) {
15???????? CaseInsensitiveString cis = new CaseInsensitiveString("Polish");
16???????? String s = "polish";
17???????? List<CaseInsensitiveString> l = new ArrayList<CaseInsensitiveString>();
18???????? l.add(cis);
19???????? if (l.contains(s))
20???????????? System.out.println("s can be found in the List");
21???? }?
?? ?? 对于上例,如果执行cis.equals(s)将会返回true,因为在该class的equals方法中对参数o的类型针对String作了特殊的判断和特殊的处理,因此如果equals中传入的参数类型为String时,可以进一步完成大小写不敏感的比较。然而在String的equals中,并没有针对CaseInsensitiveString类型做任何处理,因此s.equals(cis)将一定返回false。针对该示例代码,由于无法确定List.contains的实现是基于cis.equals(s)还是基于s.equals(cis),对于实现逻辑两者都是可以接受的,既然如此,外部的使用者在调用该方法时也应该同样保证并不依赖于底层的具体实现逻辑。由此可见,equals方法的对称性是非常必要的。以上的equals实现可以做如下修改:
1???? public class Point {
2 private final int x;
3???????? private final int y;
4???????? public Point(int x,int y) {
5???????????? this.x = x;
6???????????? this.y = y;
7???????? }
8???????? @Override public boolean equals(Object o) {
9???????????? if (!(o instanceof Point))
10???????????????? return false;
11???????????? Point p = (Point)o;
12???????????? return p.x == x && p.y == y;
13???????? }
14???? }?
? ? ? 对于该类的equals重载是没有任何问题了,该逻辑可以保证传递性,然而在我们试图给Point类添加新的子类时,会是什么样呢?
1???? public class ColorPoint extends Point {
2???????? private final Color c;
3???????? public ColorPoint(int x,int y,Color c) {
4???????????? super(x,y);
5???????????? this.c = c;
6???????? }
7???????? @Override public boolean equals(Object o) {
8???????????? if (!(o instanceof ColorPoint))
9???????????????? return false;
10???????????? return super.equals(o) && ((ColorPoint)o).c == c;
11???????? }
12???? }?
? ? ? 如果在ColorPoint中没有重载自己的equals方法而是直接继承自超类,这样的相等性比较逻辑将会给使用者带来极大的迷惑,毕竟Color域字段对于ColorPoint而言确实是非常有意义的比较性字段,因此该类重载了自己的equals方法。然而这样的重载方式确实带来了一些潜在的问题,见如下代码:
1???? public void test() {
2???????? Point p = new Point(1,2);
3???????? ColorPoint cp = new ColorPoint(1,2,Color.RED);
4???????? if (p.equals(cp))
5???????????? System.out.println("p.equals(cp) is true");
6???????? if (!cp.equals(p))
7 System.out.println("cp.equals(p) is false");
8???? }?
? ? ? 从输出结果来看,ColorPoint.equals方法破坏了相等性规则中的对称性,因此需要做如下修改:
1???? @Override public boolean equals(Object o) {
2???????? if (!(o instanceof Point))
3???????????? return false;
4???????? if (!(o instanceof ColorPoint))
5???????????? return o.equals(this);
6???????? return super.equals(o) && ((ColorPoint)o).c == c;
7???? }?
? ? ? 经过这样的修改,对称性确实得到了保证,但是却牺牲了传递性,见如下代码:
1???? public void test() {
2???????? ColorPoint p1 = new ColorPoint(1,2,Color.RED);
3???????? Point p2 = new Point(1,2);
4???????? ColorPoint p1 = new ColorPoint(1,2,Color.BLUE);
5???????? if (p1.equals(p2) && p2.equals(p3))
6???????????? System.out.println("p1.equals(p2) && p2.equals(p3) is true");
7???????? if (!(p1.equals(p3))
8???????????? System.out.println("p1.equals(p3) is false");
9???? }?
?? ?? 再次看输出结果,传递性确实被打破了。如果我们在Point.equals中不使用instanceof而是直接使用getClass呢?
1???? class MyTest {
2???????? private static final Set<Point> unitCircle;
3???????? static {
4???????????? unitCircle = new HashSet<Point>();
5???????????? unitCircle.add(new Point(1,0));
6???????????? unitCircle.add(new Point(0,1));
7???????????? unitCircle.add(new Point(-1,0));
8???????????? unitCircle.add(new Point(0,-1));
9???????? }
10???????? public static boolean onUnitCircle(Point p) {
11???????????? return unitCircle.contains(p);
12???????? }
13???? }?
? ? ? 如果此时我们测试的不是Point类本身,而是ColorPoint,那么按照目前Point.equals(getClass方式)的实现逻辑,ColorPoint对象在被传入onUnitCircle方法后,将永远不会返回true,这样的行为违反了"里氏替换原则"(敏捷软件开发一书中给出了很多的解释),既一个类型的任何重要属性也将适用于它的子类型。因此该类型编写的任何方法,在它的子类型上也应该同样运行的很好。
?? ?? 如何解决这个问题,该条目给出了一个折中的方案,既复合优先于继承,见如下代码:
1???? public class ColorPoint {
2???????? //包含了Point的代理类
3???????? private final Point p;
4???????? private final Color c;
5???????? public ColorPoint(int x,int y,Color c) {
6???????????? if (c == null)
7???????????????? throw new NullPointerException();
8???????????? p = new Point(x,y);
9???????????? this.c = c;
10???????? }
11???????? //提供一个视图方法返回内部的Point对象实例。这里Point实例为final对象非常重要,
12 //可以避免使用者的误改动。视图方法在Java的集合框架中有着大量的应用。
13???????? public Point asPoint() {
14???????????? return p;
15???????? }
16???????? @Override public boolean equals(Object o) {
17???????????? if (!(o instanceof ColorPoint))
18???????????????? return false;
19???????????? ColorPoint cp = (ColorPoint)o;
20???????????? return cp.p.equals(p) && cp.c.equals(c);
21???????? }
22???? }?
????? 4.?? ?一致性:对于任何非null的引用值x和y,只要equals的比较操作在对象中所用的信息没有被改变,多次调用x.equals(y)就会一致的返回true,或者一致返回false。
? ? ? 在实际的编码中,尽量不要让类的equals方法依赖一些不确定性较强的域字段,如path。由于path有多种表示方式可以指向相同的目录,特别是当path中包含主机名称或ip地址等信息时,更增加了它的不确定性。再有就是path还存在一定的平台依赖性。
?? ?? 5.?? ?非空性:很难想象会存在o.equals(null)返回true的正常逻辑。作为JDK框架中极为重要的方法之一,equals方法被JDK中的基础类广泛的使用,因此作为一种通用的约定,像equals、toString、hashCode和compareTo等重要的通用方法,开发者在重载时不应该让自己的实现抛出异常,否则会引起很多潜在的Bug。如在Map集合中查找指定的键,由于查找过程中的键相等性的比较就是利用键对象的equals方法,如果此时重载后的equals方法抛出NullPointerException异常,而Map的get方法并未捕获该异常,从而导致系统的运行时崩溃错误,然而事实上,这样的问题是完全可以通过正常的校验手段来避免的。综上所述,很多对象在重载equals方法时都会首先对输入的参数进行是否为null的判断,见如下代码:
1???? @Override public boolean equals(Object o) {
2???????? if (o == null)
3???????????? return false;
4???????? if (!(o instanceof MyType))
5???????????? return false;
6???????? ...
7???? }?
????? 注意以上代码中的instanceof判断,由于在后面的实现中需要将参数o进行类型强转,如果类型不匹配则会抛出ClassCastException,导致equals方法提前退出。在此需要指出的是instanceof还有一个潜在的规则,如果其左值为null,instanceof操作符将始终返回false,因此上面的代码可以优化为:
1???? @Override public boolean equals(Object o) {
2???????? if (o == this)
3???????????? return true;
4????????
5???????? if (!(o instanceof MyType))
6???????????? return false;
7????????????
8???????? MyType myType = (MyType)o;
9???????? return objField.equals(o.objField) && intField == o.intField
10???????????? && Double.compare(doubleField,o.doubleField) == 0
11???????????? && Arrays.equals(arrayField,o.arrayField);
12???? }?
? ? ? 从上面的示例中可以看出,如果域字段为Object对象,则使用equals方法进行两者之间的相等性比较,如果为int等整型基本类型,可以直接比较,如果为浮点型基本类型,考虑到精度和Double.NaN和Float.NaN等问题,推荐使用其对应包装类的compare方法,如果是数组,可以使用JDK 1.5中新增的Arrays.equals方法。众所周知,&&操作符是有短路原则的,因此应该将最有可能不相同和比较开销更低的域比较放在最前面。
? ? ? 最后需要提起注意的是Object.equals的参数类型为Object,如果要重载该方法,必须保持参数列表的一致性,如果我们将子类的equals方法写成:public boolean equals(MyType o);Java的编译器将会视其为Object.equals的过载(Overload)方法,因此推荐在声明该重载方法时,在方法名的前面加上@Override注释标签,一旦当前声明的方法因为各种原因并没有重载超类中的方法,该标签的存在将会导致编译错误,从而提醒开发者此方法的声明存在语法问题。
?? ?
九、覆盖equals时总要覆盖hashCode:
?? ?? 一个通用的约定,如果类覆盖了equals方法,那么hashCode方法也需要被覆盖。如果将会导致该类无法和基于散列的集合一起正常的工作,如HashMap、HashSet。来自JavaSE6的约定如下:
?? ?? 1.?? ?在应用程序执行期间,只要对象的equals方法的比较操作所用到的信息没有被修改,那么对这同一个对象多次调用,hashCode方法都必须始终如一地返回同一个整数。在同一个应用程序的多次执行过程中,每次执行所返回的整数可以不一致。
? ? ? 2.?? ?如果两个对象根据equals(Object)方法比较是相等的,那么调用这两个对象中任意一个对象的hashCode方法都必须产生同样的整数结果。
? ? ? 3.?? ?如果两个对象根据equals(Object)方法比较是不相等的,那么调用这两个对象中任意一个对象的hashCode方法,则不一定要产生不同的整数结果。但是程序员应该知道,给不相等的对象产生截然不同的整数结果,有可能提高散列表的性能。
? ? ? 如果类没有覆盖hashCode方法,那么Object中缺省的hashCode实现是基于对象地址的,就像equals在Object中的缺省实现一样。如果我们覆盖了equals方法,那么对象之间的相等性比较将会产生新的逻辑,而此逻辑也应该同样适用于hashCode中散列码的计算,既参与equals比较的域字段也同样要参与hashCode散列码的计算。见下面的示例代码:
1???? public final class PhoneNumber {
2???????? private final short areaCode;
3???????? private final short prefix;
4???????? private final short lineNumber;
5???????? public PhoneNumber(int areaCode,int prefix,int lineNumber) {
6???????????? //做一些基于参数范围的检验。
7???????????? this.areaCode = areaCode;
8???????????? this.prefix = prefix;
9???????????? this.lineNumber = lineNumber;
10???????? }
11???????? @Override public boolean equals(Object o) {
12???????????? if (o == this)
13???????????????? return true;
14???????????? if (!(o instanceof PhoneNumber))
15???????????????? return false;
16???????????? PhoneNumber pn = (PhoneNumber)o;
17???????????? return pn.lineNumber = lineNumber && pn.prefix == prefix && pn.areaCode = areaCode;
18???????? }
19???? }
20???? public static void main(String[] args) {
21???????? Map<PhoneNumber,String> m = new HashMap<PhoneNumber,String>();
22???????? PhoneNumber pn1 = new PhoneNumber(707,867,5309);
23???????? m.put(pn1,"Jenny");
24???????? PhoneNumber pn2 = new PhoneNumber(707,867,5309);
25???????? if (m.get(pn) == null)
26???????????? System.out.println("Object can't be found in the Map");
27???? }?
? ? ? 从以上示例的输出结果可以看出,新new出来的pn2对象并没有在Map中找到,尽管pn2和pn1的相等性比较将返回true。这样的结果很显然是有悖我们的初衷的。如果想从Map中基于pn2找到pn1,那么我们就需要在PhoneNumber类中覆盖缺省的hashCode方法,见如下代码:
1???? @Override public int hashCode() {
2???????? int result = 17;
3???????? result = 31 * result + areaCode;
4???????? result = 31 * result + prefix;
5???????? result = 31 * result + lineNumber;
6???????? return result;
7???? }?
? ? ? 在上面的代码中,可以看到参与hashCode计算的域字段也同样参与了PhoneNumber的相等性(equals)比较。对于生成的散列码,推荐不同的对象能够尽可能生成不同的散列,这样可以保证在存入HashMap或HashSet中时,这些对象被分散到不同的散列桶中,从而提高容器的存取效率。对于有些不可变对象,如果需要被频繁的存取于哈希集合,为了提高效率,可以在对象构造的时候就已经计算出其hashCode值,hashCode()方法直接返回该值即可,如:
1???? public final class PhoneNumber {
2???????? private final short areaCode;
3???????? private final short prefix;
4???????? private final short lineNumber;
5???????? private final int myHashCode;
6???????? public PhoneNumber(int areaCode,int prefix,int lineNumber) {
7???????????? //做一些基于参数范围的检验。
8???????????? this.areaCode = areaCode;
9???????????? this.prefix = prefix;
10???????????? this.lineNumber = lineNumber;
11???????????? myHashCode = 17;
12???????????? myHashCode = 31 * myHashCode + areaCode;
13???????????? myHashCode = 31 * myHashCode + prefix;
14???????????? myHashCode = 31 * myHashCode + lineNumber;
15???????? }
16???????? @Override public boolean equals(Object o) {
17???????????? if (o == this)
18???????????????? return true;
19???????????? if (!(o instanceof PhoneNumber))
20???????????????? return false;
21???????????? PhoneNumber pn = (PhoneNumber)o;
22???????????? return pn.lineNumber = lineNumber && pn.prefix == prefix && pn.areaCode = areaCode;
23???????? }
24???????? @Override public int hashCode() {
25???????????? return myHashCode;
26???????? }
27???? }?
? ? ? 另外,该条目还建议不要仅仅利用某一域字段的部分信息来计算hashCode,如早期版本的String,为了提高计算哈希值的效率,只是挑选其中16个字符参与hashCode的计算,这样将会导致大量的String对象具有重复的hashCode,从而极大的降低了哈希集合的存取效率。
?? ?
十、始终要覆盖toString:
? ? ? 与equals和hashCode不同的是,该条目推荐应该始终覆盖该方法,以便在输出时可以得到更明确、更有意义的文字信息和表达格式。这样在我们输出调试信息和日志信息时,能够更快速的定位出现的异常或错误。如上一个条目中PhoneNumber的例子,如果不覆盖该方法,就会输出PhoneNumber@163b91 这样的不可读信息,因此也不会给我们诊断问题带来更多的帮助。以下代码重载了该方法,那么在我们调用toString或者println时,将会得到"(408)867-5309"。
1???? public static void main(String[] args) {
2???????? HashSet<BigDecimal> hs = new HashSet<BigDecimal>();
3???????? BigDecimal bd1 = new BigDecimal("1.0");
4???????? BigDecimal bd2 = new BigDecimal("1.00");
5???????? hs.add(bd1);
6???????? hs.add(bd2);
7???????? System.out.println("The count of the HashSet is " + hs.size());
8????????
9???????? TreeSet<BigDecimal> ts = new TreeSet<BigDecimal>();
10???????? ts.add(bd1);
11???????? ts.add(bd2);
12???????? System.out.println("The count of the TreeSet is " + ts.size());
13???? }
14???? /*??? 输出结果如下:
15 ??????? The count of the HashSet is 2
16 ??????? The count of the TreeSet is 1
17 */????
?? ?? 由以上代码的输出结果可以看出,TreeSet和HashSet中包含元素的数量是不同的,这其中的主要原因是TreeSet是基于BigDecimal的compareTo方法是否返回0来判断对象的相等性,而在该例中compareTo方法将这两个对象视为相同的对象,因此第二个对象并未实际添加到TreeSet中。和TreeSet不同的是HashSet是通过equals方法来判断对象的相同性,而恰恰巧合的是BigDecimal的equals方法并不将这个两个对象视为相同的对象,这也是为什么第二个对象可以正常添加到HashSet的原因。这样的差异确实给我们的编程带来了一定的负面影响,由于HashSet和TreeSet均实现了Set<E>接口,倘若我们的集合是以Set<E>的参数形式传递到当前添加BigDecimal的函数中,函数的实现者并不清楚参数Set的具体实现类,在这种情况下不同的实现类将会导致不同的结果发生,这种现象极大的破坏了面向对象中的"里氏替换原则"。
? ? ? 在重载compareTo方法时,应该将最重要的域字段比较方法比较的最前端,如果重要性相同,则将比较效率更高的域字段放在前面,以提高效率,如以下代码:
1???? public int compareTo(PhoneNumer pn) {
2???????? if (areaCode < pn.areaCode)
3???????????? return -1;
4???????? if (areaCode > pn.areaCode)
5???????????? return 1;
6????????????
7???????? if (prefix < pn.prefix)
8???????????? return -1;
9???????? if (prefix > pn.prefix)
10???????????? return 1;
11????????????
12???????? if (lineNumber < pn.lineNumer)
13???????????? return -1;
14???????? if (lineNumber > pn.lineNumber)
15???????????? return 1;
16???????? return 0;
17???? }?
? ? ? 上例给出了一个标准的compareTo方法实现方式,由于使用compareTo方法排序的对象并不关心返回的具体值,只是判断其值是否大于0,小于0或是等于0,因此以上方法可做进一步优化,然而需要注意的是,下面的优化方式会导致数值类型的作用域溢出问题。
1???? public int compareTo(PhoneNumer pn) {
2???????? int areaCodeDiff = areaCode - pn.areaCode;
3???????? if (areaCodeDiff != 0)
4???????????? return areaCodeDiff;
5???????? int prefixDiff = prefix - pn.prefix;
6???????? if (prefixDiff != 0)
7???????????? return prefixDiff;
8????
9???????? int lineNumberDiff = lineNumber - pn.lineNumber;
10???????? if (lineNumberDiff != 0)
11???????????? return lineNumberDiff;
12???????? return 0;
13???? }?