Discussion :

[chabfive]

Salut,

J'ai une classe A qui étend Runnable.
Cette classe A créée deux threads, un thread B et un C.

les threads B et C démarrent en même temps.
Mais le thread B ce met en attente (wait) dès le début, et repart quand le thread C a terminé.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
 
class A extends Runnable {
 
  private B b;
  private C c;
 
  public void run(){
     b.start();
     c.start();
  }
 
  class B extends Thread {
    public void run(){
      synchronized(...){
        try{
          wait();
        }catch(...){}
      }
      /*plein de choses*/
    }
  }//class B
 
}//class A

Code :

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

1
2
3
4
5
6
7
8
9
 
class C extends Thread {
  public void run(){
    /*plein de choses*/
    synchronized(...){
       notify();
    }
  }
}//class C

Mon problème est que je ne sais pas quoi mettre dans les synchronized.
J'ai essayé plein de truc mais ca n'a pas marché.

Merci d'avance pour votre aide.

[professeur shadoko]

c.join() ?

[®om]

Tu ne fais pas wait() et notify() sur le même objet, ça ne risque pas de les faire communiquer entre eux...

[chabfive]

J'ai déjà essayé de les synchroniser sur un même objet.
Je l'ai fais sur un Vecteur qu'ils utilisent tous les deux et aussi sur leur classe mais ça ne marche pas.

[®om]

dans A par exemple, tu déclares une variable:

Code :

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

public static final Object lock = new Object();

ensuite, pour faire wait(), tu fais A.lock.wait(); et notify() tu fais A.lock.notify();
Et pour ce que tu mets dans le synchronized: synchronized(A.lock)

PS: dans ton code:

Code :

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

class A extends Runnable

c'est plutôt:

Code :

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

class A implements Runnable

[chabfive]

Je ne comprend pas, je fais ce que tu dis mais ca ne marche pas.

Est ce vraiment nécessaire que la variable partagée soit déclarée static final.

[®om]

Je ne comprend pas, je fais ce que tu dis mais ca ne marche pas.

Qu'est-ce qui ne marche pas?

Est ce vraiment nécessaire que la variable partagée soit déclarée static final.

final, de préférence car le verrou ne doit jamais être modifié ; static, dans ce cas oui pour que les 2 threads aient accès au même verrou...

[®om]

PS: j'ai écrit un tuto sur la synchronisation en java, ça pourrait t'aider:
http://rom.developpez.com/java-synchronisation/

[chabfive]

ce qui ne marche pas, c'est que le thread B qui attend la fin du thread C ne se réveille pas.

[®om]

Ca n'est pas étonnant, tu n'as aucune garantie que B s'effectue avant C, et donc aussi bien C fait notify() (ce qui ne sert à rien personne n'attend) et ensuite B fait wait(), donc bah il attend et personne ne le réveille...

Et encore, ça c'est dans le cas où tu utilises correctement la synchronisation (synchronized(...), wait et notify sur le même objet)...