Discussion :

[kesiahlex]

Bonjour,
Comment pourrais-je contourner ce problème là :

http://bugs.sun.com/bugdatabase/view...bug_id=4103931

J'ai une JFrame, une JmenuBar, des JMenu avec sous-Menu et enfin JMenuItem.

Je suis dans mon actionPerformed et event.source est un JMenuItem. Je voudrais recuperer ma JFrame pour pouvoir atteindre un autre composant.

Merci de lire vite fais le Bug decris ci dessus pour comprendre le probleme.

Est ce que ma seule solution est d'etendre JMenuItem et de rajouter un attribut qui serait ma JFrame ?

merci d'avance.

[demonia]

tu peux remonter petit a petit il me semble.
le getParent() doit te renvoyer le JMenu ou le JMenuBar d'en dessus donc fais plusieurs fois l'appel (je ne sais pas si ça marche toujours)

sinon tu peux essayer avec getRootPane()

[Gfx]

Il y a encore plus simple : SwingUtilities.get*(). Par exemple SwingUtilities.getWindowAncestor(leJMenuItem). Tu peux ensuite caster le résultat en JFrame.

[kesiahlex]

Merci de vos réponse mais malheureusement ca ne résoud pas mon probleme.

En fait si vous lisez bien ça :

This problem is caused by the fact that the
implementation of JMenu doesn't add a JMenuItem
to itself: It adds a JMenuItem to one of its
private member which is a JPopupMenu. (See
JMenu.java)

En fait ça marche (et ça marchait déjà) avec un composant quelconque mais pas avec JMenuItem.

Merci bien, sinon ben pour l'instant ça marche avec un attribut contenant la reference sur la JMenuBar dans chaque JMenuItem. Ca va parce qu'il n'y en a pas beaucoup.

Le swingUtilities est interessant je vais regarder. Je peux déjà dire après un test que ça marche pas non plus pour les JMenuItem. Le composant retourné est null.

merci

[ EDIT ] :
ps : c'est quoi la difference entre SwingUtilities.getWindowAncestor() et SwingUtilities.windowForComponent()

[Gfx]

Pour ton PS : la doc dit que ces deux méthodes font pareil.

Bref, désolé j'avais oublié que c'est différent quand un popup menu se trouve dans la hiérarchie. Voici un exemple que j'utilise dans une application et dont tu peux t'inspirer :

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
 
  public static JextFrame getJextParent(EventObject evt)
  {
    if (evt != null)
    {
      Object o = evt.getSource();
      if (o instanceof Component)
      {
        Component c = (Component) o;
        for( ; ; )
        {
          if (c instanceof JextFrame)
            return (JextFrame) c;
          else if (c == null)
            break;
          if (c instanceof JPopupMenu)
            c = ((JPopupMenu) c).getInvoker();
          else
            c = c.getParent();
        }
      }
    }
    return null;
  }

Remplace les mentions à JextFrame par JFrame et ça devrait fonctionner. Note le cas particulier pour les popup menu avec un appel à getInvoker() au lieu de getParent().

[kesiahlex]

Bon merci,

En fait ça je savais faire, il y a juste le getInvoker() pour lequel j'aurai du mieux regarder dans la doc. J'ai pas pensé à regarder dans JPopupMenu.

Sinon ben voila ce que j'ai pour recuperer ma main frame :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
public JFrame getJFrame(Component e) throws ClassCastException{
        if(e instanceof JMenuItem){
            while( null != e && !(e instanceof JFrame) )
                if(e instanceof JPopupMenu)
                    e = ((JPopupMenu)e).getInvoker();
                else
                    e = ((JComponent)e).getParent();
        }else{
            e = SwingUtilities.getWindowAncestor(e);
        }
        return (JFrame)e;
    }

Merci beaucoup.