Как я могу исправить эту ошибку java.util.ConcurrentModificationException
Я получаю сообщение об ошибке в следующей строке. Я делаю процесс добавления к jsonarray. Пожалуйста, помогите мне.
jsonArr=new JSONArray();
if(req.getSession().getAttribute("userses")!=null){
String name=(req.getParameter("name")==null?"":to_EnglishName(req.getParameter("name").toUpperCase()));
if(!name.equals("")){
for(Book c:GlobalObjects.bookList){
if(c.getBookName().startsWith(name)){
jsonObjec=new JSONObject();
jsonObjec.put("label",c.getBookName());
jsonObjec.put("value", c.getId());
jsonArr.add(jsonObjec);//java.util.ConcurrentModificationException
}
}
}
}
jsonArr.write(res.getWriter());
Ответы на вопрос(7)
когда вы изменяете его одновременно.
ConcurrentModificationException генерируется, когда вы изменяете коллекцию в то же время, пока выполняете ее.
который позволяет повторять и изменять один и тот же список
http://docs.oracle.com/javase/6/docs/api/java/util/ListIterator.html
когда вы перебираете коллекцию и изменяете ту же коллекцию в цикле. Данный фрагмент кода не показывает этого, поэтому есть что-то еще выше или ниже, изменяющее коллекцию. Попробуйте объявить jsonArr прямо в том месте, где вы его создали.
Одной из возможных причин может быть экземпляр объекта jsonArr на уровне класса, доступ к которому осуществляется несколькими потоками. Объявите объект jsonArr, в котором он создан.
Редактировать: Сделать jsonArr локальной переменной.
If your collection is not thread safe then it must not get modified with another thread when some other thread is iterating over this collection.
Есть два возможных способа решить эту проблему -
1) Одним из решений является синхронизация всего доступа к коллекции
2) Используйте поток безопасной коллекции, какCopyOnWriteArrayList
Из Java Doc -
This exception may be thrown by methods that have detected concurrent modification of an object when such modification is not permissible.
For example, it is not generally permssible for one thread to modify a Collection while another thread is iterating over it. In general, the results of the iteration are undefined under these circumstances. Some Iterator implementations (including those of all the collection implementations provided by the JRE) may choose to throw this exception if this behavior is detected. Iterators that do this are known as fail-fast iterators, as they fail quickly and cleanly, rather that risking arbitrary, non-deterministic behavior at an undetermined time in the future.
с которой я часто сталкивался при перепрограммировании. причина или детали этого исключения довольно ясны. Нельзя изменять коллекцию (вы добавляете новый элемент), пока она повторяется. По крайней мере, синтаксисfor НЕ поддержите, сделайте это.
Чтобы решить вашу проблему, есть два способа, я думаю, это просто.
1). вместо того, чтобы использоватьfor оператор зацикливания, лучше использовать итератор, чтобы избежать ConcurrentModificationException.
Iterator<Book> iterator = bookList.iterator();
while(iterator.hasNext()){
Book c = iterator.next();
if(c.getBookName().startsWith(name)){
jsonObjec=new JSONObject();
jsonObjec.put("label",c.getBookName());
jsonObjec.put("value", c.getId());
jsonArr.add(jsonObjec);
}
}
2). Зацикливая его, не добавляйте его.
List list = new ArrayList<>();
for(Book c:GlobalObjects.bookList){
if(c.getBookName().startsWith(name)){
jsonObjec=new JSONObject();
jsonObjec.put("label",c.getBookName());
jsonObjec.put("value", c.getId());
list.add(jsonObjec);//java.util.ConcurrentModificationException
}
}
jsonArr.addAll(list);