Java CHM 同步

Question

跟进这个问题（Java线程安全-多原子操作？），我不想再增加更多的问题，但现在我有一个疑问：

private final Map<String, Set<String>> data = Maps.newConcurrentMap();

... then in a method ...

if (data.containsKey("A")) {
    data.get("A").add("B");
}

应该是这样的：

synchronized(data) {
    if (data.containsKey("A")) {
        data.get("A").add("B");
    }
}

为了保证线程安全。对吗？

所以操作是原子的，但将它们组合起来需要同步，对吗？在这一点上，只使用简单的 HashMap 而不是并发 HashMap 是否有意义，因为我们手动处理同步？

CHM中是否有任何方法可以原子地使其工作？

Answer 1

匿名用户

synchronised(data) {
    if (data.containsKey("A")) {
        data.get("A").add("B");
    }
}

你可能需要展示更多的代码。

仅从这一点来看，唯一可能的问题是，在您进行< code>if检查后，有人移除了在< code >“A”找到的集合。如果你从来没有删除映射条目，你根本不需要同步。

如果同时删除映射条目，则可以使用 computeIfPresent 到达更新的映射。

你也可以做

Set<String> set = data.get("A");
if (set != null) set.add("B");

由于您实际上并没有生成一个新的Set，我发现这比computeIfPresent（它应该计算一个新值）更惯用。

请注意，您还需要使所有这些 Set 成为线程安全的。

Answer 2

在您的特定情况下，您可能希望使用Con

data.computeIfPresent("A", (k, v) -> { v.add("B"); return v; } );

来自javadocs：

如果指定键的值存在，则在给定该键及其当前映射值的情况下，尝试计算新的映射。整个方法调用是自动执行的。

所以不需要显式同步。