Problème JPopupMenu et JMenu

Version imprimable

25/01/2007, 12h09
Yannick_from_31

Problème JPopupMenu : le JMenu n'apparaît pas

Bonjour,

j'ai un JPopupMenu A auquel je veux ajouter un JMenu B pour avoir des sous-menu du style B1, B2... :

A > B > B1
------> B2
------> B3
---> C (JMenuItem OK qui fonctionne bien)

Mon JPopupMenu A fonctionne bien et les divers JMenuItem dedans (C...) aussi.

Mais, mon JMenu B ne veut pas s'ouvrir quand je clique dessus :(
Le problème ne vient pas apparemment du JMenu B car si je le met dans ma JMenuBar de mon appli, il fonctionne très bien. Mais quand il est dans mon JPopupMenu A, il ne fonctionne pas.

J'ai essayé de mettre un ActionListerner dessus B : il ne capte pas les clics ! Mais par contre avec un MouseListener, le JMenu B capte les clics, mais impossible d'ouvrir la liste des sous menus B1, B2... et un doClick sur le JMenu B à ce moment là fait bloquer mon appli !

Comment ouvrir mon sous-menu B ?

Merci d'avance pour une aide.

Yannick.

Tu as dû faire comme cela pour ajouter tes JmenuItem
Code:

1 2 3 4 5 6 7 8 JPopupMenu popup = new JPopupMenu(); menuItem = new JMenuItem("A popup menu item"); menuItem.addActionListener(this); popup.add(menuItem); menuItem = new JMenuItem("Another popup menu item"); menuItem.addActionListener(this); popup.add(menuItem);
D'après la Javadoc de la classe JPopUpMenu, la méthode add() ne fonctionne que pour ajouter des JMenuItem.

Pour ajouter tout autre composant à ton JPopUpMenu, tu dois utiliser la méthode insert().

Description methode insert() JPopUpMenu

Je pense que ça devrait aller mieux ;)

En fait, la méthode add(Component comp) fonctionne très bien mais il faut faire comme ceci.

Code:

1
2
3
4
5
6
7
 
JPopupMenu menu = new JPopupMenu();
JMenu A = new JMenu("A");
JMenuItem A1 = new JMenuItem("A1");
 
menu.add(A); // tu ajoutes le JMenu à JPopupMenu.
A.add(A1); // Tu ajoutes le JMenuItem à JMenu.

Je ne mets pas l'ActionListener, etc.

Tu peux toujours tester ç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
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
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
 
import java.awt.event.MouseEvent;
import java.awt.event.MouseListener;
 
import javax.swing.SwingUtilities;
import javax.swing.JPanel;
import javax.swing.JFrame;
import javax.swing.JPopupMenu;
import javax.swing.JMenu;
import javax.swing.JMenuItem;
import javax.swing.JLabel;
import java.awt.Rectangle;
 
