В книге я наткнулся на следующее утверждение:

Любые методы изменения, вызываемые для Iterator или ListIterator на основе копирования при записи (такие как добавление, установка или удаление), будут вызывать UnsupportedOperationException.

Но когда я запускаю следующий код, он работает нормально и не вызывает UnsupportedOperationException.

List<Integer> list = new CopyOnWriteArrayList<>(Arrays.asList(4, 3, 52));
System.out.println("Before " + list);
for (Integer item : list) {
    System.out.println(item + " ");
    list.remove(item);
}
System.out.println("After " + list);

Приведенный выше код дает следующий результат:

Before [4, 3, 52]
4 
3 
52 
After []

Почему я не получаю исключение при изменении заданного list с помощью метода remove?

3
skip 23 Сен 2018 в 18:37

2 ответа

Лучший ответ

Вы звоните remove по самому списку , и это нормально. В документации указано, что вызов remove в итераторе списка приведет к выдаче UpsupportedOperationException. Например.:

Iterator<Integer> iter = list.iterator();
while (iter.hasNext()) {
    Integer item = iter.next(); 
    System.out.println(item + " ");
    iter.remove(); // Will throw an UpsupportedOperationException
}
7
Mureinik 25 Сен 2018 в 01:51

Хорошо смотрится ответ Мурейника. Однако если вы погрузитесь в это глубоко -

При использовании ArrayList

  • Используйте цикл for и метод списка remove(current_index), он работает нормально.

А с подходом iterator -

  • Используйте метод итератора remove() вместо списка remove(), он отлично работает
  • Выбрасывает ConcurrentModificationException при использовании метода list remove()

В многопоточной среде -

Используйте CopyOnWriteArrayList

  • Со списком remove(current_element) все работает нормально
  • С помощью метода Iterator или ListIterator remove() он выбрасывает UnsupportedOperationException

Взгляните на пример ниже -

import java.util.*;
import java.util.concurrent.*;
public class AvoidCMEExample {
   public static void main(String args[]) {
        List<String> listOfBooks = new ArrayList<>();  
       listOfBooks.add("Programming Pearls");
       listOfBooks.add("Clean Code");
       listOfBooks.add("Effective Java");
       listOfBooks.add("Code Complete");
       System.out.println("List before : " + listOfBooks);
       /*for(int i=0; i<listOfBooks.size(); i++){
           String book = listOfBooks.get(i);
           if(book.contains("Programming")){
               System.out.println("Removing " + book);
               listOfBooks.remove(i); // works fine
           }
       }*/
       Iterator<String> itr = listOfBooks.iterator();
       while(itr.hasNext()){
            String book = itr.next();
            if(book.contains("Programming")){
               System.out.println("Removing " + book);
               //listOfBooks.remove(book); // will throw CME
               itr.remove(); // using iterator remove(), it works fine
           }
       }
       System.out.println("List after : " + listOfBooks);
       
       List<String> list = new CopyOnWriteArrayList<>(); 
       list.add("B"); list.add("W"); list.add("Q"); list.add("S");
       System.out.println("\n\nList before : " + list);
       Iterator<String> itr1 = list.iterator();
       while(itr1.hasNext()){
            String book = itr1.next();
            if(book.contains("Q")){
               System.out.println("Removing " + book);
               list.remove(book); // works fine on list object remove()
               itr1.remove(); // throws UnsupportedOperationException on iterator, ListIterator obj
           }
       }
       System.out.println("List after : " + list);       
   }
}
0
Ram 21 Мар 2021 в 06:57