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）{
1700442162
1700442163
if（obj instanceof Person）{
1700442164
1700442165
Person p=（Person）obj；
1700442166
1700442167
if（p.getName（）==null||name==null）{
1700442168
1700442169
return false；
1700442170
1700442171
}else{
1700442172
1700442173
return name.equalsIgnoreCase（p.getName（））；
1700442174
1700442175
}
1700442176
1700442177
}
1700442178
1700442179
return false；
1700442180
1700442181
}
1700442182
1700442183
1700442184
1700442185
[
上一页 ]
[ :1.700442136e+09 ]
[
下一页 ]