Discussion :

[une_tite_question]

Bonjour,
Je viens encore profiter de ce formidable site et de ses contributeurs.
J'ai une fenetre principale, à partir de laquelle je clique sur des JMenuitem
qui me permettent d'ouvrir d'autres fenetres.
Voilà comment je fais :
Sur les Jmenuitem j'ai des événements qui crée un objet des fenetres secondaires.Voici le code de ActionPerformed si ce n'est pas clair :

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
 
private void AvanceeActionPerformed(java.awt.event.ActionEvent evt) {                                        
        // TODO add your handling code here:
        JMenuItem monItem = (JMenuItem)evt.getSource() ;
          if (item == monItem)
        {
            System.out.println("Vous avez cliqué sur item") ;
            essai2 = new AfficheChoix2() ;
            essai2.setVisible(true) ;
        } 
 
        if ( Avancee == monItem )
        {
            System.out.println("Vous avez cliqué sur Avancée ") ;
             essai = new AjouterLic() ;
             essai.setVisible(true);
 
        }
 
    }

Première question : est ce la bonne méthode pour créer des fenetres secondaires.
Deuxième question : quelq'un peut il m'expliquer pourquoi mon JMenuitem item
ne donne rien tandis que Avancee me donne bien l'affichage.
En vous remerciant d'avance .
Merci

[CheryBen]

Bonjour, pour comparer les item, tu dois utiliser la méthode equals et non ==

[jojodu31]

d'accord avec morph41

Code :

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

if (monItem.equals(item))

est plus correct que :

Code :

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

if (item == monItem)

Après cela me parait bizarre que seul l'un des deux marche...peux tu poster plus de code (création du Menu notamment )

[Takis]

Comme il compare l'objet, il peut utiliser == puisqu'il va comparer la référence et non ce qu'il contient. De plus, c'est mieux dans ce cas ci je pense car ça ne sert à rien de vérifier ce que contient l'objet.

Pour ouvrir des fenêtres secondaire, j'ai tendance à utiliser des JOptionPane au lieu des JFrame. Mais bon, c'est un détail. Ca ne résoud pas vraiment ton problème ^^

Sinon, au lieu d'avoir toutes tes actions dans la même méthode, tu peux les séparer.
En gros, au lieu de faire :

Code :

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

item.addActionListener(this);

Tu peux faire :

Code :

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

1
2
3
4
5
item.addActionListener(new ActionListener() {
	public void actionPerformed(ActionEvent e) {
		//mon code
	}
});

Ca sera beaucoup plus lisible si jamais tu as beaucoup d'actionListener à mettre.

//edit : Je pense avoir trouver l'erreur, pourquoi la méthode s'appelle AvanceeActionPerformed ? Cela devrait être actionPerformed tout simplement.

[une_tite_question]

Bonjour ,
Effectivement le problème venait de là . Mon code est généré par netbeans donc il m'a généré automatiquement
cette fonction mais je vais refaire comme tu dis un item.addActionListener(classe interne) j'ai vu mon erreur et j'essaierai de poster
un code plus propre.
Vous pensez donc qu'il est préférable d'utiliser des JOptionPane plûtot que des JFrame qui corespondrait plus à des fenetres principales.

Je vous poste un code plus propre d'ici demain pour que vous puissiez me donner un avis.

Merci

[une_tite_question]

Bonjour ,
Je devais poster un code propre , mais pour un souci de clarté je préfère
fermer ce post, et en ouvrir un autre.
Merci à tous ceux qui m'ont aidé à résoudre mon problème.