在实践中,是否像这样返回一个空列表更好:
return Collections.emptyList();
或者像这样:
return new ArrayList<Foo>();
还是说这完全取决于你将如何处理返回的列表?
在实践中,是否像这样返回一个空列表更好:
return Collections.emptyList();
或者像这样:
return new ArrayList<Foo>();
还是说这完全取决于你将如何处理返回的列表?
当前回答
如果返回的列表没有以任何方式修改(因为列表是不可变的),我会使用Collections.emptyList(),否则我会使用选项2。
Collections.emptyList()的好处是每次都返回相同的静态实例,因此每次调用都不会创建实例。
其他回答
给出的答案强调了一个事实,即emptyList()返回一个不可变的List,但没有给出替代选项。构造函数ArrayList(int initialCapacity)特殊情况为0,因此返回新的ArrayList<>(0)而不是新的ArrayList<>()可能也是一个可行的解决方案:
/**
* Shared empty array instance used for empty instances.
*/
private static final Object[] EMPTY_ELEMENTDATA = {};
[…]
/**
* Constructs an empty list with the specified initial capacity.
*
* @param initialCapacity the initial capacity of the list
* @throws IllegalArgumentException if the specified initial capacity
* is negative
*/
public ArrayList(int initialCapacity) {
if (initialCapacity > 0) {
this.elementData = new Object[initialCapacity];
} else if (initialCapacity == 0) {
this.elementData = EMPTY_ELEMENTDATA;
} else {
throw new IllegalArgumentException("Illegal Capacity: "+
initialCapacity);
}
}
(来自Java 1.8.0_72)
集合。emptyList是不可变的,所以两个版本之间有区别,所以你必须考虑返回值的用户。
返回new ArrayList<Foo>总是会创建一个对象的新实例,因此它有一个非常小的额外开销,这可能会给你一个使用Collections.emptyList的理由。我喜欢使用emptyList,因为它可读性更好。
不过要小心。如果你返回Collections.emptyList(),然后尝试对它做一些更改,如add()或smth,你将有一个UnsupportedOperationException(),因为Collections.emptyList()返回一个不可变的对象。
主要区别是Collections.emptyList()返回一个不可变列表,即不能向其中添加元素的列表。(同样适用于Java 9中引入的List.of()。)
在极少数情况下,您确实希望修改返回的列表,Collections.emptyList()和list .of()不是一个好的选择。
我想说,返回一个不可变的列表是完全可以的(甚至是首选的方式),只要契约(文档)没有显式地声明不同。
此外,emptyList()可能不会在每次调用时都创建一个新对象。
此方法的实现不需要为每次调用创建单独的List对象。使用此方法的成本可能与使用类似名称的字段相当。(与此方法不同,该字段不提供类型安全。)
emptyList的实现如下所示:
public static final <T> List<T> emptyList() {
return (List<T>) EMPTY_LIST;
}
因此,如果您的方法(返回一个空列表)经常被调用,这种方法甚至可以在CPU和内存方面为您提供更好的性能。
如果返回的列表没有以任何方式修改(因为列表是不可变的),我会使用Collections.emptyList(),否则我会使用选项2。
Collections.emptyList()的好处是每次都返回相同的静态实例,因此每次调用都不会创建实例。