Discussion :

[lipao17]

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 :

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

1
2
3
4
5
public void actionPerformed(ActionEvent e) {
 
??
 
}

quelqu'un pourrait m'aider ?

[piff62]

Code :

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

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

[jiraiya]

si tu utilises SWING utilise plutot des JMenuBar et des JMenuItem

[lipao17]

Code :

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

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

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 :

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

1
2
JButton button = new JButton("-- ajouter -->");
        button.setName("add")

et dans le actionPerformed :

Code :

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

1
2
3
4
5
if ( ((Component)e.getSource()).getName().equals("add")){
 
...
 
}

comment je fais dans le cas des Menu et MenuItem ??

si tu utilises SWING utilise plutot des JMenuBar et des JMenuItem

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

[Nicolas_75]

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

Code :

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

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

[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...

[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

[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 ?

[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

[lipao17]

oki thx nicolas !!

mon prob est résolu !!

[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

[lipao17]

[MrDuChnok]

et qu'est ce qui t'empeche de les mettre en globale dans ta classe ?

[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

[Pill_S]

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

[lipao17]

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


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

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
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) {
		...
	}
}

:

[Pill_S]

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

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 :

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

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.

[lipao17]

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


merci à tous !!!!