您的位置:首页 > 其它

那些年犯过的错:不要在遍历的时候删除

2017-12-11 10:04 225 查看
本人学习使用java和相关自动化工具的过程中,遇到很多问题,大多数问题解决之后发现原来根本就是识文学字常见错误。但是最近公司产品出现了一个bug,经过复盘讨论之后,原因竟然是添加修改banner的时候,在遍历过程中直接把元素删了,直接导致了闪退。不知道这个问题在开发眼睛里算什么问题,也不知道这是何种级别的错误。这里就不吐槽了,说一下复现的方法吧。

重要提醒:不要再遍历的时候删除元素。下面是复现代码:
Map<Integer, Integer> abc= new HashMap<>();
abc.put(1, 2);
abc.put(2, 3);
abc.put(3, 4);
Set<Integer> keys = abc.keySet();
for(int i : keys) {
if (i==2) {
abc.remove(i);
}
}

下面是异常:
Exception in thread "main" java.util.ConcurrentModificationException
at java.util.HashMap$HashIterator.nextNode(HashMap.java:1429)
at java.util.HashMap$KeyIterator.next(HashMap.java:1453)
at source.Practise.testDemo(Practise.java:23)
at source.Practise.main(Practise.java:12)


下面放一下正确的方式:
Map<Integer, Integer> abc = new HashMap<>();
abc.put(1, 2);
abc.put(2, 3);
abc.put(3, 4);
Set<Integer> keys = abc.keySet();
List<Integer> sss = new ArrayList<>();
for (int i : keys) {
if (i == 2) {
sss.add(i);
}
}
for (int i : sss) {
abc.remove(i);
}

先把要操作的元素存起来到另外一个地方,然后便利结束之后,在去进行删除等操作。
末了,宣传一下QQ群:群号:340964272。

内容来自用户分享和网络整理,不保证内容的准确性,如有侵权内容,可联系管理员处理 点击这里给我发消息