这是撰写的并发列表多图实现。较低级别的实现会更好,但更复杂。
忽略子列表中的O(n)删除,这是将ConcurrentMap和CopyOnWriteArrayList组合成功能性ConcurrentMultimap的正确方法吗?有没有未解决的数据竞赛?
private final ConcurrentMap<K, Collection<V>> map = ...; // inconsequential
public boolean put(K key, V value) {
Collection<V> list = map.get(key);
if(list != null) {
list.add(value);
return true;
}
// put if absent double check to avoid extra list creation
list = new CopyOnWriteArrayList<V>();
list.add(value);
Collection<V> old = map.putIfAbsent(key,value);
if(old != null) old.add(value);
return true;
}
public boolean remove(Object key, Object value) {
Collection<V> list = map.get(key);
if(list == null) return false;
// O(n) remove is the least of my worries
if( ! list.remove(value)) return false;
if( ! list.isEmpty()) return true;
// double-check remove
if( ! map.remove(key,list)) return true; // already removed! (yikes)
if(list.isEmpty()) return true;
// another entry was added!
Collection<V> old = map.putIfAbsent(key,list);
if(old == null) return true;
// new list added!
old.addAll(list);
return true;
}
答案 0 :(得分:3)
我认为你有一场比赛。我看到的问题是,'put'中的线程无法确定插入的列表是否已被删除,和/或被其他列表替换。
观察:
线程1调用put(),并检索(或创建)与密钥关联的列表。同时,线程2从地图中删除该列表。数据丢失。
我认为你需要添加一个重试循环来验证添加到地图后正确的列表是否在地图中。