Discussion :

[lodacom]

Vous allez peut-être me prendre pour un fou furieux, mais lorsque je fais un parcours total de mon tableau (voici le code associé)

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
 
public void supprimeMotNonImbrique()
    {
        /**grâce à cette méthode (après traitement de supprimeMotDejaExistant()),
         * on supprime les mots dans listeTemporaire qui ne sont pas imbriqués
         * c'est à dire que lors de la génération (lorsque l'on génère le tableau listeImbrication ,voir 
         * CliqueTemporaire)on ne retrouve pas tous les mots dans listeMot.
         * (deuxième filtre)
         */
        System.out.println(this.listeTemporaireMot.size());
        for (int i=0;i<this.listeTemporaireMot.size();i++)
        {
            this.clique=new CliqueTemporaire(this.listeTemporaireMot.get(i));
            this.clique.generationTableauTemporaire();
            if (!this.clique.verificationImbrication())
            {
                this.listeTemporaireMot.remove(i);
            }
            System.out.print(i);
        }
        this.generationChaqueMot();
    }

Lorsque je veux afficher la size il me rend 64 (ce qui est tout à fait exact). Mais quand je rentre dans le for (quand j'affiche i

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

System.out.print(i);

il m'affiche de 0-31 c'est à dire exactement la moitié de mon tableau). Je signale au passage que mon tableau est de type ArrayList. Bref alors que je devrais parcourir entièrement mon tableau (0-63), je ne parcours que la moitié.
Dommage je peux pas intégrer d'image de mon ordi.
Y a-t-il quelque chose que je ne fais pas? (je n'ai aucune erreur signalé par eclipse)

[Rei Ichido]

Tu supprimes des objets de ta liste, pas étonnant que tu n'ailles pas au bout.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
this.listeTemporaireMot.remove(i);

[lodacom]

Et alors je vois pas le problème. Quelque soit les opérations que l'on fasse sur un ArrayList (si on fait un parcours total) on doit aller jusqu'au bout même si l'on supprime des mots (dans mon cas).

[Rei Ichido]

Si tu passes par l'indice, non. La façon propre de le faire serait avec un iterateur, ou bien avec un foreach en construisant une liste d'objets à retirer à la fin.

Imagine une liste de 2 éléments.
i = 0 => Tu testes le 1er élément. Pas bon, tu le retires. La taille de ta liste est maintenant 1.
i = 1. Tu sors de ton for.
Moralité tu n'as pas testé ton 2ème élément.

[lodacom]

Ok je vois pourquoi maintenant il ne fait que la moitié. Pour le foreach j'ai déjà essayé il me fait une erreur de commodification. En gros il ne veut pas que je modifie pour chaque mot. A moins que maintenant ça soit possible. Il faudrait que j'essaie. Bon j'ai essayé j'avais raison:
at java.util.AbstractList$Itr.checkForComodification(Unknown Source)
at java.util.AbstractList$Itr.next(Unknown Source)
Conclusion comment supprimer toute la liste et non pas la moitié en ayant un remove?

[Rei Ichido]

Pour le foreach comme je l'ai dit, si tu veux l'utiliser il faut construire la liste des objets à retirer, et les retirer tout à la fin.

Sinon, utiliser un itérateur.

Enfin, il y a la possibilité de faire i-- quand tu retires un objet.