Discussion :

[ceres02]

Bonjour,

j'avais une partie de code qui utilisait des hashtable et des enumeration. J'ai converti tous les hashtable en treemap et les enumeration en iterator pour remettre aux gouts du jour =)
Cependant, à un endroit précis, j'ai:

Code :

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

1
2
3
4
5
6
7
Iterator clesFils = fils.keySet().iterator();
 
            while (clesFils.hasNext()) {
               String cleTemp = (String)clesFils.next();
                Fil filTemp = ((Fil)fils.get(cleTemp));
                if (filTemp.estRelieA(codeElem))    supprimerFil(cleTemp);
            }

Ce qui engendre une erreure ConcurrentModificationException, je crois.
Tout ceci est du au fait que la methode supprimerFil implique des changements sur le treeMap "fils" et donc l'iterator n'aime pas.
Après avoir fait quelques recherches, j'ai essayé de synchroniser l'iterator et cela donne :

Code :

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

1
2
3
4
5
6
7
8
9
10
Set s = Collections.synchronizedSet(fils.keySet()); 
         synchronized(s) {
        Iterator clesFils = s.iterator();
 
            while (clesFils.hasNext()) {
               String cleTemp = (String)clesFils.next();
                Fil filTemp = ((Fil)fils.get(cleTemp));
                if (filTemp.estRelieA(codeElem))    supprimerFil(cleTemp);
            }
         }

Et cela ne résoud pas mon problème, malheureusement. Ai je mal fait quelque chose ?
Sous netbeans, voila ce qu'il me met :

Exception in thread "AWT-EventQueue-0" java.util.ConcurrentModificationException
at java.util.TreeMap$PrivateEntryIterator.nextEntry(TreeMap.java:1031)
at java.util.TreeMap$KeyIterator.next(TreeMap.java:1058)
at saisieschema.FenetreEditeurSchema.supprimerElement(FenetreEditeurSchema.java:1060)

Si vous avez des idées, merci !

[zekey]

Pour supprimer un élément il faut utiliser la méthode remove de l'interface Iterator.

[ceres02]

oui mais je ne vois pas en quoi ca résoud mon problème:

Code :

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

1
2
3
4
5
6
7
8
9
10
private void supprimerFil(String codeFil)  {
        Fil filASupprimer = ((Fil)fils.get(codeFil));
        filASupprimer.deconnecterFil(elements,fils);
        int sortieFil = filASupprimer.getSortie();
        fils.remove(codeFil);
 
        // on supprime les noeuds isolés
        eliminerNoeudsIsoles();
 
        updateGrilleTraces();

J'utilise bien un fils.remove(key) dans la methode supprimerFil

[zekey]

Ce n'est pas un remove sur l'iterator mais sur le TreeMap.
Moi je dis un remove sur l'iterator.

[adiGuba]

Salut,

Ce que veux dire zekey c'est que pendant le parcours d'une Collection avec un itérateur, il est interdit de modifier la collection !

Ce n'est pas un problème de synchronisation multithread mais bien d'algorythme.

J'utilise bien un fils.remove(key) dans la methode supprimerFil

Justement l'erreur vient de là ! Tu devrait faire :

Code :

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

clefsFils.remove();

a++

[ceres02]

Ca a l'air de marcher impec.
J'ai recrée une 2nd méthode supprimerFil avec l'iterator en argument car j'ai besoin d'une méthode

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
private void supprimerFil(String codeFil, Iterator clesFils)  {
        Fil filASupprimer = ((Fil)fils.get(codeFil));
        filASupprimer.deconnecterFil(elements,fils);
        int sortieFil = filASupprimer.getSortie();
 
        clesFils.remove();
 
        // on supprime les noeuds isolés
        eliminerNoeudsIsoles();
 
        updateGrilleTraces();
    }

Ma question est: j'ai du enlever le fils.remove(codeFil) mais pour du coup, je n'efface plus les fils de mon TreeMap. est ce que clesFils.remove() s en charge implicitement ?

[ceres02]

je viens de lire la javadoc, ca répond toujours à nos questions
merci encore pour votre aide =)