listener qui n'écoute pas

**sydius** · 18/06/2008, 11h44

Bonjour,

je débute en Swing et je m'exerce en faisant des listener pour changer la couleur de mon "panel". Je crois avoir fait tout ce qu'il fallait: implementer l'interface ActionListener, définir la méthode actionPerformed, ajouter les listeners aux boutons... je ne sais pas ce qui manque. En fait quand on clique sur les boutons, ça ne fait rien du tout.
Voici le code et merci.

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
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
 
package Frames;
 
import javax.swing.*;
import java.awt.*;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
 
 
class Model extends JFrame implements ActionListener{
 
    public Model(){
 
        setTitle("REMPLISSAGE DE DONNEES");
        setSize(350,200);
        setLocationRelativeTo(null);
        setResizable(true);
        setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        init();
 
    }
 
    public void init() {
 
       JPanel titres = new JPanel(new GridLayout(3,1));
       JLabel nom = new JLabel("Nom");
       titres.add(nom);
       JLabel prenom = new JLabel("Prénom");
       titres.add(prenom);
       JLabel adresse = new JLabel("Adresse");
       titres.add(adresse);
 
       JTextArea message = new JTextArea();
 
       JPanel saisie = new JPanel(new GridLayout(3,1));
       JTextField saisirNom = new JTextField(10);
       saisie.add(saisirNom);
       JTextField saisirPrenom = new JTextField(10);
       saisie.add(saisirPrenom);
       JTextField saisirAdresse = new JTextField(10);
       saisie.add(saisirAdresse);
 
       JPanel panelBoutons = new JPanel(new FlowLayout());
       JButton bvalider = new JButton("JAUNE");
       panelBoutons.add(bvalider);
       JButton bannuler = new JButton("ROUGE");
       panelBoutons.add(bannuler);
 
       bannuler.addActionListener(this);
       bvalider.addActionListener(this);
 
       getContentPane().setLayout(new BorderLayout(10,10));
       getContentPane().add(message, BorderLayout.NORTH);
       getContentPane().add(titres, BorderLayout.WEST);
       getContentPane().add(saisie, BorderLayout.CENTER);
       getContentPane().add(panelBoutons, BorderLayout.SOUTH);
 
    }
 
    private JButton valider;
    private JButton annuler;
 
   public void actionPerformed(ActionEvent e){
       Object source = e.getSource();
       Color color = getBackground();
       if(source == valider) color = Color.yellow;    
       else if (source == annuler) color = Color.red;
       setBackground(color);
       repaint();
   }
 
}

**NainzguL** · 18/06/2008, 11h50

Deja, je n'aurais pas fais ce genre de test :

Code :

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

if(source == valider)

En cours du moins, on nous l'as fortement deconseiller. utiliser plutot les methodes de Java :

Code :

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

if(source.equals(valider))

Ensuite, tu cherche a savoir si ton bouton enfoncer est "valider", alors que tu ajoute sur ta fenetre "bvalider".
Normal dans ce cas que ton Listener ne fasse rien.

**fnobb** · 18/06/2008, 11h53

Bonjour,

la condition "source == annuler" n'est jamais vérifiée car annuler reste à null

il faut remplacer "JButton bvalider = new JButton("JAUNE");" par "valider = new JButton("JAUNE");" et idem pour annuler

**sydius** · 18/06/2008, 11h58

Merci beaucoup pour ta réponse NainzguL. J'ai corrigé le code, que voici:

Code :

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

1
2
3
4
5
6
7
8
9
 
JPanel panelBoutons = new JPanel(new FlowLayout());
       JButton jaune = new JButton("JAUNE");
       panelBoutons.add(jaune);
       JButton rouge = new JButton("ROUGE");
       panelBoutons.add(rouge);
 
       jaune.addActionListener(this);
       rouge.addActionListener(this);

Cependant NetBeans me conseille fortement de renommer rouge et jaune.
Par ailleurs, si j'enlève ces lignes

Code :

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

1
2
3
 
private JButton jaune;
    private JButton rouge;

il m'affiche un problème au niveau de la méthode actionPerformed()
Des idées?

**sydius** · 18/06/2008, 12h01

Envoyé par fnobb

Bonjour,

la condition "source == annuler" n'est jamais vérifiée car annuler reste à null

il faut remplacer "JButton bvalider = new JButton("JAUNE");" par "valider = new JButton("JAUNE");" et idem pour annuler

merci fnobb, j'ai corrigé aussi ce que tu m'as proposé.
Avantage: il ne me demande plus de renomer jaune et rouge.
Inconvenient: ça ne marche toujours pas

**CheryBen** · 18/06/2008, 13h01

Bonjour, en faisant

Code :

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

1
2
3
private JButton jaune;
[...]
JButton jaune = new JButton("JAUNE");

Tu utilises 2 fois le même nom pour 2 variables différentes, sauf que pour appeler la 1ère il faudra faire this.jaune
C'est pour cela qu'il est conseillé de ne pas faire de doublon, pour éviter la confusion.

Il faut simplement faire :

Code :

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

