在Java中,当使用foreach循环遍历集合时,对集合调用remove是否合法?例如:

List<String> names = ....
for (String name : names) {
   // Do something
   names.remove(name).
}

作为附录,移除尚未迭代的项目是否合法?例如,

//Assume that the names list as duplicate entries
List<String> names = ....
for (String name : names) {
    // Do something
    while (names.remove(name));
}

当前回答

那些说你不能安全地从集合中删除一个项的说法是不正确的,你可以使用一个并发集合,比如ConcurrentHashMap。

其他回答

为了在迭代集合时安全地从集合中删除,您应该使用迭代器。

例如:

List<String> names = ....
Iterator<String> i = names.iterator();
while (i.hasNext()) {
   String s = i.next(); // must be called before you can call i.remove()
   // Do something
   i.remove();
}

摘自Java文档:

该类的迭代器和listIterator返回的迭代器 方法是快速失败的:如果列表在任何地方被结构修改 方法创建迭代器后的任何时间 迭代器自己的remove或add方法时,迭代器将抛出 并发修改异常因此,面对并发 修改后,迭代器会快速而干净地失败,而不是 在不确定的时间冒着武断的、不确定的行为的风险 在未来。

也许许多新手不清楚的是,使用for/foreach构造隐式遍历列表会创建一个不可访问的迭代器。这些信息可以在这里找到

你不会想这么做的。它可能导致未定义的行为,具体取决于集合。你想直接使用迭代器。虽然for每个构造都是语法糖,并且实际上使用了迭代器,但它对代码隐藏了迭代器,因此您无法访问它来调用iterator .remove。

迭代器的行为是 未指定,如果基础 属性时,将对集合进行修改 迭代正在以任何方式进行 除了调用这个方法。

相反,编写代码:

List<String> names = ....
Iterator<String> it = names.iterator();
while (it.hasNext()) {

    String name = it.next();
    // Do something
    it.remove();
}

注意,代码调用Iterator。remove,而不是List.remove。

附录:

即使您正在删除一个尚未迭代的元素,您仍然不想修改集合,然后使用Iterator。它可能会以一种令人惊讶的方式修改集合,并影响Iterator上的后续操作。

我不知道迭代器,但是直到今天我才从循环中的列表中删除元素:

List<String> names = .... 
for (i=names.size()-1;i>=0;i--) {    
    // Do something    
    names.remove(i);
} 

这总是有效的,并且可以在其他不支持迭代器的语言或结构中使用。

Use

.remove() Interator或

Use

CopyOnWriteArrayList

是的你可以使用for-each循环, 要做到这一点,你必须维护一个单独的列表来保存删除项,然后使用removeAll()方法从名称列表中删除该列表,

List<String> names = ....

// introduce a separate list to hold removing items
List<String> toRemove= new ArrayList<String>();

for (String name : names) {
   // Do something: perform conditional checks
   toRemove.add(name);
}    
names.removeAll(toRemove);

// now names list holds expected values