java：Set.remove（）方法不从Set中移除对象

Question

在尝试从foreach()循环中的集合中移除对象时，我想我犯了一个小错误。remove()没有删除对象，有人能指出问题是什么吗？谢了！

Set<RoleExtEntity> roleExtEntitySet = roleEntity.getExt(); //set size = 3
Map<String, RoleExtEntity> roleExtEntityMap = 
        new HashMap<>(roleExtEntitySet.size());

roleExtEntitySet.stream().forEach(roleExtEntity -> {
      if (role.getName() == null && roleExtEntity.getKey().equals("name")) {
            //this line doesn't work; set size should be 2 but it's still 3
            roleExtEntitySet.remove(roleExtEntity);
      } else {
            roleExtEntityMap.put(roleExtEntity.getKey(), roleExtEntity);
      }
});

Answer 1

有人能指出问题是什么吗

如果没有看到您的代码，我想这是因为您没有正确地重写equals和/或hashcode，或者您修改了RoleextEntity，因为它是以影响hashcode的方式放入映射的。

尽管如此，如果此remove确实有效，您将冒ConcurrentModificationException，因为您正在从正在迭代的集合中删除。

for (Iterator<RoleExtEntity> it = roleExtEntitySet.iterator(); it.hasNext();) {
  RoleExtEntity roleExtEntity = it.next();
  if (...) {
    it.remove();
  } else {
    // ...
  }
}

另一种方法是将foreach拆分为执行移除的部分和执行放入的部分：

if (role.getName() == null) { // Since role appears to be unchanged in the loop
  roleExtEntitySet.removeIf(roleExtEntity -> roleExtEntity.getKey().equals("name"));
}

Map<String, RoleExtEntity> roleExtEntityMap =
    roleExtEntitySet.stream().collect(
        Collectors.toMap(RoleExtEntity::getKey, r -> r));