在编程中,返回null而不是空数组或集合可能导致客户端代码需要额外处理,增加错误风险。通常,应返回空的容器或数组,而不是null。即使在性能考虑下,返回空集合或数组也不涉及额外开销,因为可以复用不可变的空实例。对于数组,同样应返回零长度的数组,而不是null,以避免性能损耗和错误。优化时,可以重复返回相同的空数组,但应先验证是否真正需要这种优化,以免适得其反。在现实项目中我们经常会对查询的结果进行非空判断CollectionUtils.isEmpty(productItems)或者用try catch捕获异常。
例如:
// 缓存中没有则从数据库中查找
List<ProductItem> productItems = productItemMapper.selectAll();
if (CollectionUtils.isEmpty(productItems)) {
log.error("查询结果为空");
}
ProductItemComposite productItemComposite = generateProductTree(productItems);
像下面这样的方法并不少见:
private final List<Cheese> cheesesInStock = ...;
public Cheese[] getCheeses(){
if (cheesesInStock.size == 0)
return null;
}
把没有奶酪可买的情况当做一种特例,这是不合常理的。这样做会要求客户端必须有额外的代买来处理null的返回值。例如
Cheese[] cheeses = shop.getCheeses();
if(cheeses != null && Arrays.asList(cheeses).contains(Cheese.STILTON)){
System.out.println("Jolly good,just the thing.") ;
}
而不是下面这段代码:
if( Arrays.asList(cheeses).contains(Cheese.STILTON)){
System.out.println("Jolly good,just the thing.") ;
}
这样做很容易出错,因为编写客户端程序的程序员可能会忘记写这种专门的代码来处理null返回值。这样的错误也许几年都不会被注意到,因为这样的方法通常返回一个或者多个对象。
有时候会有人认为:null返回值比零长度数组更好,因为它避免了分配数组所需要的开销。
这个观点站不住脚,首先这个级别担心性能是不明智的,除非分析表明这个方法就是造成性能的根本原因;其次对于不返回任何元素的调用,每次都返回同一个零长度数组是可能的,因为零长度数组是不可变的,而不可变对象有可能被自由共享。实际上,当使用标准做法把一些元素从一个集合转存到一个类型化的数组中时,它正式这样做的:
private final List<Cheese> cheeseInStock = …;
private static final Cheese[] EMPTY_CHEESE_ARRAY =new Cheese[0];
public Cheese[] getCheeses(){
return cheeseInStock.toArray(EMPTY_CHEESE_ARRAY);
}
习惯用法中,零长度数组常量被传递给toArray方法,以指明所期望的返回类型。正常情况下,toArray方法分配了返回的数组,如果集合是空的,它将使用零长度输入数组。
集合值方法可以做成需要返回空集合时返回同一个不可变的空集合。Collections.emptySet(),
Collections.emptyList(),Collections.emptyMap()方法正是所需要的。
private final List<Cheese> cheeseInStock = …;
public List<Cheese> getCheesesList(){
if (cheeseInStock.isEmpty())
rerurnCollections.emptyList();
else
return new ArrayList(cheeseInStock);
}
总之,永远不要返回null,而是返回空的数组或者集合。返回null会使我们的API更难使用,更容易出错,而且性能上并没有优势。
所有文章无条件开放,顺手点个赞不为过吧!