最近入职新公司,组长让我改改bug,其中看到一段代码:
/**
* 全部复制生产环境代码,只改动了str的值
* @param args
*/
public static void main(String[] args) {
String str = "1001, 1002, 1006";
List<String> strList = Arrays.asList(str.split(","));
Iterator iterator = strList.iterator();
while (iterator.hasNext()) {
String s = (String) iterator.next();
strList.remove(iterator);
}
}
鄙人一经思量,得出结论,strList.remove() 这一行代码肯定报错,在CSDN的文章以及《阿里巴巴Java开发手册》中已经说明:
此时,信心满满,打开测试环境,调用接口,返回成功。
看着接口返回,陷入了沉思。难道网络上的东西都是虚拟的,我把握不住吗?于是,翻看Arrays.asList() 方法源码,才发现问题所在。
关于 Arrays.asList 详细介绍,网上有很多优秀的讲解,这里不赘述。
Arrays.asList 方法返回的类为 java.util.Arrays.ArrayList,该类的 remove()方法分别来自于两个类:
1、java.util.AbstractList#remove,如果使用下标移除,直接抛异常
public E remove(int index) {
throw new UnsupportedOperationException();
}
2、java.util.AbstractCollection#remove
public boolean remove(Object o) {
Iterator<E> it = iterator();
if (o==null) {
while (it.hasNext()) {
if (it.next()==null) {
it.remove();
return true;
}
}
} else {
while (it.hasNext()) {
if (o.equals(it.next())) {
it.remove();
return true;
}
}
}
return false;
}
根据代码逻辑,此时调用的应该是第二个方法,最后得出结论,代码里remove了寂寞,因为入参非集合 strList 中的元素,所有没有调用抛出异常的 it.remove方法。所以,上述代码并非是对集合的更改。
看来,网络上虚拟的东西鄙人把握住了。