如果一个方法的返回值类型是集合或者数组 ,如果在方法内部需要返回的集合或者数组是零长度的,也就是没有实际对象在里面,我们也应该放回一个零长度的数组或者集合,而不是返回null。如果返回了null,客户端程序员就要检测返回的是不是null,然后才能进行下一步操作,否则就会引发NullPointException。但是如果是返回的的是空数组或者集合,就不会再后续的使用这个对象上,引发空指针异常,我们可以根据代码的行为和表现,来判断数组和集合是不是为空。
在Collections中有专门针对List,Set,Map的空的实现。如:
Collections.emptyList()
Collections.emptySet();
Collections.emptyMap();
在返回这些集合的方法中,尽量不要返回空值,避免NullPointerException。而就尽量返回非空值,这时候上面所示的三个空(不是null)集合就有用武之地了,可以使用它们。
如在某方法中这样做:
publci List<String> getInterests(){
if(interests == null){
return Collections.emptyList();
}
return interests;
}
返回字符串的方法是不是也应该返回一个空串呢?
之前在项目开发的时候,一直喜欢在一些public方法中来判断一些条件,如果不满足就return null
比如
public String getSubString(String data, int start, int end){
if(TextUtil.isEmpty(data)){
return null;
}
return data.subString(start, end);
}
或者
private final List data =…;
.
.
.
public String getDataItem(int i){
if(data == null){
return null;
}
return data.get(i);
}
一直对于这种写法深信不宜,认为即安全又高效。
但是今天看了中的第43条
发现这样写是非常愚蠢的。因为对于这种写法的API的调用者,无时不刻需要对你返回的东西进行非常判断。
而且我一直认为return null开销比较小,不用对内存进行重新分配(这也是看了深入理解java虚拟机之后没有完全理解造成的弊端,总想要尽量避免GC)。
其实这些担心都是多余的,这种方法对内存的影响几乎是没有的。
所以正确的写法应该是
public String sgetDataItem(int i){
if(data == null){
return “”;
}
return data.get(i);
}
返回一个空String