java - 带有迭代器的 ConcurrentModificationException

Question

在 DTO 中我有

public class ProductTypesDto extends BaseDto {
  private List<Integer> colors = new ArrayList<>();
  ...
}

在我的 bean 里

@Entity
public class ProductTypes 
   @ManyToMany
   private Set<Colors> colors = new HashSet<>();
   ...
}

用户可以添加和删除productTypes的颜色

在 DTO 到 bean 的转换中，我这样做

private void convertToBeans(ProductTypesDto dto, ProductTypes beans) {
    //add element
    for (Integer color : dto.getColors()) {
        if (beans.getColors().stream().noneMatch(e -> Objects.equals(e.getId(), color))) {
            Optional<Colors> optColors = colorsRepository.findById(color);
            if (optColors.isPresent()) {
                beans.addColor(optColors.get());
            }
        }
    }
    //remove element
    for (Iterator<Colors> iterator = beans.getColors().iterator(); iterator.hasNext();) {
        Colors color = iterator.next();

        if (dto.getColors().stream().noneMatch(e -> e.intValue() == color.getId())) {

            Optional<Colors> optColors = colorsRepository.findById(color.getId());
            if (optColors.isPresent()) {
                beans.removeColor(optColors.get());
            }

        }
    }
}

当代码运行时，我得到

java.util.ConcurrentModificationException: null

似乎在 iterator.next(); 上得到这个

Answer 1

您无法迭代 beans.getColors() 集合并通过调用 beans.removeColor() 从中删除，这很可能执行 colors.remove ().

要从底层集合中删除当前遍历的元素，请使用 Iterator.remove()方法例如替换:

beans.removeColor(optColors.get());

与:

iterator.remove();