反例
public Collection<String> sort(Collection<String> foos){List<String> sortedFoos = new ArrayList<>(foos);Collection.sort(sortedFoos);return sortedFoos;
}
正例
public Collection<String> sort(Collection<String> foos){Collection.sort(new ArrayList<>(foos));return foos;
}
解读
java.utils.Collections中的一些方法,如sort和shuffle。会修改集合本身,如果在一个新创建的集合上调用了这些方法,又没有在之后使用这个新创建的集合,那个这个方法的调用显然是冗余的,或者很有可能是忘记返回新创建的集合的引用;