Mettre un écouteur sur chaque bouton dans une boucle

Version imprimable

28/04/2014, 18h49
BillGates62

Mettre un écouteur sur chaque bouton dans une boucle
Bonjour à tous.

Le titre n'étant pas forcément explicite, je vais détailler...

Voici la partie de mon code qui nous intéresse ici :
Code:

1 2 3 4 5 6 7 for (int i = 0; i < 9; i++) { JButton bouton = new JButton("Machin " + (i+1)); bouton.setBackground(Color.blue); bouton.setForeground(Color.white); panel.add(bouton); }
Comme vous l'aurez remarqué, ce code me permet de mettre 9 boutons dans un panel.
J'aimerais mettre un écouteur sur chaque bouton, sachant que chaque bouton amène une action différente.
Le problème est que si je mets l'écouteur dans la boucle, tous les boutons amènent à la même action, vu qu'ils ont tous le même nom,
et si je la mets après, il n'y a que le bouton créé en dernier qui contient un écouteur (ce qui est assez logique).
Donc si quelqu'un pouvait me dire comment faire, je lui en serai très reconnaissant.

P.S. : (Si c'est possible bien sûr) Je souhaite faire tout ceci avec la boucle... Je sais bien que ce serait beaucoup plus facile (pour moi en tout cas) en créant les boutons un par un ... mais ce serait aussi plus long...
28/04/2014, 20h27
adiGuba

Salut,

Je ne comprend pas trop le problème...
A quoi correspondent ces actions ?

a++
29/04/2014, 10h17
BillGates62

Citation:

Envoyé par adiGuba

Je ne comprend pas trop le problème...
A quoi correspondent ces actions ?

Bonjour et merci de t'intéresser à ma question.

Ces actions peuvent être n'importe quoi.

Exemple :
Je clique sur le bouton n°1, ça affiche un cercle.
Je clique sur le bouton n°2, ça affiche un carré.
Je clique sur le bouton n°3, ça affiche un triangle.
... etc

Et avec ceci ? Une seul écouteur pour tout les bouton, mais on détecte quel bouton on a appuyé. Il est possible aussi de mettre autant d'ecouteur que de boutons, mais il faudra faire autrement pour appeler les méthodes que tu veux qui soient executés.

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
26
ActionListener action = new ActionListener(){
			@Override
			public void actionPerformed(ActionEvent e)
			{
				String buttonname = (((JButton)e.getSource()).getName());
				if (buttonname.equalsIgnoreCase("Boutton_0"))
				{
					System.err.println("bouton 1");
				}
				if (buttonname.equalsIgnoreCase("Boutton_1"))
				{
					System.err.println("bouton 2");
				}
				if (buttonname.equalsIgnoreCase("Boutton_2"))
				{
					System.err.println("bouton 3");
				}
			}
		};
 
		for (int i = 0; i < 3; i++) {
	        JButton bouton = new JButton("Machin " + (i+1));
	        bouton.setName("Boutton_"+i);
	        contentPane.add(bouton);
	        bouton.addActionListener(action);
		}

Citation:

Envoyé par BillGates62

Ces actions peuvent être n'importe quoi.

Ben si c'est des actions différentes, il faudra bien que tu les distingues d'une manière ou d'une autre.
C'est curieux d'avoir des boutons nommées de la même manière ("Machin 1", "Machin 2", etc.) mais qui ont des actions totalement différentes.

M'enfin reste que je ne vois toujours pas le problème :
Code:

1 2 3 4 5 6 7 for (int i = 0; i < 9; i++) { JButton bouton = new JButton("Machin " + (i+1)); bouton.setBackground(Color.blue); bouton.setForeground(Color.white); bouton.addActionListener(createActionFor(i)); panel.add(bouton); }
Avec :
Code:

1 2 3 4 5 6 7 8 9 public ActionListener createActionFor(int i) { switch(i) { case 0: return new ActionListener() { ... }; case 1: return new ActionListener() { ... }; case 2: return new ActionListener() { ... }; .... } throw new IllegalStateException(); }
a++

29/04/2014, 12h23
BillGates62

Merci à tous les deux.
Les deux méthodes fonctionnent très bien pour ce que je veux.

Je sais que ma demande peut paraître "bizarre" mais je débute en Java donc j'expérimente...

Quoi qu'il en soit, je vous remercie.

Salut,

Une troisième solution serait d'utiliser les commandes (Voir JButton.setActionCommand(), ou faire des Action), genre une commande "CARRE", "CERCLE"... (utiliser des constantes, et un tableau de String pour gérer facilement l'accès dans la boucle) et de faire une seule instance d'ActionListener, et un switch sur la commande.

Exemple de principe :

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
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
...
public final static String COMMAND_CARRE = "CARRE";
public final static String COMMAND_CERCLE = "CERCLE";
...
 
public final static String[] COMMANDS = {COMMAND_CARRE, COMMAND_CERCLE, ...};
 
...
 
public void methodDeCreation....
 
...
 
ActionListeneter actionListener = createActionListener();
 
for (int i = 0; i < 9; i++) {
        JButton bouton = new JButton("Machin " + (i+1));
	bouton.setBackground(Color.blue);
	bouton.setForeground(Color.white);
        bouton.addActionListener(actionListener);
        bouton.setActionCommand(COMMANDS[i]);
	panel.add(bouton);
}
 
...
}
 
private ActionListener createActionListener() {
 
    return new ActionListener() {
 
			@Override
			public void actionPerformed(ActionEvent e) {
				switch (e.getActionCommand()) {
				case COMMAND_CARRE:
					...
					break;
                                case COMMAND_CERCLE:
                                        ...
                                        break;
				default:
					break;
				}
			}
		}
 
}

Alternative avec Action :

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
26
27
28
29
30
31
32
33
34
35
36
37
38
39
public void methodDeCreation....
 
...
 
for (int i = 0; i < 9; i++) {
        JButton bouton = new JButton(new MyAction("Machin " + (i+1),COMMANDS[i]);
	bouton.setBackground(Color.blue);
	bouton.setForeground(Color.white);
	panel.add(bouton);
}
 
...
 
	private class MyAction extends AbstractAction {
 
		public MyAction(String name, String command) {
			super(name);
			putValue(ACTION_COMMAND_KEY, command);
		}
 
		@Override
		public void actionPerformed(ActionEvent e) { 
			 switch (e.getActionCommand()) {
				switch (e.getActionCommand()) {
				case COMMAND_CARRE:
					...
					break;
                                case COMMAND_CERCLE:
                                        ...
                                        break;
				default:
					break;
				}
		} 
 
	}
 
 
}