Discussion :

[ArchFolken]

Bonjour, j'ai un petit soucis avec le ActionListenner.
Je reprends actuellement un code existant (que je ne maitrise pas totalement). Il s'agit d'une appli Java visuel (JFrame), et lorsque dans cette appli j'ajoute une nouvelle fenetre l'ActionListenner ne fonctionne pas correctement, je m'explique.
Voici un bout de code simple que j'aimerai inserer :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
/* choix couleur */
	JButton	colorButton = new JButton("Couleur");
	panelM.add(colorButton) ;
	colorButton.addActionListener(this) ;
....
....
....
public void actionPerformed (ActionEvent e)
{ 
     if (e.getSource() == m_colorButton)
	System.out.println("Ca passe"); 
}

Bien si je fais cela le click sur mon bouton est completement ignoré, en regardant le code de plus pres, j'ai vu qu'ils utilisaient cette facon de faire :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
ButEnd = new JButton();
      ButEnd.setActionCommand(endString);
      ButEnd.addActionListener(this);
.....
.....
.....
public void actionPerformed(ActionEvent e)
{
    String str = e.getActionCommand();
    if (str.equals(endString))
      {....}
}

Qqun peut il m'éclairer sur cette facon de faire, et si c'est normal que ma premiere solution ne fonctionne pas ?

Merci bien.

[GLDavid]

Voici la manière dont j'implémente un JButton (extrait de l'un de mes prog) :

Code :

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

1
2
3
4
5
6
7
8
9
10
 
jButton2.addActionListener(new java.awt.event.ActionListener() {
            public void actionPerformed(java.awt.event.ActionEvent evt) {
                callback(evt);
            }
        });
 
private void callback(java.awt.event.ActionEvent evt) {
        //Traitement de données
    }

J'espère que ça t'aidera.

@ ++

[iuz]

On ne compare pas des objets avec == mais equals()

Code :

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

1
2
3
4
5
public void actionPerformed (ActionEvent e) 
{ 
   if (e.getSource().equals(m_colorButton))
     System.out.println("Ca passe"); 
}

Là ça devrait passer. Mais le mieux c'est de passer par actionCommand ou d'utiliser la méthode de GLDavid

[narfiggo]

La normalement ca devrait marcher, t'avais juste une petite erreur

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
/* choix couleur */ 
   JButton   colorButton = new JButton("Couleur"); 
   panelM.add(colorButton) ; 
   colorButton.addActionListener(this) ; 
.... 
.... 
.... 
public void actionPerformed (ActionEvent e) 
{ 
     if (e.getSource() == colorButton) //colorButton et pas m_colorButton
   System.out.println("Ca passe"); 
}

En ce qui concerne la difference entre la premiere et la seconde solution :
Dans la premiere:
Tu utilise pour savoir d'ou vien l'action, la fonction e.getSource(). Donc tu fais un test sur l'objet correspondant à l'action.
Dans la seconde :
Il associe un string au bouton par la methode setActionCommand. Donc son test ne se fait plus sur l'objet ayant entrainé l'action mais sur le String representant cet objet. e.getActionCommand(), il recupere le string associé au bouton.
L'interet de cette methode est de pouvoir associé la même action à differents boutons en leur associant la même String à chacun. De plus lors du test pour savoir qui a engendré l'action, c'est plus lisible de reconnaitre un String representant la commande que de reconnaitre un objet boutton.

Voila dans les grandes lignes la différences entre les 2, en esperant avoir ete clair

[peppena]

essaye ça :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
    JButton   colorButton = new JButton("Couleur"); 
 
    colorButton .addActionListener(new ActionListener() 
      { 
               public void actionPerformed(ActionEvent e) 
         { 
             System.out.println("Ca passe --- avant le if "); 
                if (e.getSource() == m_colorButton) 
   System.out.println("Ca passe"); 
} 
          });

[narfiggo]

On ne compare pas des objets avec == mais equals()[code]

Ca passe trés bien avec un == pour la comparaison de ces 2 boutons. Mais t'as raison, c'est plus propre de passer par des equals() pour la comparaison d'objet entre eux.

[ArchFolken]

Merci bcp pour votre aide, c'est un plaisir d'en apprendre un peu plus chaque jour