在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));
}
当前回答
for (String name : new ArrayList<String>(names)) {
// Do something
names.remove(nameToRemove);
}
克隆列表名称并在从原始列表中删除时遍历克隆名称。比上面的答案简洁一点。
其他回答
那些说你不能安全地从集合中删除一个项的说法是不正确的，你可以使用一个并发集合，比如ConcurrentHashMap。
for (String name : new ArrayList<String>(names)) {
// Do something
names.remove(nameToRemove);
}
克隆列表名称并在从原始列表中删除时遍历克隆名称。比上面的答案简洁一点。
Use
.remove() Interator或
Use
CopyOnWriteArrayList
“增强型for循环”的java设计是不向代码公开迭代器，但安全删除项的唯一方法是访问迭代器。所以在这种情况下，你得用老办法:
for(Iterator<String> i = names.iterator(); i.hasNext();) {
String name = i.next();
//Do Something
i.remove();
}
如果在实际代码中，增强的for循环确实值得这样做，那么您可以将这些项添加到临时集合中，并在循环之后调用列表上的removeAll。
EDIT(重编):不，在迭代时以iterator.remove()方法之外的任何方式更改列表都会导致问题。解决这个问题的唯一方法是使用CopyOnWriteArrayList，但这实际上是为了解决并发问题。
删除重复项最便宜的方法(就代码行数而言)是将列表转储到LinkedHashSet中(如果需要，再转储回list中)。这样可以在删除重复项的同时保留插入顺序。
确保这不是代码的味道。有可能颠倒逻辑，“包容”而不是“排他”吗?
List<String> names = ....
List<String> reducedNames = ....
for (String name : names) {
// Do something
if (conditionToIncludeMet)
reducedNames.add(name);
}
return reducedNames;
将我引导到这个页面的情况涉及到使用indecies从List中删除元素循环遍历List的旧代码。我想重构它以使用foreach样式。
它循环遍历整个元素列表，以验证用户有权限访问哪些元素，并从列表中删除没有权限的元素。
List<Service> services = ...
for (int i=0; i<services.size(); i++) {
if (!isServicePermitted(user, services.get(i)))
services.remove(i);
}
要反转此操作而不使用remove:
List<Service> services = ...
List<Service> permittedServices = ...
for (Service service:services) {
if (isServicePermitted(user, service))
permittedServices.add(service);
}
return permittedServices;
什么时候“remove”更合适?一个考虑因素是，如果给定一个大列表或昂贵的“添加”，与列表大小相比，只删除了一些内容。只做少量的删除可能比大量的添加更有效。但在我的案例中，情况并不值得这样的优化。
