[Collections] Synchroniser un iterator

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:

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:

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 :

Citation:

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 !

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

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

Code:

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

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

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.

Citation:

Envoyé par ceres02

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

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

Code:

clefsFils.remove();

a++

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:

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 ?

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