[Débutant] reconnaitre la provenance des ActionEvent

Version imprimable

27/04/2005, 15h54
lipao17

[Débutant] reconnaitre la provenance des ActionEvent
bonjour

j'utilise pour mon application SWING un menu (MenuBar) composé d'Item (MenuItem), j'ai aussi des boutons (JButton) et d'autres choses qui ont des Listener ...

Mon problème est que je ne sais pas comment reconnaitre d'où provient la source dans mon actionPerformed
Code:

1 2 3 4 5 public void actionPerformed(ActionEvent e) { ?? }
quelqu'un pourrait m'aider ?

Code:

1
2
3
4
5
 
if(e.getSource() == lesNomsDeTesDifferentsBouttonOuAutre )
{
  actionAssosieACeBoutton();
}

27/04/2005, 15h59
jiraiya

si tu utilises SWING utilise plutot des JMenuBar et des JMenuItem
27/04/2005, 16h20
lipao17
Citation:
Envoyé par piff62

Code:

1 2 3 4 5 if(e.getSource() == lesNomsDeTesDifferentsBouttonOuAutre ) { actionAssosieACeBoutton(); }
:arrow: piff62 : si je fais ça il faut que mes bouttons ou autres soit defini en tant qu'attribut de ma classe !! et ce n'est pas le cas dans mon application. si je n'ai que des bouttons je fais :
Code:

1 2 JButton button = new JButton("-- ajouter -->"); button.setName("add")
et dans le actionPerformed :
Code:

1 2 3 4 5 if ( ((Component)e.getSource()).getName().equals("add")){ ... }
comment je fais dans le cas des Menu et MenuItem ??

Citation:

Envoyé par jiraiya

si tu utilises SWING utilise plutot des JMenuBar et des JMenuItem

:arrow: jiraiya : ok je vais essayer mais comment tu fais des "setShortcut" ???

Un exemple de JMenuItem avec listener "intégré" :

Code:

1
2
3
4
5
6
7
8
JMenuItem quitterItem = new JMenuItem("Quitter");
        quitterItem.setMnemonic(KeyEvent.VK_Q);
        quitterItem.setAccelerator(KeyStroke.getKeyStroke(KeyEvent.VK_Q, Event.CTRL_MASK));
        quitterItem.addActionListener(new ActionListener() {
            public void actionPerformed(ActionEvent ae) {
                System.exit(0);
            }
        });

Nicolas

27/04/2005, 16h35
lipao17

merci nicolas pour le raccourci clavier ça marche !!

ça marche nikel sauf que je lui ajoute un ActionListener que j'ai crée et qu'il faut que je le reconnaisse dans le actionPerformed.

cf question précédente...
27/04/2005, 17h00
Nicolas_75

Je ne suis pas sur de bien avoir compris ton problème.

Dans ton ActionListener personnel, tu peux récupérer le texte de l'élément de menu cliqué en utilisant getSource() puis getText() ou getActionCommand() (à condition pour ce dernier d'avoir fait un setActionCommand lors de la création du JMenuItem).

Nicolas
27/04/2005, 17h10
lipao17

mon problème est que je ne peut faire un getSource() puis getText() uniquement sur un JMenuItem ! non ??

getSource() renvoit un Objet.

Comment savoir si l'action provient d'un JMenuItem, d'un JButton ou autres ??

avec un instanceOf ?
27/04/2005, 17h19
Nicolas_75

