[Collection] ConcurrentModificationException

Version imprimable

18/05/2004, 15h34
calvin

[Collection] ConcurrentModificationException

Voila j'ai un probleme d'exception en fait je l'ai une fois sur deux ...la premiere iteration marche mais l exception est generee ensuite, je ne sais pas comment la virer...Elle intervient lorsque j'ai une array list d'ecouteur d'evenement que je parcour avec un iterator afin de dispatcher les messages ( en fait il y a un seul ecouteur ) l'action effectué par cet ecouteur est un update ui.. a la premiere iteration pas de pb tout se passe bien lors de la deuxieme il ya une exception de type

Code:

java.util.ConcurrentModificationException

qui est generée. J'ai lu , dans la java doc par ex, que cet evenement etait generer lorsqu'on faisait une modification d'un item qui se trouve dans un iterator mais la je ne veux que mettre a jour mon affichage si vous avez une solution je vous remercie pour votre aide.. :?:
18/05/2004, 15h40
nmujdzic

peux-tu envoyer un bout de code???
18/05/2004, 15h47
narfiggo

Tu dois faire 2 acces concurrents sur tes collections.
Si tu parcours ta collection et en meme temps tu lui rajoute, supprime ou modifie des elements ca pose probleme...
Ce que tu peux faire c'est créer une copie de ta collection pour la parcourir et faire les changement dans ta veritable collection.
Mais sans bout de code c'est chaud de te donner une solution :wink:
18/05/2004, 15h49
calvin

ben c'aurait ete avec grand plaisir mais je sais pas si ca va etre tres utile parceque c'est un probleme assez globale donc il va falloir que je fasse une selection de code et je sais pas si cela sera suffisant mais je vais essayer... en tout cas merci de t'interesser a mon pb.. :D

je me rend bien compte que c difficile sans code je vais essayer alors d'etre le plus clair possible dans ma selection de code
voila dans un premier temps j'appelle cette methode depuis l'objet qui recoit l'evenement et qui se trouve dans l'objet generateur d'evenement

Code:

1
2
3
4
5
6
7
8
9
10
11
12
public void addNoeud(TreePath loca,Itec nouveau){
   try{
     Itec temp = (Itec) loca.getLastPathComponent();
     temp.addModule(nouveau);
     this.data2=this.rootItec.getNivo(nivo);
     fireTableChanged();
     fireTreeChanged(loca);
   }catch(Exception e){
     System.out.println("exception"+e);
   }
// fireTreeChanged(loca);
 }

il appelle la methode fireTreeChanged

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
public  void  fireTreeChanged(TreePath loca){
   try {
     List malist=Collections.synchronizedList(treeModelListeners);
     synchronized(malist){
       ListIterator iterator = malist.listIterator();
       Object tmp;
       while (iterator.hasNext()) {
         tmp = iterator.next();
         if (tmp instanceof TreeModelListener) {
           //TreeModelListener listener = (TreeModelListener) iterator.next();
           TreeModelListener listener = (TreeModelListener) tmp;
           TreeModelEvent tme = new TreeModelEvent(this, loca);
           //listener.treeNodesChanged(new TreeModelEvent(this, loca));
           listener.treeNodesChanged(tme);
         }
         else {
           // System.out.println("ERREUR DE CAST ln 94 ModelPrincipal");
         }
       }
     }
   } catch (Exception ex) {
     ex.printStackTrace();
   }

a partir de cette methode on retourne dans l'objet qui avait invoqué la methode add noeud et qui se trouve dans la liste des listener et on realise l'operation suivante

Code:

1
2
3
4
5
6
7
8
public void Insert_ActionPerformed(TreeModelEvent e){
try{
      this.updateUI();
    }catch(Exception j){
      j.printStackTrace();
    }
 
  }

Voila j'espere avoir été clair mais j'en suis pas sur :?

18/05/2004, 16h02
bulbo
La classe Vector etant synchronisee, elle pourrait peut-etre regler ton probleme sans que tu ai besoin de changer quoi que ce soit ...

Sinon tu peux essayer de faire:
Code:

1 2 List liste = Collections.synchronizedList(tonArrayList);
Et utiliser l'objet liste pour parcourir ton ArrayList ..

Bulbo :wink:
18/05/2004, 16h19
calvin

Ben Vector j'ai deja essayer ca marcher pas c pour ca que je suis passer sur une ArrayList en plus ce qui me gene un peu avec Vector c'est que selon la java doc elle va etre deprecated....merci pour ta reponse en tout cas. Mais si j'ai synchroniser ma liste ca devrait etre bon non?

Code:

1 2 3 4 5 6 while (iterator.hasNext()) { tmp = iterator.next(); if (tmp instanceof TreeModelListener) { TreeModelListener listener = (TreeModelListener) tmp; TreeModelEvent tme = new TreeModelEvent(this, loca); listener.treeNodesChanged(tme);
Je pense que le probleme vient du fait que tu fais treeNodesChanged(tme) en meme temps que tu parcours ta liste de listener... Quand tu fais un treeNodesChanged tout les listener sur cette liste sont "appelé" par java donc ca provoque un acces concurrent (enfin je crois :wink: ).
Si tu peux sortir de ton while avant d'appeler treeNodesChanged, devrait plus y avoir de probleme.
Si tu peux pas, fais ton parcours sur une copie de maList, comme ca tu travaille sur une liste séparée; apres pour l'appel de treeNodesChanged il faut que tu recupere le bon element (maList.get(position du treeModelListener) et tu lui applique le nodesChanged.

Ca a l'air assez chiant mais bon... :D

18/05/2004, 16h36
calvin

merci narfigoo c'est sympa de ta part je vais essayer, mais c'est bizarre qd mem non? c pourtant bien le principe des listener ca devrait marcherje l'ai deja fait plusieurs fois sans pb sinon comment je peux faire une copie de ma liste? :D

Essaye a peu pres ca : (il se peut u'il y est qques erreurs mais l'idée y est :lol: )

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
public  void  fireTreeChanged(TreePath loca){ 
   try { 
     List malist=Collections.synchronizedList(treeModelListeners);
     List malistCopie = new ArrayList();
     malistCopie.addAll(maList);
     synchronized(malistCopie){ 
       ListIterator iterator = malistCopie.listIterator(); 
       Object tmp; 
       while (iterator.hasNext()) { 
         tmp = iterator.next(); 
         if (tmp instanceof TreeModelListener) {
           //tu recup le listener de la vrai list            
           TreeModelListener listener = (TreeModelListener) (maList.get(iterator.previousIndex() + 1)); 
           TreeModelEvent tme = new TreeModelEvent(this, loca); 
           listener.treeNodesChanged(tme); 
         } 
         else { 
           // System.out.println("ERREUR DE CAST ln 94 ModelPrincipal"); 
         } 
       } 
     } 
   } catch (Exception ex) { 
     ex.printStackTrace(); 
   }

Un truc dans ce genre :wink:

19/05/2004, 09h00
calvin

ok ca marche merci beaucoup :lol: