ConcurrentModificationException for ArrayList

collections concurrency java

2022-08-31 11:27:07

我有以下代码：

private String toString(List<DrugStrength> aDrugStrengthList) {
    StringBuilder str = new StringBuilder();
        for (DrugStrength aDrugStrength : aDrugStrengthList) {
            if (!aDrugStrength.isValidDrugDescription()) {
                aDrugStrengthList.remove(aDrugStrength);
            }
        }
        str.append(aDrugStrengthList);
        if (str.indexOf("]") != -1) {
            str.insert(str.lastIndexOf("]"), "\n          " );
        }
    return str.toString();
}

当我尝试运行它时，我得到，任何人都可以解释为什么它会发生，即使代码在同一线程中运行？我该如何避免呢？ConcurrentModificationException

答案 1

如果您使用“for each”循环浏览它，则无法从列表中删除。您可以使用 .取代：Iterator

for (DrugStrength aDrugStrength : aDrugStrengthList) {
    if (!aDrugStrength.isValidDrugDescription()) {
        aDrugStrengthList.remove(aDrugStrength);
    }
}

跟：

for (Iterator<DrugStrength> it = aDrugStrengthList.iterator(); it.hasNext(); ) {
    DrugStrength aDrugStrength = it.next();
    if (!aDrugStrength.isValidDrugDescription()) {
        it.remove();
    }
}

答案 2

就像其他答案说的那样，您无法从要迭代的集合中删除项目。您可以通过显式使用 a 并在此处删除项目来解决此问题。Iterator

Iterator<Item> iter = list.iterator();
while(iter.hasNext()) {
  Item blah = iter.next();
  if(...) {
    iter.remove(); // Removes the 'current' item
  }
}