以下代码不是线程安全的吗？

Question

以下代码有什么问题？

private Map<Integer, Integer> aMap = new ConcurrentHashMap<Integer, Integer>();    
Record rec = records.get(id);
  if (rec == null) {
      rec = new Record(id);
      records.put(id, rec);
  }
  return rec;

1. 以上代码是不是线程安全的？putIfAbsent在这种情况下我为什么要在这里使用？
2. 锁定仅适用于更新.在检索的情况下,它允许完全并发.这句话是什么意思？

Answer 1

这不是线程安全的.

1. 如果有另一个线程,那么在它之间的时间records.get和records.put其他线程可能也放置了记录.

2. 只读操作(即不修改结构的操作)可以由多个线程同时完成.例如,1000个线程可以安全地读取a的值int.但是,如果int没有某种锁定操作,那1000个线程无法更新值.

我知道这可能听起来像是一个非常不可能发生的事件,但请记住,1百万分之一的事件在1GHz时每秒发生1000次.

这是线程安全的:

private Map<Integer, Integer> aMap = new ConcurrentHashMap<Integer, Integer>();
// presumably aMap is a member and the code below is in a function
aMap.putIfAbsent(id, new Record(id))
Record rec = records.get(id);
return rec;

请注意,这可能会创建一个Record从不使用它.