Discussion :

[Jb_One73]

Bonjour à tous !

Je me joins à vous car là je colle et je n'arrive pas à trouver la solution.

Explication :
J'ai plusieurs menus dans lesquels je peux via la commande "add" qui est un sous-menu de ceux-ci ajouter un nouveau sous-menu ... je sais pas si j'ai réussi à être claire ?
Je vous joins une image qui représente mon interface et mes différents menus et dessous celle-ci mon code.

Le problème :
Dès que j'ajoute via la boite de dialogue un cuisinier, il m'est impossible ré-invoquer celle-ci via le bouton "add" ... Je ne comprend pas pourquoi ?

En complément :
Lorsque je clique sur l'un des sous-menu (un cuisinier) celui-ci doit afficher une boite de dialogue me demandant si je veux le supprimer et si oui, je le retire du menu en question.

Je vous remercie d'avance à tous !




Mon code :

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
 
private void createAndPlaceComponents() {
		menuBar = new JMenuBar(); 
 
		/** différents menus */
		menuCuisinier = new JMenu("Cuisinier");
 
		/** MENU CUISINIER */
		addCuisinier = new JMenuItem("add");
		addCuisinier.addActionListener(this);
		menuCuisinier.add(addCuisinier);		
		menuCuisinier.addSeparator();
 
		/** Ajouter les menu sur la bar de menu */
		menuBar.add(menuCuisinier);
 
		/** Ajout de la barre de menu dans la frame */
		frame.setJMenuBar(menuBar);
}
 
public void actionPerformed (ActionEvent e) {
                /** AJOUT D'UN CUISINIER */
		if(e.getSource() == addCuisinier && e.getActionCommand() == "add"){
 
		    String NomCuisinier = JOptionPane.showInputDialog(null, "Entrer le nom du cuisinier", "");
 
		    if( NomCuisinier != null){
 
		    	addCuisinier = new JMenuItem(NomCuisinier);
		    	addCuisinier.addActionListener(this);
				menuCuisinier.add(addCuisinier);				
				textArea.append(NomCuisinier + " prend son service !" + "\n");
		    }
		}
}

[tchize_]

Code :

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

1
2
3
4
5
6
7
		if(e.getSource() == addCuisinier && e.getActionCommand() == "add"){
 
		    String NomCuisinier = JOptionPane.showInputDialog(null, "Entrer le nom du cuisinier", "");
 
		    if( NomCuisinier != null){
 
		    	addCuisinier = new JMenuItem(NomCuisinier);

Tu teste que la source est égale à addCuisinier, mais plus loin dans le code tu remplace addCuisinier par le nouveau cuisinier => Le test ne marchera plus.


Autre remarque, évite d'avoir un actionListener = this qui fais tout le boulot, ça va vite être complexe à réaliser. Tu peux faire plus simple comme 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
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
private void createAndPlaceComponents() {
		menuBar = new JMenuBar(); 
 
		/** différents menus */
		menuCuisinier = new JMenu("Cuisinier");
 
		/** MENU CUISINIER */
		addCuisinier = new JMenuItem("add");
		addCuisinier.addActionListener((e) -> ajouterUnCuisinier(e));
		menuCuisinier.add(addCuisinier);		
		menuCuisinier.addSeparator();
 
		/** Ajouter les menu sur la bar de menu */
		menuBar.add(menuCuisinier);
 
		/** Ajout de la barre de menu dans la frame */
		frame.setJMenuBar(menuBar);
}
 
public void ajouterUnCuisinier (ActionEvent e) {
 
		    String NomCuisinier = JOptionPane.showInputDialog(null, "Entrer le nom du cuisinier", "");
 
		    if( NomCuisinier != null){
 
		    	addCuisinier = new JMenuItem(NomCuisinier);
		    	addCuisinier.addActionListener((e)-> supprimerCuisinier(e));
				menuCuisinier.add(addCuisinier);				
				textArea.append(NomCuisinier + " prend son service !" + "\n");
		    }
}
public void supprimerCuisinier(ActionEvent e) {
      //.....
}

[Jb_One73]

Merci de ta réponse tchize_ !

Je n'arrive pas à comprendre le morceau de ton programme suivant

Code :

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

addCuisinier.addActionListener((e)-> supprimerCuisinier(e));

.

Pour tout t'avouer je débute en java donc c'est un peux compliquer pour moi.

Merci d'avance.

[tchize_]

C'est une lambda expression. C'est l'équivalent de

Code :

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

1
2
3
4
5
addCuisinier.addActionListener(new ActionListener() {
   public void actionPerformed (ActionEvent e) {
       supprimerCuisinier(e);
   }
});

en beaucoup moins verbeux.

Qui lui même est l'équivalent, en moins verbeux de [CODE]

Code :

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

1
2
3
4
5
6
7
8
addCuisinier.addActionListener(new RetirerCuisinierActionListener());
 
//....
public class RetirerCuisinierActionListener {
   public void actionPerformed (ActionEvent e) {
       supprimerCuisinier(e);
   }
}

[Jb_One73]

Bonjour tchize_,

J'y suis presque, maintenant le soucis c'est que lorsque je clique de nouveau sur le bouton"add" il ne se passe rien.
Pour la suppression c'est ok (enfin du moins je pense).

Je te joins mon code pour que tu puisses analyser mon erreur.


En attendant merci d'avance.

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
 
	public void actionPerformed (ActionEvent e) {
 
		/** AJOUT D'UN CUISINIER */
		if(e.getSource() == addCuisinier && e.getActionCommand() == "add"){
 
		    String NomCuisinier = JOptionPane.showInputDialog(null, "Entrer le nom du cuisinier", "");
 
		    if( NomCuisinier != null){
 
		    	addCuisinier = new JMenuItem(NomCuisinier);
		    	addCuisinier.addActionListener(new RetirerCuisinierActionListener());
			menuCuisinier.add(addCuisinier);				
			textArea.append(NomCuisinier + " prend son service !" + "\n");
		    }
		}
	}
 
 
	public class RetirerCuisinierActionListener implements ActionListener {
	   public void actionPerformed (ActionEvent e) {
	       supprimerCuisinier(e);
	   }
	}
 
	public void supprimerCuisinier(ActionEvent e) {
		menuCuisinier.remove(addCuisinier);
	}