1
2
3
private JButton jaune;
[...]
jaune = new JButton("JAUNE");

**sydius** · 18/06/2008, 13h14

C'est déjà fait, merci morph41, mais ça n'arrange pas le problème

**CheryBen** · 18/06/2008, 13h32

Suite à toutes les modifications, pourrais-tu reposter le code complet?

**sydius** · 18/06/2008, 14h04

Envoyé par morph41

Suite à toutes les modifications, pourrais-tu reposter le code complet?

Bien sûr, le voilà:

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
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
 
package Frames;
 
import javax.swing.*;
import java.awt.*;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
 
 
class Model extends JFrame implements ActionListener{
 
    public Model(){
 
        setTitle("REMPLISSAGE");
        setSize(350,200);
        setLocationRelativeTo(null);
        setResizable(true);
        setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        init();
 
    }
 
    public void init() {
 
       JPanel titres = new JPanel(new GridLayout(3,1));
       JLabel nom = new JLabel("Nom");
       titres.add(nom);
       JLabel prenom = new JLabel("Prénom");
       titres.add(prenom);
       JLabel adresse = new JLabel("Adresse");
       titres.add(adresse);
 
       JTextArea message = new JTextArea();
 
       JPanel saisie = new JPanel(new GridLayout(3,1));
       JTextField saisirNom = new JTextField(10);
       saisie.add(saisirNom);
       JTextField saisirPrenom = new JTextField(10);
       saisie.add(saisirPrenom);
       JTextField saisirAdresse = new JTextField(10);
       saisie.add(saisirAdresse);
 
       JPanel panelBoutons = new JPanel(new FlowLayout());
       jaune = new JButton("JAUNE");
       panelBoutons.add(valider);
       rouge = new JButton("ROUGE");
       panelBoutons.add(annuler);
 
       jaune.addActionListener(this);
       rouge.addActionListener(this);
 
 
       getContentPane().setLayout(new BorderLayout(10,10));
       getContentPane().add(message, BorderLayout.NORTH);
       getContentPane().add(titres, BorderLayout.WEST);
       getContentPane().add(saisie, BorderLayout.CENTER);
       getContentPane().add(panelBoutons, BorderLayout.SOUTH);
 
    }
 
 
 
