关于List.addAll(Collection

今天关于一个漏洞查了很久,原来是因为List.addAll(Collection collection)传入参数为null产生的。下面就是我解决问题的过程

bug现象是activity有时会出现闪退,于是我查看了logcat,截图如下:

关于List.addAll(Collection_第1张图片

黑色箭头标注处是关键,一般NullPointerException都是由于空指针(null)或者没有初始化产生的,那我再看看下面ArrayList.addAll(),应该是说这个调用这个函数导致了NullPointerException异常的产生,难道是因为ArrayList.addAll()不能传入空指针,函数没有对空指针的情况进行处理,于是我查看了ArrayList.addAll()源码,如下:

关于List.addAll(Collection_第2张图片


由上可知,在ArrayList.addAll()中对传进来的参数没有做null判断,于是,在调用collection.toArray()函数的时候就抛异常了,activity就崩溃了

在使用ArrayList.addAll()的时候一定要注意传入的参数会不会出现为null的情况,如果有,那么我们可以做以下判断

if (collection!= null)
mInfoList.addAll(Collection collection);

如果为null,就不执行下面的了,作为程序员,我们也不能确保是不是存在null的情况,所以为了确保不会出错,在前面加个判断是一个有经验的程序员该做的。


你可能感兴趣的:(Android)