Discussion :

[Linquisiteur]

Bonjour, je commence les thread en Java et je bloque sur un problème entre wait et notifyAll(). Mon programme possède un wait() qui attend dans une boucle un notifyAll() mais lorsque le notifyAll s'active les autres thread rèste bloquer sur le wait()...

Merci d'avance

[joel.drigo]

Salut,

L'appel de notifyAll() sur une instance envoit un signal sur cette instance : tous les threads qui sont en attente dans un wait() sur cette même instance reçoive le signal, et arrête d'attendre. Dans ton code, le thread fait un wait sur lui-même, dans une boucle, et après la fin de la boucle, se notifie lui-même : autant dire qu'il rentre dans la première itération et n'en sort jamais, mais quand bien même, il se notifirait lui-même, ce qui ne servirait à rien, puisqu'il n'est plus en attente.

En fait, il faut que l'instance sur laquelle tu fais le wait soit une seule instance commune , accessible par les autres threads, pour qu'ils puissent appeler notify() sur cette instance ( si notifyAll() est appelé, tous les threads en wait sur l'instance se "réveillent").

[Linquisiteur]

Salut et merci

excuse moi j'ai du mal à comprendre, tu veux dire que je ne peux pas faire de notifyAll() dans la même instance courante ?
J'aimerais synchroniser mes thread et je ne possède que x fois ce thread qui sont tous indépendant et doivent s'appeler en boucle dans un certain ordre

Edit: est-il possible de notifier 1 seul thread précisément (avec notify) sans que cela ne soit aléatoire ou fixer sur la première instance..?

[tchize_]

Ce n'est pas un thread que l'on notifie, mais un sémaphore. En faisaint this.notifyAll et this.wait, tu utilise le threadCourant comme sémaphore. C'est bien, mais comme les autres threads s'utilisent eux même comme sémaphore, ça ne risque pas d'aller bien loin. T'as plein de sémaphores différents et pas deux threads qui utilisent le même.


Avec notify, tu ne peux pas choisir le thread qui se réveille. Ca peut se mettre en place de manière détournée: mettre dans une variable commune le nom du thread qui dois travailler, tous les autres se rendormant immédiatement. Mais du coup, il faut synchroniser aussi l'appel à cette variable et si tu ne comprends pas déjà les base de wait/notify, tu codera ça n'importe comment et ce sera la catastrophe


Par contre, comme chaque thread fait un wait() sur lui même (this.wait()) dans ton cas, il suffit que tu fasse un leThreadAReveiller.notify().

[Linquisiteur]

Salut tchize et merci de ta réponse.

Il faudrait donc que chaque thread ai la liste de tout les autres thread ?
par exemple public static Thread listeThread[nbThread] avec le tableau de thread du main

et donc que je face :

Code :

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

1
2
3
4
5
 
   private synchronized void donneLaMain() {
	   tour = (tour%nbThreads)+1;
           listeThread[tour].notify();
   }

Est-ce que j'ai bien comprit?

[joel.drigo]

C'est ça l'idée pour la sélection du thread, mais il faut synchroniser sur l'instance que tu notifies (comme il faut synchroniser sur l'instance sur laquelle on appelle wait() le cas échéant).

soit

Code :

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

1
2
3
4
5
6
7
 
private synchronized void donneLaMain() {
   tour = (tour%nbThreads)+1;
   synchronized ( listeThread[tour] ) {
        listeThread[tour].notify();
   }
}