java – 迭代器空集合

很常见的是,在不确定集合引用为空的情况下,必须先检查null,否则返回null.
样品：

Collection<Object> collection = ...
...
if(collection != null)//troublesome
    for(Object o : collection)

当然,我知道空集合比null好得多,但在某些情况下,客户端代码无法控制其他模块的可空值集合(例如,从第三方代码返回值).
所以我写了一个实用方法：

public static <T> Iterable<T> nullableIterable(Iterable<T> it){
    return it != null ? it : Collections.<T>emptySet();
}

在客户端代码中,无需再检查null：

for(Object o : nullableIterable(collection))
...

你认为nullableIterable()是否合理？任何建议？有什么关系？谢谢！

解决方法

看起来不错.我个人也这样做.你会永远得到开发人员不同意这一点,因为它是一种防御性的程序设计.想象一下,你有一个工作流或一个不应该返回null的类.这意味着从它获取一个null是你的代码将隐藏的一个错误,因为它会将null变成一个空集合,并且该错误永远不会出现.

如果你是编写不支持null集合的API,那么你应该避免这种情况.如果客户端代码为您提供不支持的空集合,那么您应该抛出一个IllegalArgumentException,让客户端代码知道提供的集合有问题.就像是：

public void myApiNoSupportForNull(Collection<Object> collection){
   // Pre condition
   if(collection == null) 
     throw new IllegalArgumentException("This API does not support null collections!");
   //...
}