Discussion :

[Pépé Lélé]

Bonjour,

J'aimerais savoir pourquoi, dans cet exemple donné, le modificateur "synchronized" ne permet pas d'afficher entièrement "BONJOUR" puis "AU REVOIR" ? Que faudrait-il changer dans ce code pour que cela soit le cas ?

Merci pour vos réponses.

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
24
25
class Main {
    public static void main(String[] arg) {
        new Afficher("BONJOUR").start();
        new Afficher("AU REVOIR").start();
    }
 
}
 
class Afficher extends Thread {
    String txt;
 
    public Afficher (String t) {
        txt=t;
    }
 
    synchronized public void run() {
        for (int j=0;j<2;j++) {
            for (int i=0;i<txt.length();i++) {
                try { sleep(100); }
                catch (InterruptedException e) {};
                System.out.print(txt.charAt(i));
            }
        }
    }
}

[herve91]

A mon avis, il faudrait synchroniser sur une autre méthode que run(). Par exemple :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
public void run() { 
  for (int j=0;j<2;j++) { 
    for (int i=0;i<txt.length();i++) { 
      try { sleep(100); } 
      catch (InterruptedException e) {};
      synchronizedPrint(txt.charAt(i));
    } 
  } 
} 
 
synchronized private void synchronizedPrint(char c) {
  System.out.print(c);
}

[Pill_S]

c'est impossible dans ton cas, car la synchronisation agit sur les objets (c'est un verrou)

quand tu met le "synchronized" sur une déclaration de méthode, le verrou utilisé est "this" (sauf les méthodes static), ce qui indique qu'un seul thread à la fois peut invoquer cette méthode pour l'objet courant

dans ton cas, tu crée 2 instances de "Afficher", donc les verrous n'agissent pas sur les mêmes objets, donc ils ne se bloquent pas mutuellement

De plus, la méthode run ne devrait normallement pas être marquée "synchronized". Si tu as besoin de la synchroniser, il faut le faire à l'intérieur du corps

Code :

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

1
2
3
4
5
6
 
public void run(){
  synchronized(this){
    // ...
  }
}

il ne faut pas confondre "appeler une méthode depuis 2 threads différents sur le même objet" et "appeler une méthode sur 2 objets différents depuis des threads quelconques"

Une solution pourrait être de faire comme ceci (mais c pas bo !):

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
24
25
 
class Afficher extends Thread {
  Afficher(String mess, Object lock) {
    this.mess = mess;
    this.lock = lock;
  }
 
  public void run(){
    synchronized(lock) {
      for(int i = 0; i<mess.length(); i++){
        System.out.print(mess.charAt(i));
 
        try{ sleep(100); } catch (Throwable t) {;}
      }
    }
 
  }
 
 
  public static void main (String[] arg) {
    Object lock = new Object();
    new Afficher("BONJOUR", lock).start();
    new Afficher("ADIEU", lock).start();    
  }
}

j'ai pas compilé mais ça doit tourner