我有一个Java类,该线程可以同时被许多线程访问,并希望确保它是线程安全的.该类具有一个私有字段,该私有字段是字符串到字符串列表的映射.我已经将Map实现为ConcurrentHashMap,以确保获取和放置都是线程安全的:
public class ListStore {
private Map> innerListStore;
public ListStore() {
innerListStore = new ConcurrentHashMap>();
}
...
}
因此,鉴于Map的获取和放置是线程安全的,我关心的是Map中存储的列表.例如,考虑以下方法,该方法检查商店中给定列表中是否存在给定条目(为了简洁起见,我省略了错误检查):
public boolean listEntryExists(String listName, String listEntry) {
List listToSearch = innerListStore.get(listName);
for (String entryName : listToSearch) {
if(entryName.equals(listEntry)) {
return true;
}
}
return false;
}
似乎我需要同步此方法的全部内容,因为如果在此方法对其进行迭代时,如果另一个方法更改了innerListStore.get(listName)上列表的内容,则将引发ConcurrentModificationException.
那是正确的吗?如果是这样,我应该在innerListStore上进行同步还是在本地listToSearch变量上进行同步?
更新:感谢您的答复.听起来我可以在列表本身上进行同步.有关更多信息,这是add()方法,可以在另一个线程中运行listEntryExists()方法的同时运行该方法:
public void add(String listName, String entryName) {
List addTo = innerListStore.get(listName);
if (addTo == null) {
addTo = Collections.synchronizedList(new ArrayList());
List added = innerListStore.putIfAbsent(listName, addTo);
if (added != null) {
addTo = added;
}
}
addTo.add(entryName);
}
如果这是唯一修改存储在映射中的基础列表的方法,并且没有公共方法返回对映射的引用或映射中的条目,我是否可以同步列表本身的迭代,并且add()的这种实现是否足够?