Discussion :

[mimi2403]

salut
moi j'ai un probleme lors de l'exécution de mon application (sous jbuilder9), j'utilise des threads et pdt l'éxécution j'ai une exception qui est affiché

java.lang.IllegalMonitorStateException: current thread not owner
at java.lang.Object.notify(Native Method)

malgrés que j'ai utilisé la methode synchronized

Code :

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

1
2
3
4
5
6
7
8
synchronized (this){
try {this.wait();}
catch (InterruptedException ex4){}
}
synchronized (this){
try {this.notify();}
catch (InterruptedException ex4){}
}

merci d'avance pour votre aide

[tchize_]

on peux voir le vrai code? Ce message s'affiche quadn tu fait un wait ou notify sans être le owner de l'objet sur lequel tu le fait (via synchronized)

[mimi2403]

oui bien sur voici mon code

Code :

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

1
2
3
4
5
6
  Frame2.fin+=1;
            Frame2.file_global[Frame2.fin]=thisreq;
            synchronized(Frame2.file_global[Frame2.fin]){
              try{Frame2.file_global[Frame2.fin].wait();}
              catch(InterruptedException ex4){}
            }

la file globale c'est pour bloquer les requetes (les requetes sont des threads)

[tchize_]

si, comme je l'ai bien compris, ce code est exécuté par plusieurs threads en parallèle, ton problème est que tu ne fait pas de synchronized sur un lock global avant de tripatouiller ta liste. Résultat, entre ces deux lignes

synchronized(Frame2.file_global[Frame2.fin]){
try{Frame2.file_global[Frame2.fin].wait();}

Frame2.fin peut avoit changé et tu fait donc le wait sur un objet différent de celui du synchronized.

[mimi2403]

j'ai pas su où je dois placé exactement synchronized();
pouvez vs m'expliquer plus S.V.P c'est vraiment urgent

[tchize_]

affiche ton code dans ton ensemble, notement avec les différents thread impliqués

[mimi2403]

mon prg est trop long, je ne eut pas l'afficher ds l'ensemble
mais j'ai une question est ce que on peut remplacer wait() par suspend() et replacer notify () par resume () ????????

[tchize_]

suspend et resume sont dépréciés, donc à ne pas utiliser. Comme je l'ai dit, ton problème, c'est que ton tableau est suceptible de changer au cours de l'opération (c'est à dire entre le synchronized et le wait) et donc tu fait ton wait sur un objet autre que celui pour lequel tu fait un synchronized. Sans plus de code, difficile d'en avoir la certitude. un solution serait de récupérer l'objet de ton tableau à part, par exemple:

Code :

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

1
2
3
4
5
6
7
8
9
 
synchronized(Frame2.file_global){  // si plusieurs threads manipulent un objet, toujours synchroniser ses modifications!
  Frame2.fin+=1;
  Frame2.file_global[Frame2.fin]=thisreq;
}
synchronized(thisreq){ 
              try{thisreq.wait();}
              catch(InterruptedException ex4){}
            }

Note: ca suppose que thisreq est bien une varible locale!