java - 避免 map.get(key) 方法

Question

我有以下代码，但我看到在迭代 Map 键时从 Map 中检索值keySet()是一个错误，即使使用 findBugs 我收到警告WMI_WRONG_MAP_ITERATOR

for(String elementId : mapElements.keySet()){

     element = mapElements.get(elementId); 

     doSomething(element);
}

那么为什么这不好，我该如何解决呢？

谢谢。

Answer 1

如果您要遍历地图中的所有内容，您不妨这样做：

for (Map.Entry<String, String> entry : mapElements.entrySet()) {
    String key = entry.getKey();
    String value = entry.getValue();
    // Use the key and the value
}

或者，如果您真的不需要密钥，只需遍历值：

for (String value : mapElements.values()) {
    doSomething(value);
}

编辑：语法

Answer 2

在迭代地图本身的同时从地图中检索值不是问题 - 成为问题的是当您修改地图同时对其进行迭代时。在您的情况下，情况似乎并非如此，因此这本身并不危险。

当你迭代一个 map 时，你得到的迭代器是基于你获取迭代器时所有 map 条目的快照。在随后的 midification 中，此迭代器的行为变得未定义。这是不好的。但同样，在您的情况下，这并不适用，因为您没有更新地图。

Answer 3

另一点是，如果地图很大，查找每个键的值可能会很昂贵。所以 Jon Skeet 的建议更有效率。但是，我承认遍历地图条目集的代码有点笨拙。