不要在遍历的时候删除

最新推荐文章于 2023-08-12 18:05:32 发布

chenglang2310

最新推荐文章于 2023-08-12 18:05:32 发布

阅读量181

点赞数

文章标签： java 操作系统测试

原文链接：https://my.oschina.net/u/3973795/blog/3099158

版权

本人学习使用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);
		}

先把要操作的元素存起来到另外一个地方，然后便利结束之后，在去进行删除等操作。