Pourquoi devons-nous créer un objet copy au début de la méthode supprimerNombresSuperieursA10 ?

Question sur la mission

Discussion en cours

public static HashSet<Integer> supprimerNombresSuperieursA10(HashSet<Integer> ensemble) { HashSet<Integer> copy = new HashSet<>(ensemble); for (Integer n : copy) { if (n > 10) { ensemble.remove(n); } } return ensemble; Pourquoi ne pas coder directement for(Integer n : ensemble) ?

Crée un ensemble de nombres (Set<Integer>) et ajoute-lui 20 nombres différents.
Supprime de l'ensemble tous les nombres supérieurs à 10.

Impératifs :

Le programme ne doit pas afficher de texte à l'écran.
Le programme ne doit pas lire de valeurs au clavier.
La classe Solution ne doit contenir que trois méthodes.
La méthode creerSet() doit créer et envoyer un HashSet contenant 20 nombres différents.
La méthode supprimerNombresSuperieursA10() doit supprimer de l'ensemble tous les nombres supérieurs à 10.

package fr.codegym.task.task08.task0814; import java.util.HashSet; import java.util.Set; /* Plus de 10 ? Ça ne va pas marcher. */ public class Solution { public static HashSet<Integer> creerSet() { // écris ton code ici HashSet<Integer> set = new HashSet<Integer>(); int valNb = 5; for(int i = 0; i<20; ++i) { set.add(valNb); valNb = valNb + 1; } return set; } public static HashSet<Integer> supprimerNombresSuperieursA10(HashSet<Integer> ensemble) { // écris ton code ici for (int nbSauv : ensemble) { if(nbSauv > 10) { ensemble.remove(nbSauv); } } return ensemble; } public static void main(String[] args) { } }

Commentaires (1)

Populaires
Nouveau
Anciennes

Tu dois être connecté(e) pour laisser un commentaire

Thomas

niveau Bayreuth, Bayreuth, Germany

14 mars, 17:58

utile

try it and you'll see that changing a collection (not elements inside the collection) in a for-each loop will throw a ConcurrentModificationException (I always recommend to write test code in the main method to check if your code is working) That's because internally a for-each loop uses an iterator. And when using an iterator you're only allowed to use methods of the iterator interface to modify the collection you loop over. As you do not have access to the underlying iterator in a for-each loop, you can not modify the collection in that very loop. So the alternative of looping over the copy and modifying the original would simply be to use an iterator

Iterator<Integer> it = ensemble.iterator();
while (it.hasNext())
    if (it.next() > 10)
        it.remove();

or you use the removeIf method from the Collection interface (but that may look weird at the moment)

ensemble.removeIf(e -> e > 10);