Findbugs-调用java.util.concurrent.ConcurrentHashMap的顺序可能不是原子的

Question

我的项目中有以下代码

// Declaration

private ConcurrentHashMap<Long, ConcurrentHashMap<String, Object>> instanceSpecificBeanMap = new ConcurrentHashMap<>();

地图的使用

ConcurrentHashMap<String, Object> beanMapForInstance = instanceSpecificBeanMap.get(currentInstanceId);
    Object beanObject = null;
    if(beanMapForInstance == null){
        synchronized (initLockObject) {
            beanMapForInstance = instanceSpecificBeanMap.get(currentInstanceId);
            if(beanMapForInstance == null){
                beanMapForInstance = new ConcurrentHashMap<>();
                instanceSpecificBeanMap.put(currentInstanceId, beanMapForInstance);
            }
        }
    }

我认为上述操作是原子的，尽管findbugs将其显示为问题？ 我在这里想念什么吗？

编辑

beanMapForInstance仅可从此类访问。 没有其他类可以访问此变量。

另外，目前我们正在使用Java 7

Answer 1

使用putIfAbsent方法。 如果给定键不存在新值，则恰恰是打算在映射中自动添加一个新值。

ConcurrentHashMap<String, Object> newMap = new ConcurrentHashMap<>();
ConcurrentHashMap<String, Object> beanMapForInstance = instanceSpecificBeanMap.putIfAbsent(currentInstanceId, newMap);
if (beanMapForInstance == null) {
    beanMapForInstance = newMap;
}
(... work with beanMapForInstance ...)

Answer 2

您可以使用Java 8中的computeIfAbsent简化此过程并解决问题：

ConcurrentHashMap<String, Object> beanMapForInstance = 
        instanceSpecificBeanMap.computeIfAbsent(
            currentInstanceId, 
            k -> new ConcurrentHashMap<>()  // might need explicit generics
        );