Java中使用索引for循环遍历列表时删除元素是否安全？

好问题！咱们来拆解一下你的代码和这种写法的安全性：

从你说代码可正常运行反推，应该是你的list是在循环过程中逐个添加元素的（虽然代码里没写这一步，但逻辑上是这样）。当i=3时，你先把3加入列表，此时列表的索引0-3是有效的，调用list.remove(i)删除的是刚添加的索引3的元素，这一步不会有问题；之后continue跳过打印，剩下的i从4到9只是添加元素和打印，不会触发删除操作，所以不会碰到索引越界的情况，看起来运行正常。

这种基于普通索引循环删除元素的写法，存在很多潜在风险：

• 索引混乱导致漏删/误删：如果需求变成删除多个元素（比如同时删除i=3和i=5），问题立刻显现。当你删除i=3的元素后，列表中原本索引4及之后的元素都会前移一位，原来的索引5的元素现在在索引4的位置。当循环到i=5时，你删除的是当前列表的索引5（也就是原来的索引6的元素），而原来的索引5的元素（现在的索引4）会被直接跳过，根本没被处理。
• 索引越界风险：如果循环中删除元素的次数变多，当循环到后期的i值时，列表的大小会小于i，此时调用list.remove(i)会直接抛出IndexOutOfBoundsException。比如如果删除3个元素，当i=9时，列表的大小只有7，索引9明显超出范围，程序直接崩溃。
• 逻辑极度脆弱：你的代码能运行完全依赖「只删除一次，且删除时索引刚好是当前列表的有效索引」这个极端特定的前提。如果列表在循环前已经有其他元素，或者你想删除的是某个值而非对应索引的元素，这段代码的逻辑就完全不符合预期了。

还是推荐你用这些经过Java集合框架验证的标准方式：

• Iterator.remove()：这是专门为遍历删除设计的方式，它会自动维护遍历状态，彻底避免索引混乱问题：

  Iterator<Integer> iterator = list.iterator();
  while (iterator.hasNext()) {
      Integer num = iterator.next();
      if (num == 3) {
          iterator.remove();
      }
  }
  

• Java Streams（创建新列表）：这种方式不修改原列表，而是生成一个过滤后的新列表，简洁且安全，尤其适合不需要修改原列表的场景：

  list = list.stream().filter(num -> num != 3).collect(Collectors.toList());
  

• 批量删除（收集待删元素）：先把要删除的元素收集到另一个列表，再批量删除，避免遍历中修改原列表：

  List<Integer> toBeRemoved = new ArrayList<>();
  for (Integer num : list) {
      if (num == 3) {
          toBeRemoved.add(num);
      }
  }
  list.removeAll(toBeRemoved);
  

你的这段代码在当前的特定场景下能跑通，但这是个巧合，不是安全、可维护的写法。一旦需求变化或者列表初始状态改变，很容易出现各种问题。所以强烈建议换成上面提到的标准安全方案。

内容的提问来源于stack exchange，提问作者Bick