java.util.Map getOrDefault的陷阱

最近使用Java 8新增加的java.util.Map.getOrDefault()方法，结果却发现是一个大坑。

先上代码。

Map<String, String> map = Maps.newHashMap();
map.put("test", null);
System.out.println(map.getOrDefault("test", "default"));
System.out.println(map.getOrDefault("test", "default") == null);
System.out.println(org.apache.commons.collections4.MapUtils.getString(map, "test", "default"));

再看执行结果。

接下来开始分析。根据方法getOrDefault(Object key, V defaultValue)，我天真地认为该方法是先从Map中get(key)，如果获取到的值为null，那么就会取defaultValue，类似于org.apache.commons.collections4.MapUtils.getString(final Map<? super K, ?> map, final K key, final String defaultValue)。直到使用过后遇到了NPE时，我才认真看了下源码，源码如下。

源码的这个逻辑不是一般得坑人啊！从Map中get(key)的值不为null或者包含key时，就返回get(key)的值。但是这里有一个问题，Map中包含某个key时，该key对应的值也是可以为null的！本来想用getOrDefault来避免null并设置默认值的，结果还是有可能返回null并可能造成NPE问题。

结论：java.util.Map.getOrDefault() 方法并不能很贴切实际业务的使用，还是使用MapUtils.getXxx(final Map<? super K, ?> map, final K key, final Xxx defaultValue)来安全地获取非null值吧！