Problème avec ActionListener

Bonsoir :)

Je suis en train de développer une application Java (avec Eclipse ), cette application contient deux boutons :
- bouton valider (qui se trouve dans un JFrame) permet d'ouvrir une autre JFrame
- bouton annuler permet de quitter l'application

Voilà le code que j'ai écrit :

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

..... panneau_Bouton.add(valide=new JButton("valider : ")); panneau_Bouton.add(annul=new JButton("Annuler : "));   valide.addActionListener(this); valide.setActionCommand("valider");   annul.addActionListener(this); annul.setActionCommand("annuler");   } //Gestion des evenements     public void Actionperformed(ActionEvent e) {     if (e.getActionCommand() == "valider") { new menu_generale().setVisible(true);   // menu_generale est une classe qui contient la deuxiéme JFrame } else if (e.getActionCommand() == "annuler"){ System.exit(0); } }

---

le problème ce lorsque je clique sur une des deux boutons ne passe rien , y a t'il des solutions pour ça !!?

Tu as une question ?

Et si t'essaie avec des e.getSource() == leNomDeTonBouton il se passe quoi ?

Le nom de la méthode "actionPerformed" n'est pas correct.
Tu devrais utiliser "@Override" juste avant le nom de la méthode si tu es en Java >= 1.5, ça éviterait ce genre de problème.

Citation:

---

Envoyé par dinobogan

Le nom de la méthode "actionPerformed" n'est pas correct.
Tu devrais utiliser "@Override" juste avant le nom de la méthode si tu es en Java >= 1.5, ça éviterait ce genre de problème.

---

bon :( j'ai pas compris ta proposition , si tu peux mieux expliquer !!? et merci :D

Citation:

---

Envoyé par sebabarre

Et si t'essaie avec des e.getSource() == leNomDeTonBouton il se passe quoi ?

---

il se passe rien , même problème avec le e.getsource()

dans ton code, tu as écris actionPerformed sans majuscule d'où l'erreur.

@Override signale au compilateur de te prévenir si la méthode que tu utilises n'est pas une méthode surchargée, ce qui évite ce genre d'erreur.

Citation:

---

Envoyé par druzy

dans ton code, tu as écris actionPerformed sans majuscule d'où l'erreur.

@Override signale au compilateur de te prévenir si la méthode que tu utilises n'est pas une méthode surchargée, ce qui évite ce genre d'erreur.

---

même problème , ça marche pas !!!! :'(

Peux-tu poster le code corrigé et les résultats des erreurs ?

Citation:

---

Envoyé par druzy

Peux-tu poster le code corrigé et les résultats des erreurs ?

---

voilà le code

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

..... valide.addActionListener(this); valide.setActionCommand("valider"); annul.addActionListener(this); annul.setActionCommand("annuler");   public void actionperformed(ActionEvent e) {   try{   if (e.getActionCommand() == "valide"){ if (login.getText().equals ("") == true) { JOptionPane.showMessageDialog (this,"erreur"); } } else { new menu_generale().setVisible(true);   } } catch (Exception er){ JOptionPane.showMessageDialog(this,er.getMessage()); }   if (e.getActionCommand() == "annuler"){ System.exit(0);   }   }       @Override public void actionPerformed(ActionEvent e) { // TODO Auto-generated method stub   }

---

l'erreur c'est la non fonctionnement des boutons (valider & annuler ) , c'est à dire le clique sur un des deux boutons ne fait aucune action ..

La méthode appelée par ActionListener est actionPerformed (sensible à la casse ! actionPerformed est différent de actionperformed).

Place le code de ton action dans la fonction actionPerformed

Citation:

---

Envoyé par muad'dib

La méthode appelée par ActionListener est actionPerformed (sensible à la casse ! actionPerformed est différent de actionperformed).

Place le code de ton action dans la fonction actionPerformed

---

non plus :( , mais je demande
est ce que l'erreur peut être dans une des deux JFrame surtout dans la deuxième JFrame (générale ) parce que je ne l'ai pas testé !!!!

rappelons que mon problème est le suivant :

j'ai un Jframe (connexion) qui contient deux boutons valider et annulé , le bouton valider permet d'ouvrir un autre Jframe (générale )

Avant d'implémenter différentes Frames, mon conseil est d'essayer de faire fonctionner tes boutons déjà dans une seule.

Si après ça tu n'y arrives pas postes nous ton code épuré au maximum, avec juste la gestion des clics sur les boutons.

Ce tutoriel (http://baptiste-wicht.developpez.com.../?page=boutons) te donne le code minimal pour implémenter une action à ton bouton. Pars de cet exemple et ensuite tu pourras envisager ton projet.

reposte ton code corrigé correctement stp

et n'oublie pas le @Override devant la méthode, comme déjà dit. Aussi, on compare des String avec equals, pas avec ==

Je pense également que comparer les String avec == ne te fait pas rentrer dans ton if

bon , voilà mon code modifié (j'ai enlevé la partie qui fait le test sur le zone de texte "login"
sachant que mon problème est le même

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

...... panneau_label.add(new JLabel ("Login : ")); panneau_saisie.add(login = new JTextField(8)); panneau_label.add(new JLabel (" Mot de passe : ")); panneau_saisie.add(mot_passe=new JTextField(8)); // manipulation des boutons panneau_Bouton.add(valide=new JButton("valider : ")); panneau_Bouton.add(annul=new JButton("Annuler : "));   valide.addActionListener(this); valide.setActionCommand("validation"); annul.addActionListener(this); annul.setActionCommand("annuler");     } //Gestion des evenements   public void Actionperformed (ActionEvent e) {       if (e.getActionCommand()=="validation"){   new menu_generale().setVisible(true);}   @Override public void actionPerformed(ActionEvent e) { // TODO Auto-generated method stub   }

---

toujours la même erreur :cry: , y a t'il d'autres solutions !!?? :D

et désolé pour le retard de ma réponse ....

Dans ton dernier code, tu as le code utile dans

Code:

---

public void Actionperformed (ActionEvent e)

---

qui en fait est censé s'appeler :

Code:

---

public void actionPerformed(ActionEvent e)

---

qui lui ne contient aucun code... donc forcement rien ne risque de se passer si tu ne supprime pas Actionperformed en y mettant le code dans actionPerformed.

Donc pour résumer une dernière fois tu dois avoir ceci :

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

...... panneau_label.add(new JLabel ("Login : ")); panneau_saisie.add(login = new JTextField(8)); panneau_label.add(new JLabel (" Mot de passe : ")); panneau_saisie.add(mot_passe=new JTextField(8)); // manipulation des boutons panneau_Bouton.add(valide=new JButton("valider : ")); panneau_Bouton.add(annul=new JButton("Annuler : "));   valide.addActionListener(this); valide.setActionCommand("validation"); annul.addActionListener(this); annul.setActionCommand("annuler");     } //Gestion des evenements     @Override public void actionPerformed(ActionEvent e) {   if (e.getActionCommand()=="validation"){new menu_generale().setVisible(true);}   }

---

enfin , ...

merci beaucoup pour ta réponse , ça marche très bien :mouarf::mouarf::mouarf: