Discussion :

[sydius]

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]

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.

[sydius]

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?

[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

[sydius]

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]

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]

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

[CheryBen]

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

[offspring]

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+