public class Test extends JFrame implements MouseListener
    {
 
        private static final long serialVersionUID = 1L;
 
        private JPanel panel;
 
        private JPopupMenu pop = null;
 
        private JMenu jMenu = null;
 
        private JMenuItem jMenuItem = null;
 
        private JMenu jMenu1 = null;
 
        private JMenuItem jMenuItem1 = null;
 
        private JLabel jLabel = null;
 
        /**
         * This method initializes pop    
         *     
         * @return javax.swing.JPopupMenu    
         */
        private JPopupMenu getPop()
            {
                if (pop == null)
                    {
                        pop = new JPopupMenu();
                        pop.add(getJMenu());
                        pop.add(getJMenu1());
                    }
                return pop;
            }
 
        /**
         * This method initializes jMenu    
         *     
         * @return javax.swing.JMenu    
         */
        private JMenu getJMenu()
            {
                if (jMenu == null)
                    {
                        jMenu = new JMenu("Salut");
                        jMenu.add(getJMenuItem());
                    }
                return jMenu;
            }
 
        /**
         * This method initializes jMenuItem    
         *     
         * @return javax.swing.JMenuItem    
         */
        private JMenuItem getJMenuItem()
            {
                if (jMenuItem == null)
                    {
                        jMenuItem = new JMenuItem("Hello");
                    }
                return jMenuItem;
            }
 
        /**
         * This method initializes jMenu1    
         *     
         * @return javax.swing.JMenu    
         */
        private JMenu getJMenu1()
            {
                if (jMenu1 == null)
                    {
                        jMenu1 = new JMenu("World");
                        jMenu1.add(getJMenuItem1());
                    }
                return jMenu1;
            }
 
        /**
         * This method initializes jMenuItem1    
         *     
         * @return javax.swing.JMenuItem    
         */
        private JMenuItem getJMenuItem1()
            {
                if (jMenuItem1 == null)
                    {
                        jMenuItem1 = new JMenuItem("coucou");
                    }
                return jMenuItem1;
            }
 
        /**
         * @param args
         */
        public static void main(String[] args)
            {
                // TODO Raccord de méthode auto-généré
 
                SwingUtilities.invokeLater(new Runnable()
                    {
                        public void run()
                            {
                                Test thisClass = new Test();
                                thisClass.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
                                thisClass.setLocationRelativeTo(thisClass.getParent());
                                thisClass.setVisible(true);
                            }
                    });
            }
 
        /**
         * This is the default constructor
         */
        public Test()
            {
                super();
                initialize();
            }
 
        /**
         * This method initializes this
         * 
         * @return void
         */
        private void initialize()
            {
                this.setSize(300, 200);
                this.setContentPane(getPanel());
                this.setTitle("JFrame");
            }
 
        /**
         * This method initializes panel
         * 
         * @return javax.swing.JPanel
         */
        private JPanel getPanel()
            {
                jLabel = new JLabel();
                jLabel.setBounds(new Rectangle(61, 61, 175, 16));
                jLabel.setText("Click quelque part");
                panel = new JPanel();
                        panel.setLayout(null);
                        panel.add(getPop());
                        panel.add(jLabel, null);
                        panel.addMouseListener(this);
                return panel;
            }
 
        public void mouseClicked(MouseEvent e)
            {
                // TODO Raccord de méthode auto-généré
                if (e.getSource() == panel)
                    {
                        pop.show(e.getComponent(), e.getX(), e.getY());
                    }
 
            }
 
        public void mouseEntered(MouseEvent e)
            {
                // TODO Raccord de méthode auto-généré
 
            }
 
        public void mouseExited(MouseEvent e)
            {
                // TODO Raccord de méthode auto-généré
 
            }
 
        public void mousePressed(MouseEvent e)
            {
                // TODO Raccord de méthode auto-généré
 
            }
 
        public void mouseReleased(MouseEvent e)
            {
                // TODO Raccord de méthode auto-généré
 
            }
 
    }

Tu dois en fait mettre tes JMenuItem dans des JMenu et pas directement le JMenuItem dans le JPopupMenu
Bonne journée

29/01/2007, 12h28
Yannick_from_31

Merci beaucoup pour votre aide :D .

En fait, j'ai réussi à trouver grâce au code de FredPsy même si l'erreur ne venait pas de ce que vous disiez : l'ordre des ajouts des menu et sous-menus ne change rien (j'ai vérifié et testé).

Mon erreur était de mettre visible mon JPopupMenu par setVisible(true) après l'avoir positionné.

Changer setVisible par ".show(e.getComponent(), e.getX(), e.getY());" fait tout rentrer dans l'ordre.
Le problème venait apparemment du JPopupMenu qui ne prenait pas le focus car mes menus n'étaient même pas mis en surbrillance et les sous-menus étaient alors inaccessibles.

Encore merci.
29/01/2007, 12h35
FredPsy

Bien heureux de t'avoir aidé.

Bonne journée.