我正在做:
for (Object key : map.keySet())
if (something)
map.remove(key);
它抛出了一个ConcurrentModificationException，所以我把它改为:
for (Object key : new ArrayList<Object>(map.keySet()))
if (something)
map.remove(key);
这个过程以及修改映射的任何其他过程都在同步块中。
有没有更好的解决方案?
当前回答
从Java 8开始，你可以这样做:
map.entrySet().removeIf(e -> <boolean expression>);
entrySet()
集合是由映射支持的，所以对映射的更改会反映在集合中，反之亦然
其他回答
这也应该可以工作。
ConcurrentMap<Integer, String> running = ... create and populate map
Set<Entry<Integer, String>> set = running.entrySet();
for (Entry<Integer, String> entry : set)
{
if (entry.getKey()>600000)
{
set.remove(entry.getKey());
}
}
也许您可以遍历映射，寻找要删除的键，并将它们存储在单独的集合中。然后从映射中删除键集合。在迭代过程中修改地图通常是不受欢迎的。如果地图非常大，这个想法可能是可疑的。
Set s=map.entrySet();
Iterator iter = s.iterator();
while (iter.hasNext()) {
Map.Entry entry =(Map.Entry)iter.next();
if("value you need to remove".equals(entry.getKey())) {
map.remove();
}
}
ConcurrentHashMap
你可以使用java.util.concurrent.ConcurrentHashMap。
它实现了ConcurrentMap(扩展了Map接口)。
例如:
Map<Object, Content> map = new ConcurrentHashMap<Object, Content>();
for (Object key : map.keySet()) {
if (something) {
map.remove(key);
}
}
这种方法不会影响代码。只是映射类型不同。
有没有更好的解决方案?
当然，在单个语句中有更好的方法，但这取决于基于删除哪些元素的条件。
例如:删除所有值为test的元素，然后使用下面的代码:
map.values().removeAll(Collections.singleton("test"));
更新 这可以在一行中使用Java 8中的Lambda表达式完成。
map.entrySet().removeIf(e-> <boolean expression> );
我知道这个问题太老了，但更新更好的做事方式并没有什么坏处:)
