Discussion :

[zied_m]

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 :

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
 
.....
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 !!?

[druzy]

Tu as une question ?

[sebabarre]

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

[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.

[zied_m]

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

[zied_m]

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()

[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.

[zied_m]

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 !!!! :'(

[druzy]

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

[zied_m]

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

voilà le 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
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 ..

[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

[zied_m]

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 )

[muad'dib]

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.

[Robin56]

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.

[druzy]

reposte ton code corrigé correctement stp

[tchize_]

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

[druzy]

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

[zied_m]

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 :

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
 ......
 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 , y a t'il d'autres solutions !!??

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

[wax78]

Dans ton dernier code, tu as le code utile dans

Code :

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

public void Actionperformed (ActionEvent e)

qui en fait est censé s'appeler :

Code :

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

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 :

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
 ......
 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);}
 
}

[zied_m]

enfin , ...

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