Si tous tes objets étendent AbstractButton (c'est le cas pour les JButton et les JMenuItem), il suffit de caster le résultat de getSource() en AbstractButton, et de comparer getText() ou getActionCommand() avec ce que tu veux, puisque ces deux dernières méthodes existent dès AbstractButton.

Nicolas
27/04/2005, 19h07
lipao17

oki thx nicolas !!

mon prob est résolu !! :D :D
28/04/2005, 13h30
lipao17

j'ai un autre problème :

j'ai fait des "setActionCommand" pour mes boutons, pour mescheckbox et pour chaque menuItem (ce sont tous des "AbstractButton"). Mais comment récupérer ces AbstractButtons ??

il n'y a pas de methode maJFrame.getComponentByActionCommand ??

en fait j'ai un JRadioButton et un JCheckBoxMenuItem et je voudrais que quand on clique sur l'un des 2, l'autre soit aussi activé.

comment savoir si un JCheckBoxMenuItem est activé ou pas ? (je précise qu'ils ne sont pas déclaré en globale)

pouvez vous m'aider ??

merci
29/04/2005, 10h45
lipao17

:help:
29/04/2005, 11h03
MrDuChnok

et qu'est ce qui t'empeche de les mettre en globale dans ta classe ?
29/04/2005, 11h14
lipao17

j'ai beaucoup de bouton, d'itemMenu, de checkbox, ... ça ferait un peu lourd !!!

et j'ai dans ma jframe plusieurs éléments qui implements actionListener, et je voudrais appelé une méthode actionPerformed globale pour tous.

j'ai vu qu'avec : maJFrame.getJMenuBar().getMenu(0).getItem(0) je pouvais récupérer un certain item d'un menu.

mais comment faire avec les bouttons de l'application ???

merci
29/04/2005, 13h15
Pill_S

Citation:

Envoyé par lipao17

j'ai vu qu'avec : maJFrame.getJMenuBar().getMenu(0).getItem(0) je pouvais récupérer un certain item d'un menu.

Attention, mauvais code !!!

Qui te dit que ce menu existe ? Tu risque de lever pas mal de NullPointerException en faisant de cette manière. Par ex., si tu supprime un menu de ton application mais que tu oublie de changer dans les écouteurs, ça va marcher à la compilation mais tu auras une erreur à l'exécution, qui sera peut-être beaucoup plus difficile à identifier !!

Moi je me pose une question: pourquoi tu refuse d'avoir des classes qui contiennent beaucoup de champs ? Chez moi, quasiment chaque composant swing de mes JFrame possède son propre champ d'instance et sa propre méthode d'initialisation ! (sauf peut-être les JLabel complètement figés) et comme ça, même si mon GUI est assez chargé, le code reste centralisé dans une seule classe et facile à lire

Je te conseille de garder sous forme de champs de class chacun de tes menuItem & autres boutons :wink:

oki si y a pas d'autres soluces ... :(

je vais tester ce que tu dis. ça donnerai un truc comme ça :

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
public class EditorGUI extends JFrame implements ActionListener{
	private JMenuItem itemQuitter;
	private JMenuItem itemSave;
	...
 
	private static void initItemQuitter(){
		itemQuitter=new JMenuItem("Quitter");
		itemQuitter.setActionCommand("quitter");
		itemQuitter.setMnemonic('Q');
		itemQuitter.setAccelerator(KeyStroke.getKeyStroke('Q', Event.CTRL_MASK));
		itemQuitter.addActionListener(this);
	}
 
	private static void initItemSave(){
		...
	}
 
	public EditorGUI() {
		initItemQuitter();
		initItemSave();
		JMenuBar menuBar=new JMenuBar();
		setJMenuBar(menuBar);  
		JMenu menuFichier=new JMenu("Fichier");
		menuFichier.setMnemonic('F');
		menuBar.add(menuFichier);
		menuFichier.add(itemQuitter);
		...
	}
 
	public void actionPerformed(ActionEvent e) {
		...
	}
}

:?:

oui, c'est presque ça (sauf que les méthode d'init ne sont pas static) :wink:

tu peux encore améliorer en utilisant l'initialisation paresseuse, afin de n'initialiser ton composant que si cela n'a pas encore été fait, et aussi en retournant une référence au lieu de void:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 private Component monCompo=getMonCompo(); //... private Component getMonCompo(){ if(monCompo==null) { monCompo = new JPanel(); monCompo.fairequelquechose(); } return monCompo; }
et là tu te retrouve avec un code un peu plus robuste: tu as réuni en une seule méthode celle qui initialise ton composant et celle qui en retourne la référence. C'est un code que je trouve assez plaisant et subtil, que j'utilise dès que l'occasion se présente.

La dernière faille qu'il y ait, c'est dans le cas du multi-threading: si tu utilise plusieurs threads qui accèdent à ces composants, il faut synchroniser les méthodes d'initialisation.

:wink:

30/04/2005, 14h13
lipao17

Je te remercie j'ai fait comme t'as dit ! :wink:

merci à tous !!!!