Pouvez vous m'aidez?

Question sur la mission

Discussion en cours

Je ne suis pas sur ou est mon erreur. Merci :)

1. Crée une liste de mots et remplis-la par toi-même.
2. La méthode corriger doit :
2.1. supprimer tous les mots contenant la lettre « r » de la liste
2.2. dupliquer tous les mots contenant la lettre « l ».
2.3. si un mot contient à la fois « r » et « l », laisse-les inchangés.
2.4. ne fais rien aux autres mots.

Par exemple :
rose lyre love

Sortie :
lyre love love

Impératifs :

Le programme ne doit pas lire de données du clavier.
La méthode corriger doit supprimer tous les mots contenant la lettre « r » de la liste. Il y a une exception : les mots contenant à la fois « r » et « l » doivent rester inchangés.
La méthode corriger doit dupliquer les mots contenant la lettre « l » (ajouter une autre instance de ce mot à la liste). Il y a une exception : les mots contenant à la fois « l » et « r » ne doivent pas être dupliqués.
La méthode corriger ne doit rien faire avec les mots qui contiennent à la fois « l » et « r ».
La méthode corriger ne doit rien faire avec les mots qui ne contiennent ni « l » ni « r ».

package fr.codegym.task.task07.task0716; import java.util.ArrayList; /* R ou L */ public class Solution { public static void main(String[] args) throws Exception { ArrayList<String> liste = new ArrayList<String>(); liste.add("rose"); // 0 liste.add("love"); // 1 liste.add("lyre"); // 2 liste = corriger(liste); for (String s : liste) { System.out.println(s); } } public static ArrayList<String> corriger(ArrayList<String> liste) { for(int i = 0; i < liste.size(); i++){ String s = liste.get(i); if (liste.get(i).contains ("r")) if (liste.get(i).contains ("r") && !liste.get(i).contains ("l")) liste.remove(s); else i++; else if (liste.get(i).contains ("l")) if (liste.get(i).contains ("l") && !liste.get(i).contains ("r")) liste.add(s); else i++; } return liste; }}

Commentaires (2)

Populaires
Nouveau
Anciennes

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

Romain

niveau Paris, Paris, France

4 janvier 2021, 09:52

utile

Bonjour, Avec ce code tu obtiens en résultat

love
lyre

Pourtant l'impératif 3, de duplication des mots contenant la lettre 'l' sans lettre 'r' est validée... cela se comprend car si tu changes l'ordre d'entrée des mots dans la liste initiale et place le mot "love" en première ou dernière place le mot est effectivement dupliqué mais cela entraine aussi une exception OutOfMemoryError correspondant à une boucle infini, car son ajout induit une augmentation de la taille de ta liste, du coup ta boucle se relance pour traiter le mot supplémentaire qui sera donc un mot à dupliquer alors la liste augmentera à nouveau, ta boucle se relancera à nouveau pour traiter ce nouveau mot ajouté, et cela se répètera sans cesse. Je suppose que c'est pour cela que tes mots sont placés dans un ordre différent de celui présenté en exemple, car avec l'ordre de ta liste (rose// 0, love// 1, lyre// 2) et ton code, le mot "love" n'est jamais traité, là est un autre soucis de ton code et explique le résultat obtenu. Avec ce code en fait tout mot placé après un mot destiné à être supprimé ou un mot destiné à être inchangé n'est pas traité. Dans le premier cas, après la suppression du mot, le second ne sera pas traité car l'index de parcours de la liste continu son incrémentation bien que le second mot prenne la place du mot supprimé. Dans le second cas, la non validation des différentes conditions "if" conduit à l'incrémentation de la boucle avant que celle ci ne s'incrémente d'elle même. Donc il te faut corriger : - la prise en compte dans ta boucle de l'ensemble des mots composants la liste initiale. - la non prise en compte des mots dupliqués (soit en définissant un compteur max de nombre de tour de boucle à effectuer variant au gré des suppressions des mots, soit en plaçant le mot dupliqué juste après le mot traité et en sautant un tour de boucle pour ne pas le traiter, chose que tu fais déjà dans ton code)

david

4 janvier 2021, 10:27

Merci. Je croit que j'ai compris :)