ArrayList.remove (i) не удаляет все объекты, как это должно быть

Question

У меня есть простой список (recyclerView), где у каждого элемента списка есть флажок. Все элементы, которые были проверены, должны быть удалены с помощью этого кода:

for(int i = 0; i<shopListItems.size(); i++) { 
    if(shopListItems.get(i).isBought == 1) { 
     shopListItems.remove(i); 
    } 
} 

Но по какой-то причине он не разбирается. Если проверено 10 элементов, то только 7 из них будут удалены, и я должен снова запустить код, чтобы удалить остальные.

кода кажется нормально, но я действительно не понимаю, почему это Dont удалить все отмеченное или все, что (isBought == 1)

Answer 1

remove(index) не только удалить объект, но переложить все остальные объекты влево, чтобы закрыть этот пробел. Вам необходимо выполнить итерацию с конца списка до начала

for(int i = shopListItems.size() - 1 ; i >= 0 ; i--) { 
    if(shopListItems.get(i).isBought == 1){ 
     shopListItems.remove(i); 
    } 
} 

Answer 2

Обратный цикл!

 for(int i = shopListItems.size() -1; i >=0 i--){ 
      if(shopListItems.get(i).isBought == 1){ 
       shopListItems.remove(i); 
      } 
     } 

Если вы не собираетесь удалить всю ArrayList, вы не должны беспокоиться, потому что, как изменения размера, это не повлияет на остальную часть испытаний.

Answer 3

Помимо способов, упомянутых выше, вы также можете сделать это в порядке возрастания:

for(int i = 0; i < list.size(); i++){ 
    if(list.get(i).isBought == 1){ 
     list.remove(i--); 
    } 
} 

Существует еще один простой альтернативой, если вы используете Java 8:

list = list.stream() 
     .filter(item -> item.isBought == 1) 
     .collect(Collectors.toList()); 

Ссылка: http://zeroturnaround.com/rebellabs/java-8-explained-applying-lambdas-to-java-collections/