Discussion :

[Xavier76]

Bonjour à tous !

J'ai un petit soucis dans mon programme ... Je vous joins directement un programme très simplifié avec en commentaire le problème que j'ai

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
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
 
// import ...
public class test
 
{
 
	public static void main(String []args)
 
	{
		cadre fenetre = new cadre("Fenetre",150,150,1365,718);
	}
 
}
 
 
 
class cadre extends JFrame
{
	public cadre(String nom,int X,int Y,int H,int L)               
 
	{
 
		super(nom);
		setBounds(X,Y,H,L);
		setDefaultCloseOperation(EXIT_ON_CLOSE);            
		getContentPane().add(new PanneauCadre());
		setVisible(true);
 
	}
}
 
class PanneauCadre extends JPanel
{
       private JRadioButton a1 = new JRadioButton("choix1");
       private JRadioButton a2 = new JRadioButton("choix2");
       private JRadioButton a3 = new JRadioButton("choix3");
       public int choix = 0;  // declaration d'une variable entiere du choix
       // On les réunit dans un buttonGroup, je le fais pas ici pour supprimer des lignes de codes :)
 
      public PanneauCadre()
      {
               a1.addItemListener(new Ecouteur1());
               a2.addItemListener(new Ecouteur2());
               a3.addItemListener(new Ecouteur3());
      }
 
       public class Ecouteur1 implements ItemListener
 
	{
		public void itemStateChanged(ItemEvent evt)
 
		{
 
			evt.getItem();
    		        if(evt.getStateChange() == ItemEvent.SELECTED)
    		     	       choix = 1;
		}
 
	}
 
	public class Ecouteur2 implements ItemListener
 
	{
		public void itemStateChanged(ItemEvent evt)
 
		{
 
			evt.getItem();
    		        if(evt.getStateChange() == ItemEvent.SELECTED)
    		     	choix = 2;
 
		}
	}
 
	public class Ecouteur3 implements ItemListener
 
	{
		public void itemStateChanged(ItemEvent evt)
 
		{
 
			evt.getItem();
    		        if(evt.getStateChange() == ItemEvent.SELECTED)
    		     	choix = 3;
 
		}
    // Dans les classes évènements choix prend bien sa valeur
	}
 
       public int getChoix()     // une méthode qui retourne la valeur du choix si c'est 1,2, ou 3 dans une autre classe qui n'est pas ici.
	{      
                if(choix!=0)
		return choix;    // Le problème est qu'à cet endroit choix est toujours égal à 0
	}
}

J'ai mis mes petits commentaires dans le programme, là ou ça ne marche pas ...
Il faudrait que lorsque que l'on modifie cette variable, cela s'applique à toutes les classes.

SI vous avez des idées, je suis preneur !

Merci d'avance !

[fraco]

peut-être déclarer choix dans cadre et pas dans panneaucadre ?

[rtg57]

Bonjour,

essayez peut être de déclarer la variable choix en public static int

@ bientôt ...

[dinobogan]

Je ne saisis pas bien le problème. Est-ce la récupération de "choix" par un autre objet qui pose problème ? Ou alors la variable "choix" qui ne change jamais de valeur en dehors des listener ?

[Xavier76]

Hélas, passer en static ne change pas le problème ni de le déclarer dans la classe cadre ...
Disons que le programme que j'ai mis là est très simplifié.
Je vais essayer de vous résumer la situation.
- On dispose d'une fenetre : cadre extends JFrame.
- Dedans j'ai plusieurs panneaux organisés avec un cardLayout : PanneauPrincipal, PanneauOption, PanneauJouer
-Dans le PanneauOption, il y a, entre autre, 3 JRadioButton contenus dans un ButtonGroup, indiquand trois choix de difficultés différents.
-En plaçant un événement sur ces 3 JRadioButton, j'aimerai que cette variable entière choix, issus du résultat de l'événement, prenne la valeur 1,2 ou 3 en fonction du niveau choisi.
-Cette variable est ensuite retournée dans la classe cadre où en fonction du choix, elle affiche le panneau avec le niveau souhaité. Soit PanneauJouer1 / PanneauJouer2 ou PanneauJouer3.

J'ai donc quelque chose comme ca :
Classe PanneauOption :

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
 
public int choixDifficulte;
 
	public class niv1selected implements ItemListener
	{
		public void itemStateChanged(ItemEvent evt)
		{
			evt.getItem();
    		     if(evt.getStateChange() == ItemEvent.SELECTED)
    		     	choixDifficulte = 1;
		}
	}
 
	public class niv2selected implements ItemListener
	{
		public void itemStateChanged(ItemEvent evt)
		{
			evt.getItem();
    		     if(evt.getStateChange() == ItemEvent.SELECTED)
    		     	choixDifficulte = 2;
    		}
	}
 
	public class niv3selected implements ItemListener
	{
		public void itemStateChanged(ItemEvent evt)
		{
			evt.getItem();
    		     if(evt.getStateChange() == ItemEvent.SELECTED)
    		     	choixDifficulte = 3;
		}
	}
 
 
	public int getChoixDifficulte()
	{				
	   	return choixDifficulte;   // Le problème est ici, choixDifficulte est toujours égal à 0 ...
	}

Et dans ma classe cadre j'aurai ceci :

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
 
Pp.getJouer().addActionListener(new ecout3());    // Pp est le PanneauPrincipal et jouer le bouton dessus
private int choixDifficulte = Po.getChoixDifficulte();
 
class ecout3 implements ActionListener
	{									
	    @Override
	    public void actionPerformed(ActionEvent arg0)
	    {
			if(choixDifficulte == 1)
				cardLayout.show(Pane, "PanneauJouer1");
 
			if(choixDifficulte == 2)
				cardLayout.show(Pane, "PanneauJouer2");
			if(choixDifficulte == 3)
 
				cardLayout.show(Pane, "PanneauJouer3");
	    }
	}

J'espère que vous m'avez mieux compris, et merci encore pour votre aide ! =)

[rtg57]

Bonjour,

petite question bête: avez-vous regarder en mode débogage, si le programme passe bien par les lignes de code modifiant l'état de la variable choixDifficulte ?

@ bientôt...

[dinobogan]

Effectivement, ça ne peut pas fonctionner puisque tu fige "choixDifficulte" lorsque tu fais :

Code :

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

private int choixDifficulte = Po.getChoixDifficulte();

Tu dois revoir ton architecture.
De plus, il est dommage de créer une classe par listener.
Autant utiliser une seule classe pour le listener qui va gérer un groupe de JRadioButton. Tu ranges tes JRadioButton dans un tableau, et le listener modifie la variable "choixDifficulte" avec l'indice du JRadioButton dans le tableau. Tu as un code plus simple et le nombre de niveaux n'est pas figé.
En plus, tu simplifies le code.