       public void actionPerformed(ActionEvent e){
 
           Object source = e.getSource();
           Color color = getBackground();
           if (source.equals(jaune)) color = Color.yelow;
           else if (source.equals(rouge)) color = Color.red;
           setBackground(color);
           repaint();
 
 
       private JButton jaune;
       private JButton rouge;

Remarque: je vois dans la doc trouvée sur le net très souvent "if (source == jaune)"

**CheryBen** · 18/06/2008, 14h28

Il y a plusieurs erreurs, ce code ne compile pas, pourquoi ne pas lire les messages d'erreur du compilateur?

Tu as laissé valider et annuler alors qu'ils ont été renommés en jaune et rouge.

Code :

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

1
2
3
4
jaune = new JButton("JAUNE");
panelBoutons.add(jaune);
rouge = new JButton("ROUGE");
panelBoutons.add(rouge);

Il manque des accolades (peut-être une erreur de copier/collé dans le forum).

L'écoute fonctionne, c'est simplement que tu changes la couleur de fond de la JFrame, or son fond n'est pas visible, on voit seulement le fond des JPanels qu'elle contient (ils sont par-dessus). En faisant ceci, ça fonctionne :

Code :

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

1
2
3
4
5
6
7
    public void actionPerformed(ActionEvent e) {
        Object source = e.getSource();
        if (source == jaune)
            jaune.setBackground(Color.yellow);
        else if (source == rouge)
            rouge.setBackground(Color.red);
    }

Le == fonctionne car les variables pointent sur le même espace mémoire, mais parfois cet opérateur est utilisé à mauvais escient, alors que la méthode equals() est beaucoup moins ambigüe, c'est pour cela qu'il est conseillé de l'utiliser.

**NainzguL** · 18/06/2008, 14h28

Ce que je trouve etrange, c'est cette derniere partie :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
       public void actionPerformed(ActionEvent e){
 
           Object source = e.getSource();
           Color color = getBackground();
           if (source.equals(jaune)) color = Color.yelow;
           else if (source.equals(rouge)) color = Color.red;
           setBackground(color);
           repaint();
 
 
       private JButton jaune;
       private JButton rouge;

Ne manque-t-il pas une accolade ? Et, pourquoi declares-tu tes variables a la fin, et non avant toute chose ? Ca n'a peut etre rien a voir, mais, Eclipse te dirais qu'il ne trouve pas tes variables si elles sont placees a la fin.

Pour ce qui est du source == jaune, je me contente simplement de repeter ce que j'ai appris en DUT (et ce pourquoi on se faisait tirer des points....).

**CheryBen** · 18/06/2008, 14h32

Envoyé par NainzguL

Et, pourquoi declares-tu tes variables a la fin, et non avant toute chose ? Ca n'a peut etre rien a voir, mais, Eclipse te dirais qu'il ne trouve pas tes variables si elles sont placees a la fin.

Non pas du tout, ce sont des attributs de la classe, ils sont chargés au chargement de la classe, avant tout appel des méthodes, on peut très bien les déclarer entre 2 méthodes. Si on les mets en général en début ou fin de classe, ce n'est qu'une question de clarté.

**NainzguL** · 18/06/2008, 14h35

ah, d'accord, au temps pour moi. Merci pour cette precision Morph41.

**sydius** · 18/06/2008, 14h48

Merci beaucoup pour toutes ces précisions. Le problème est réglé (pour les accolades, c'était un pb de copier-coller)

**sydius** · 18/06/2008, 15h36

Rebonjour,

je ne voudrais pas être lourd, mais voici ce qui se passe maintenant.
Suite aux suggestions données sur le forum, et après vérification que cela marchait, j'ai essayé une autre chose:
- on clique sur le bouton "valider" et le message "Bienvenue" est affiché dans le JTextArea
- on clique sur le bouton "Annuler" et "Au revoir" s'affiche au même endroit.

J'ai fait une classe interne et ça marchait. Ensuite j'ai copié le code de la classe interne sur une autre classe, mais toujors dans le même package (avec tous les imports nécessaires et rendant la classe public) Aucune erreur était signalée au niveau du code, la fenêtre s'affichait mais aucune réaction lorsqu'on appuyait sur les boutons. Je reviens sur le code premier, une classe public avec une classe interne, et OH, surprise! cela ne marche pas non plus.
J'ai refait les imports, j'ai revu le code, j'ai même relancé NetBeans au cas où... rien n'y fait. Et je ne comprends pas où est le problème.
Je poste 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
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
 
package Frames;
 
import javax.swing.*;
import java.awt.*;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
 
 
public class Model extends JFrame {
 
    public Model(){
 
        setTitle("REMPLISSAGE DE DONNEES");
        setSize(350,200);
        setLocationRelativeTo(null);
        setResizable(true);
        setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        init();
 
    }
 
    public void init() {
 
       JPanel titres = new JPanel(new GridLayout(3,1));
       JLabel nom = new JLabel("Nom");
       titres.add(nom);
       JLabel prenom = new JLabel("Prénom");
       titres.add(prenom);
       JLabel adresse = new JLabel("Adresse");
       titres.add(adresse);
 
       JTextArea message = new JTextArea();
 
       JPanel saisie = new JPanel(new GridLayout(3,1));
       JTextField saisirNom = new JTextField(10);
       saisie.add(saisirNom);
       JTextField saisirPrenom = new JTextField(10);
       saisie.add(saisirPrenom);
       JTextField saisirAdresse = new JTextField(10);
       saisie.add(saisirAdresse);
 
       JPanel panelBoutons = new JPanel(new FlowLayout());
       JButton valider = new JButton("VALIDER");
       panelBoutons.add(valider);
       JButton annuler = new JButton("ANNULER");
       panelBoutons.add(annuler);
 
       valider.addActionListener(new ClicListener(message));
       annuler.addActionListener(new ClicListener(message));
 
 
       getContentPane().setLayout(new BorderLayout(10,10));
       getContentPane().add(message, BorderLayout.NORTH);
       getContentPane().add(titres, BorderLayout.WEST);
       getContentPane().add(saisie, BorderLayout.CENTER);
       getContentPane().add(panelBoutons, BorderLayout.SOUTH);
 
    } 
 
   class ClicListener implements ActionListener{
 
       JTextArea cible;
       JButton vali;
       JButton annu;
 
       public ClicListener(JTextArea cible)
       {
           this.cible=cible;
       }
 
       public void actionPerformed(ActionEvent e){
           Object source = (JButton)e.getSource();
           if (source.equals(vali)) cible.append("Bienvenue");
           else if (source.equals(annu)) cible.append("Au revoir");
 
    }
}
}

Merci de votre aide.

**CheryBen** · 18/06/2008, 15h51

Rebonjour, tu as refait la même erreur.
Que valent vali et annu dans ClicListener ?

null

Si un code fonctionne à un moment et que quand tu le réessaie il ne fonctionne plus, c'est qu'il a changé

Tu as redéclaré tes boutons en variables locales à init()

**sydius** · 18/06/2008, 16h14

Envoyé par morph41

Rebonjour, tu as refait la même erreur.
Que valent vali et annu dans ClicListener ?

null

Si un code fonctionne à un moment et que quand tu le réessaie il ne fonctionne plus, c'est qu'il a changé

Tu as redéclaré tes boutons en variables locales à init()

Bon, j'ai beau retourner mes variables dans tous les sens, je suis toujours coincé, mais je trouverai.
Merci beaucoup morph41 pour la piste.

**offspring** · 18/06/2008, 16h18

ta essayé :

Code :

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

1
2
 if(source == bvalider) color = Color.yellow;    
       else if (source == bannuler) color = Color.red;

A+

**sydius** · 18/06/2008, 16h20

Cette fois c'est bon, ouf!

Merci à tous de votre aide, en particulier morph41 qui a pris le temps de répondre à mes questions

listener qui n'écoute pas

Composants Java

Discussions similaires

Partager

Partager