并发哈希表作为带同步的singletone缓存

Question

您认为，我们需要使用同步块来更好地优化对广告实例的访问吗？Ad.class的实例可以从不同的线程中检索。Synchronized有助于通过ConcurrentHashMap中的一个get操作一次性获取实例。ConcurrentHashMap将所有值存储为易失性。我在android的java 1.7上使用它，computeIf Absent在java 1.8中可用。

如果能得到详细的答案，为什么不可以或者为什么可以，那就太好了。谢谢大家！

public final class Ad {

    private final static Map<String, Ad> ads = new ConcurrentHashMap<>();

    public static Ad get(@NonNull String appId) {
        if (appId == null) appId = "";

        boolean containsAd = ads.containsKey(appId);

        Ad localInstance = containsAd ? ads.get(appId) : null;

        if (localInstance == null) {
            synchronized (Ad.class) {

                containsAd = ads.containsKey(appId);

                localInstance = containsAd ? ads.get(appId) : null;

                if (localInstance == null) {
                    localInstance = new Ad();
                    localInstance.setAdId(appId);
                    ads.put(appId, localInstance);
                }
            }
        }
        return localInstance;
    }

    private Ad() {
    }
}

更新：感谢所有人的帮助。我将ConcurrentHashMap替换为HashMap。

Answer 1

据我了解，您实际想要实现的是 putIfAbsent，因此这比您所做的要简单得多（您正在使用双重检查锁定）：

public static Ad get(String appId) {
    String newId = appId == null ? "" : appId;
    ads.putIfAbsent(newId, new Ad());
    return map.get(newId);
}

Answer 2

这不是一个理想的选择。如果多个线程尝试同时初始化值，那么即使它们正在寻找不同的键，它们也会相互阻塞。

您应该使用 ConcurrentHashMap.computeIfAbsent 来检查添加并在一个步骤中创建缺失的添加。这样，您就不会创建任何未使用的广告，并且两个线程只会在尝试初始化同一条目时相互阻止：

public static Ad get(@NonNull String appId) {
    if (appId == null) appId = "";

    return ads.computeIfAbsent(appId, Ad::new);
}

private Ad(String appId) {
    this();
    setAdId(appId);
}