Problème avec méthodes instanceof() et remove()

Bonjour !

Je suis étudiant (pas en informatique), je bosse sur un projet et je suis face à un problème que j'arrive pas à résoudre.

Voilà, je testait mon code qui met notamment en jeu des méthodes de ce type :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15

public static ArrayList<Ouvrage> garderRomans(ArrayList<Ouvrage> listouvr){   for (int i = 0; i < listouvr.size(); i++) { if (listouvr.get(i) instanceof DVD){ DVD dvd = (DVD)listouvr.get(i); listouvr.remove(dvd); } if (listouvr.get(i) instanceof CD){ CD cd = (CD)listouvr.get(i); listouvr.remove(cd); } }   return listouvr; }

---

Et en remarquant des problèmes dans mon code, j'ai fait les tests suivants pour essayer de mieux comprendre ce qu'il se passe (en vain ^^):
La je teste si un élément d'une liste d'ouvrage (dont Roman, CD et DVD sont des sous-classes) est un DVD, et l'affiche si oui. Jusque là tout va bien :
Pièce jointe 232817

Puis je teste si un élément de cette liste est un DVD, et l'enlève de cette liste si oui. Ensuite j'affiche la liste. Je m'attends donc à ce que la liste contienne tous les ouvrages présents précédemment, sans les DVDs. Or ce n'est pas le cas.
Pièce jointe 232819

Voilà, j'attend votre aide ! :)

PS : Je tiens à préciser que je code depuis très peu de temps, et ce n'est que mon deuxième vrai "projet" depuis. J'ai donc des connaissances du langage et de la pensée informatique limitées.

Quand tu supprime une élément, le reste des éléments de ta liste, qui étaient derrière, bougent pour combler le trou. Si tu retire l'élément en position 4 par exemple, l'élément qui étaient en position 5 bouge en 4. Celui qui était en 6 bouge en 5. Etc.
Ta boucle, elle fait 1 2 3 4 5 ... Donc elle suppose que les éléments ne changent pas de place.


Pour parcourir une liste en supprimant des élément, on peut utiliser un iterator, avec ses méthodes next() et remove(). Si tu utilise java 8, tu peux préférer créer une nouvelle liste avec stream().filter() ou tu peux utiliser la méthode removeIf() qui fait déjà le boulot de supprimer les éléments correspondant à un critère. Au passage, si le but est de garder tous les romans, teste que c'est un roman pas que ce n'est ni un DVD ni un CD. Si demain tu ajoute des cassettes tu ve devoir revoir ton code.

Code:

---

listouvr.removeIf(ouvrage -> !(ouvrage instanceof Roman));

---

Merci beaucoup, j'ai pu régler mes soucis, et simplifier mon code par la même occasion ! Au passage j'ai une nouvelle question :

lorsque j'affecte à une ArrayList A nouvellement créée une ArrayList déjà existante B (étant dans ce cas un attribut d'une classe), je remarque que les deux se retrouvent liées. En effet, lorsque je retire un élément de A, celui ci est aussi retiré de B, (ce qui ne me paraît pas normal, car si on fait le même type de raisonnement avec des entiers tout colle) Code :

Pièce jointe 232876

Finalement j'ai réussi à réglé mon problème (ArrayList <> A = new ArrayList (B))