Java CHM синхронизации

Question

Следуя по этому вопросу (Java thread safety - multiple atomic operations?), я не хочу, чтобы добавить больше вопросов к нему, но теперь у меня есть это сомнение:

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 вместо параллельного, поскольку мы вручную обрабатываем синхронизацию?

Есть ли какой-либо метод в CHM, чтобы сделать эту работу атомарно?

Answer 1

В вашем конкретном случае, вы можете захотеть использовать computeIfPresent method из ConcurrentHashMap:

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

Из Javadocs:

Если значение для указанного ключа присутствует, пытается вычислить новый отображение с учетом ключа и текущего отображаемого значения. Весь вызов метода выполняется атомарно.

Так что нет необходимости в явной синхронизации.

Answer 2

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

Возможно, вам потребуется больше кода.

Если вы хотите узнать только об этом, то единственная возможная проблема заключается в том, что кто-то удаляет набор, найденный по адресу "A" после вашего if чек. Если вы никогда не удаляете записи на карте, вам совсем не нужна синхронизация.

Если вы одновременно удаляете записи на карте, вы можете использовать computeIfPresent, чтобы получить обновленную карту.

Вы также мог бы сделать

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

Так как вы на самом деле не новую производственной набор, я нахожу это более идиоматический чем computeIfPresent (который должен вычислить новое значение).

Обратите внимание, что вам необходимо также сделать все эти наборы потокобезопасными.