Discussion :

[maurojeff]

Bonjour,

Je suis occupé à faire des tests sur le mot clé synchronized.

Le mot clé synchronized, si j'ai bien compris, permet de s'assurer qu'un seul thread à la fois n'exécute du code se trouvant dans un bloc synchronized ou dans une méthode synchronized.

Je dispose d'un exemple et suite à des éxécutions successives du même code, les résultats obtenus sont différents.

La classe implementant Runnable :

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
 
public class Task implements Runnable {
 
	private Counter counter;
 
	public Task(Counter counter) {		
		this.counter = counter;
	}
 
	@Override
	public void run() {
		for (Long i = 0L; i < 5; i++) {						
 
				synchronized (counter) {
					counter.add(i);				
				}			
		}
	}
 
}

La classe bean :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
public class Counter {
	private Long count = 0L;
	private Long secondCount = 0L;
 
	public void add(Long value){		 
			count += value;
			System.out.println("The value of count is : " + count + " on " + Thread.currentThread().getName());				
	}
 
 
 
}

La classe de test:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
public class Test {
	public static void main(String args[]) {
 
		Counter counter = new Counter();
 
		Task threadA = new Task(counter);
		Task threadB = new Task(counter);
 
		new Thread(threadA).start();
		new Thread(threadB).start();
	}
}

Les résultats sont aléatoires, alors que les 2 threads devraient s'éxécuter les un après les autres.

Résultat 1:

The value of count is : 0 on Thread-0
The value of count is : 1 on Thread-0
The value of count is : 3 on Thread-0
The value of count is : 3 on Thread-1
The value of count is : 4 on Thread-1
The value of count is : 6 on Thread-1
The value of count is : 9 on Thread-1
The value of count is : 13 on Thread-1
The value of count is : 16 on Thread-0
The value of count is : 20 on Thread-0

Résulat 2 : (résultat désiré)

The value of count is : 0 on Thread-0
The value of count is : 1 on Thread-0
The value of count is : 3 on Thread-0
The value of count is : 6 on Thread-0
The value of count is : 10 on Thread-0
The value of count is : 10 on Thread-1
The value of count is : 11 on Thread-1
The value of count is : 13 on Thread-1
The value of count is : 16 on Thread-1
The value of count is : 20 on Thread-1

Merci d'avance pour vos réponses.

[Rei Ichido]

Je ne vois pas ce qui te pose problème !

Quand tu utilises synchronized, tu places un lock sur l'objet ... Mais étant donné que tu le fais au sein d'une boucle, tout ce que tu garantis avec ton code c'est qu'il n'y aura pas deux exécutions simultanées de add sur ton counter. L'ordre d'exécution des threads n'est en rien déterminé. Il pourrait l'être si tu synchronisais sur ton counter avant d'entrer dans la boucle : le 1er thread prendrait le lock, et du coup le 2nd thread serait forcé d'attendre que le 1er en ait fini.

[fr1man]

Les résultats sont aléatoires, alors que les 2 threads devraient s'éxécuter les un après les autres.

Non, tes deux threads vont s'exécuter en parallèle.

[OButterlin]

Si la liste est la bonne, il y a quand même un problème, on retrouve 2x la valeur 3 sur 2 thread

[Rei Ichido]

C'est parce que la variable de boucle i est initialisée à 0, donc le 1er passage par add dans un Thread garde la valeur constante.