Discussion :

[brahimouledriah]

Bonjour a tous;
j'ai fait un petit programme pour mettre en pratqiue l'interblicage entre les Thread, mais ca na pas marché, je vous montre mon programme , si quelqu'un sait ou est le probleme merci de me le signaler
le programme normalement doit afficher alternativement A B A B A B....

merci davance

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
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
class Jeu{
 int tour=1;
		public synchronized void jouer1() {
		                        if(tour==2){
		                                   try{ wait();}
					                        catch(InterruptedException e){}
		                        }
		System.out.print("A");
		 tour=2;
 
		 notify();
		 }
       public   synchronized void jouer2() {
		                        if(tour==1){
		                                 try{ wait();}
					                       catch(InterruptedException e2 ){}  
		                        }
		 if (tour==2){System.out.print("B");
		 tour=1;
 
		 notify();
		 }
		 }
}
public class Joueur extends Thread {
        char x;
 
		 Joueur(char x){this.x=x;}
		 public static void main(String[] args){
		         Joueur j1= new Joueur('A');
				 j1.start();
				 Joueur j2= new Joueur('B');
				 j2.start();
 
	     }
		 public void run(){while(true){
 
		 if(x=='A'){
            		 new Jeu().jouer1();} 
 
		 if(x=='B'){
		           new Jeu().jouer2(); }
				   }
 
		 }
}

[KahRiboo]

Salut !

Et bien tu as plusieurs probleme ...

Tout d'abord tu methodes synchronized jouer1 et jouer2 ... sont par definition en exclusion mutuelle sur l'objet "this" ...

Or tu les apelle dans tes threads ainsi :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
	public void run(){
 
		while(true){
 
			if(x=='A'){
				new Jeu().jouer1();} 
 
			if(x=='B'){
				new Jeu().jouer2(); }
		}
	}

il faut les appeler ainsi plutot (pour qu'elle soint synchronisée elle doivent utiliser la même instance (par exemple en static ou en la construisant dans le main)

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
	static Jeu j = new Jeu() ;
	public void run(){
 
		while(true){
 
			if(x=='A'){
				j.jouer1();} 
 
			if(x=='B'){
				j.jouer2(); }
		}
	}

Ensuite tu verra ton programme fonctionner et afficher un unique 'A' ... explications :
- L'appel joueur1 se lance... c'est a son tour il affiche 'A' et se met en attente...
- L'appel joueur2 se lance... or il est synchronisé avec joueur 1 donc tombe en attente que joueur1 libere la place (alors qu'en fait joueur1 est en wait)

Donc il va encore falloir rendre ton synchronised moins selectif
exemple (enlever les wait.... un wait dans un synchronized je prefere eviter)

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
	public synchronized void jouer1() {
		if (tour == 1) {
			System.out.println("A") ;
			tour = 2 ;
		}
	}
 
	public synchronized void jouer2() {
		if (tour == 2) {
			System.out.println("B") ;
			tour = 1 ;
		}
	}

Au final voici tes deux classes (un peu nettoyées)

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
26
27
 
public class Joueur extends Thread {
	private char x;
	private Jeu j ;
 
	public Joueur(char x, Jeu j){
		this.x=x;	
		this.j=j;
	}
 
	public static void main(String[] args){
		Jeu j = new Jeu() ;
		Joueur j1= new Joueur('A', j);
		Joueur j2= new Joueur('B', j);
		j1.start();
		j2.start();
	}
 
	public void run(){
		while(true){
			switch (x) {
				case 'A': j.jouer1(); break ;
				case 'B': j.jouer2(); break ;
			} 
		}
	}
}

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
 
class Jeu{
 
	int tour=1;
 
	public synchronized void jouer1() {
		if (tour == 1) {
			System.out.println("A") ;
			tour = 2 ;
		}
	}
 
	public synchronized void jouer2() {
		if (tour == 2) {
			System.out.println("B") ;
			tour = 1 ;
		}
	}
}

Voila j'espere avoir pu t'aider !
K.