如何删除check/put的非原子使用并使代码线程安全？

Question

我有一个代码，我试图让我的班级的实例，因为我已经写了一个包装在java.util.logging.Logger。

下面的代码片段在我ClientLogger类 -

private static final Map<Class<?>, ClientLogger> s_classLoggers = new ConcurrentHashMap<Class<?>, ClientLogger>(); 

final private Logger m_logger; 

private ClientLogger(final Class<?> caller) { 
    m_logger = Logger.getInstance(caller); 
} 

public static ClientLogger getInstance(final Class<?> klass) { 
    final ClientLogger result; 

    if (s_classLoggers.containsKey(klass)) { 
     result = s_classLoggers.get(klass); 
    } else { 
     result = new ClientLogger(klass); 
     s_classLoggers.put(klass, result); 
    } 

    return result; 
} 

这是我初始化它在我，我需要用我上面记录的其他类的方式 -

private static final ClientLogger s_logger = ClientLogger.getInstance(TestLogger.class); 

现在当我运行我的静态分析工具时，它正在抱怨，因为在我的ClientLogger类中 -

Non-atomic use of check/put on this line s_classLoggers.put(klass, result); 

所以我修改了这样上面的代码，使其线程安全 -

private static final ConcurrentHashMap<Class<?>, ClientLogger> s_classLoggers = new ConcurrentHashMap<Class<?>, ClientLogger>(); 

public static ClientLogger getInstance(final Class<?> klass) { 
    ClientLogger result; 

    result = s_classLoggers.putIfAbsent(klass, new ClientLogger(klass)); 
    // is below line thread safe and efficient? 
    if (result == null) { 
     result = new ClientLogger(klass); 
    }  

    return result; 
} 

下面是我将初始化它让我的记录器实例的方式 -

private static final ClientLogger s_logger = ClientLogger.getInstance(TestLogger.class); 

所以我上面的代码线程安全？我正在做result == null检查，因为这是第一次，它不会出现在地图中，所以我需要为它创建一个新的值，因此我需要删除结果的最终修饰符。

Answer 1

你需要什么是Java 8的

s_classLoggers.computeIfAbsent(klass, ClientLogger::new); 

这只会创建对象，如果它真的有。

注意，ClientLogger::new是惊人的手短，因为它是短期的k -> new ClientLogger(k)这是短期的

new Function<Class<?>, ClientLogger>() { 
    public ClientLogger apply(Class<?> k) { 
     return new ClientLogger(k); 
    } 
} 

和拉姆达甚至不会在编译时产生的一类，尽管JVM可以（而且确实在Java 8中）在运行时创建一个类。

否则，您可能会发现使用写入锁定更安全。

public static ClientLogger getInstance(final Class<?> klass) { 
    ClientLogger result = s_classLoggers.get(klass); 
    if (result != null) 
     return result; // fast path 

    // slow, but rare path 
    synchronized (s_classLoggers) { 
     result = s_classLoggers.get(klass); 
     if (result == null) 
      s_classLoggers.put(klass, result = new ClientLogger(klass)); 
    } 
    return result; 
}