Thread : mot clé synchronized

Bonjour !

Je n'arrive pas à comprendre pourquoi ce bout de code n'est pas juste dans un environnement multi-thread, pouvez-vous m'éclairer ?

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19

public class Cpt implements Runnable {   private Thread t; private static int compteur = 0;   public Cpt(){ t = new Thread(this); }   public void run(){ synchronized(this) cpt++; System.out.println(cpt); }     }

---

On ne va pas deviner ce que devrait faire ce bout de code.
Le Thread t c'est un peu n'importe quoi, et le compteur on ne le voit modifier qu'une fois.

Mais bon, je vois que le compteur est static, donc l'idée est probablement d'avoir plusieurs objets Cpt, et que chacun soit utilisé dans des threads différents.

Dans ce cas c'est simple, pour écrire dans une variable partagée entre threads, il faut synchroniser l'accès à cette variable, à l'aide d'un objet partagé entre tous les threads.
Or là tu utilises this pour synchroniser. Donc seuls les threads qui utilisent ce même objet Cpt sont synchronisés. Ceux qui en utilisent un autre ne sont pas concernés.
Il aurait fallu synchroniser sur Cpt.class au lieu de this, ou passer par une méthode synchronized static (ce qui fait la même chose.)

Oh oui c'est vrai que mon exemple est incomplet :oops:

Merci pour cette explication :)