记录一次看书遇到的问题,描述如下:
Java中 提供了一些线程安全的容器和对象,有些事支持并发的,java.util.concurrent 包中 有很多这样的类。这种方式的效率会比简单的加互斥锁的实现更好,但是有时通过加锁把使用线程不安全的容器代码改为使用线程安全容器的代码时,会遇到一个陷阱。 使用map存储统计信息,先用HashMap
private HashMap<String, Integer> map = new HashMap<>();
public synchronized void add(String key){
Integer value = map.get(key);
if (value == null){
map.put(key, 1);
}else{
map.put(key, value + 1);
}
}
再试试ConcurrentHashMap,这次把synchronized去掉
private ConcurrentHashMap<String, Integer> concurrentHashMap = new ConcurrentHashMap<>();
public static void add(String key){
Integer value = concurrentHashMap.get(key);
if (value == null){
concurrentHashMap.put(key, 1);
}else{
concurrentHashMap.put(key, value + 1);
}
}
作者提出了后者的代码会出现问题,我一开始想ConcurrentHashMap不至于出问题啊,线程安全的HashMap,分段锁balabala的,但问题出现在synchronized上
synchronized修饰了方法,这个方法就类似同步代码块,线程安全,因此里面的两个map方法整体就是线程安全的,再看后面没加synchronized的方法,get和put之间并不是线程安全的,concurrentHashMap只负责单个map操作线程安全,并不负责两个map操作之间线程安全,因此会出现问题。
还请各位看官多多指教。