当使用map.get（）时，使用javaMap.containsKey（）是多余的吗？

Question

一段时间以来，我一直想知道，在最佳实践中，是否可以避免在< code>java.util.Map上使用< code>containsKey()方法，而是对< code>get()的结果进行空检查。

我的理由是，对值进行两次查找似乎是多余的——第一次是对< code>containsKey()进行查找，然后再次对< code>get()进行查找。

另一方面，可能是大多数Map的标准实现缓存了最后的查找，或者编译器可以以其他方式消除冗余，并且为了代码的易读性，最好保持包含Key（）部分。

我非常感谢您的评论。

Answer 1

正如assylias指出的，这是一个语义问题。通常，Map.get(x) == null是您想要的，但是在某些情况下，使用containsKey很重要。

缓存就是这样一个例子。我曾经在一个Web应用程序中处理过一个性能问题，该应用程序经常查询其数据库，寻找不存在的实体。当我研究该组件的缓存代码时，我意识到如果cache.get（key）==null，它就是在查询数据库。如果数据库返回null（未找到实体），我们将缓存该键-

切换到containsKey解决了这个问题，因为映射到null值实际上意味着什么。到null的键映射与不存在的键具有不同的语义含义。

Answer 2

我认为这样写是相当标准的:

Object value = map.get(key);
if (value != null) {
    //do something with value
}

而不是

if (map.containsKey(key)) {
    Object value = map.get(key);
    //do something with value
}

它的可读性和效率都没有降低，所以我认为没有任何理由不这样做。显然，如果您的映射可以包含null，那么这两个选项就没有相同的语义。

Answer 3

允许某些Map实现具有空值，例如HashMap，在这种情况下，如果get（key）返回null，则不能保证map中没有与此键关联的条目。

因此，如果您想知道映射是否包含键，请使用< code>Map.containsKey。如果您只需要映射到键的值，请使用< code>Map.get(key)。如果这个映射允许null值，那么null的返回值不一定表示这个映射不包含这个键的映射；在这种情况下，< code>Map.containsKey没有用，并且会影响性能。此外，在并发访问映射(例如< code>ConcurrentHashMap)的情况下，在您测试了< code > map . contains key(key)之后，在您调用< code>Map.get(key)之前，该条目可能会被另一个线程删除。