1700442112
1700442113
System.out.println（“列表中是否包含张三：”+l.contains（p2））；
1700442114
1700442115
}
1700442116
1700442117
上面的代码产生了两个Person对象（注意p2变量中的那个张三后面有一个空格），然后放到List中，最后判断List是否包含了这两个对象。看上去没有问题，应该打印出两个true才是，但是结果却是：
1700442118
1700442119
列表中是否包含张三：true
1700442120
1700442121
列表中是否包含张三：false
1700442122
1700442123
刚刚放到list中的对象竟然说没有，这太让人失望了，原因何在呢？List类检查是否包含元素时是通过调用对象的equals方法来判断的，也就是说constains（p2）传递进去，会依次执行p2.equals（p1）、p2.equals（p2），只要有一个返回true，结果就是true，可惜的是比较结果都是false，那问题就出来了：难道p2.equals（p2）也为false不成？
1700442124
1700442125
还真说对了，p2.equals（p2）确实是false，看看我们的equals方法，它把第二个参数进行了剪切！也就是说比较的是如下等式：
1700442126
1700442127
“张三”.equalsIgnoreCase（“张三”）
1700442128
1700442129
注意前面的“张三”是有空格的，那这个结果肯定是false了，错误也就此产生了。这是一个想做好事却办成了“坏事”的典型案例，它违背了equals方法的自反性原则：对于任何非空引用x, x.equals（x）应该返回true。
1700442130
1700442131
问题知道了，解决也非常容易，只要把trim（）去掉即可，注意解决的只是当前问题，该equals方法还存在其他问题。
1700442132
1700442133
1700442134
1700442135
1700442137
编写高质量代码：改善Java程序的151个建议 建议46：equals应该考虑null值情景
1700442138
1700442139
继续上一建议的问题，我们解决了覆写equals的自反性问题，是不是就很完美了呢？再把main方法重构一下：
1700442140
1700442141
public static void main（String[]args）{
1700442142
1700442143
Person p1=new Person（“张三”）；
1700442144
1700442145
Person p2=new Person（null）；
1700442146
1700442147
/*其他部分没有任何修改，不再赘述*/
1700442148
1700442149
}
1700442150
1700442151
很小的改动，那运行结果是什么呢？是两个true吗？我们来看运行结果：
1700442152
1700442153
列表中是否包含张三：true
1700442154
1700442155
Exception in thread”main”java.lang.NullPointerException
1700442156
1700442157
竟然抛异常了！为什么p1就能在List中检查一遍，并且执行p1.equals方法，而到了p2就开始报错了呢？仔细分析一下程序，马上明白了：当执行到p2.equals（p1）时，由于p2的name是一个null值，所以调用name.equalsIgnoreCase方法时就会报空指针异常了！出现这种情形是因为覆写equals没有遵循对称性原则：对于任何引用x和y的情形，如果x.equals（y）返回true，那么y.equals（x）也应该返回true。
1700442158
1700442159
问题知道了，解决也很简单，增加name是否为空进行判断即可，修改后的equals代码如下：
1700442160
1700442161
public boolean equals（Object obj）{
[
上一页 ]
[ :1.700442112e+09 ]
[
下一页 ]