Question sur la synchronisation

Version imprimable

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:

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));
            }
        }
    }
}

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

Code:

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);
}

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:

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:

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

:wink: