Action Listener qui ne fonctionne pas

**pinps** · 16/11/2013, 16h12

Bonjour,
Je suis débutant et ne comprend pas pourquoi dans mon action performed il ne reconnait les btn_trinagle, btn_carre et btn_cercle.
Mon but est d'afficher un message différent en fonction du bouton cliqué.

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
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
import java.awt.BorderLayout;
import java.awt.Color;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JPanel;
 
 
/**
 *
 * @author pinps
 */
public class FenPrincipal extends JFrame {
 
    /**
     * @param args the command line arguments
public     */
 
    public static void main(String[] args) {
        //Description de la fenetre principal
        FenPrincipal fen=new FenPrincipal();
        fen.setTitle("Dessinons");
        fen.setSize(500,300);
        fen.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        fen.setVisible(true);
 
    }
    public FenPrincipal(){
        BoutonForme boutonforme=new BoutonForme();
        //Création de la zonne du haut
        JPanel zone_haut=new JPanel();
        //Création de boutons et libéllés
        JButton btn_triangle=new JButton("Triangle");
        JButton btn_carre=new JButton("Carré");
        JButton btn_cercle=new JButton("Cercle");
        JButton btn_raz=new JButton("RAZ");
        JLabel lbl_forme=new JLabel("Forme : X");
        JLabel lbl_couleur=new JLabel("Coul. : X");
        //Ajout des boutons et libellés à la zonne du haut
        zone_haut.add(btn_triangle);
        zone_haut.add(btn_carre);
        zone_haut.add(btn_cercle);
        zone_haut.add(btn_raz);
        zone_haut.add(lbl_forme);
        zone_haut.add(lbl_couleur);
        //Enregistrement du recepteur d'action
        btn_triangle.addActionListener(boutonforme);
        btn_carre.addActionListener(boutonforme);
        btn_cercle.addActionListener(boutonforme);
        btn_raz.addActionListener(boutonforme);
        //Placement de la zone du haut
        getContentPane().add(BorderLayout.NORTH, zone_haut);
 
        //Création de la zone du bas
        JPanel zone_bas=new JPanel();
        //Création des boutons de la zone du bas
        JButton btn_bleu=new JButton("Bleu");
        btn_bleu.setBackground(Color.blue);
        JButton btn_jaune=new JButton("Jaune");
        btn_jaune.setBackground(Color.yellow);
        JButton btn_rouge=new JButton("Rouge");
        btn_rouge.setBackground(Color.red);
        JButton btn_vert=new JButton("Vert");
        btn_vert.setBackground(Color.green);
        JButton btn_noir=new JButton("Noir");
        btn_noir.setBackground(Color.black);
        //Ajout des boutons à la zone du bas
        zone_bas.add(btn_bleu);
        zone_bas.add(btn_jaune);
        zone_bas.add(btn_rouge);
        zone_bas.add(btn_vert);
        zone_bas.add(btn_noir);
        //Placement de la zone du bas
        getContentPane().add(BorderLayout.SOUTH, zone_bas);
 
        //Création de la zone du centre
        JPanel zone_centre=new JPanel();
        zone_centre.setBackground(Color.white);
        //Placement de la zone du centre
        getContentPane().add(BorderLayout.CENTER, zone_centre);
 
    }
 
 
    public class BoutonForme extends JButton implements ActionListener{
 
        @Override
        public void actionPerformed(ActionEvent e) {
            //Action à effectuer sur click du bouton
            if (e.getSource()== btn_triangle){
                System.out.println("triangle");
            }else if (e.getSource()== btn_carre){
                System.out.println("carre");
            }else if (e.getSource()== btn_cercle){
                System.out.println("cercle");
            }
        }
 
    }
    }

Je vous remercie d'avance.

**joel.drigo** · 16/11/2013, 16h31

Salut,

déjà ta classe FenPrincipal ne doit pas compiler, parce que dans la méthode actionPerformed de ta classe BoutonForme tu essayes d'accèder à des variables locales créées dans le constructeur de ta classe (btn_triangle ou btn_carre).

On ne peut pas accéder à des variables locales à une méthode depuis une autre méthode (sauf s'il s'agit d'une méthode d'une classe locale à la méthode qui déclarent ces variables locales). Une façon de résoudre le problème est de déclarer les variables dans la classe FenPrincipal :

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
80
81
82
83
84
85
86
87
88
public class FenPrincipal extends JFrame {
 
    private JButton btn_triangle;
    private JButton btn_carre;
    private JButton btn_cercle;

    public static void main(String[] args) {
        //Description de la fenetre principal
        FenPrincipal fen=new FenPrincipal();
        fen.setTitle("Dessinons");
        fen.setSize(500,300);
        fen.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        fen.setVisible(true);
 
    }
    public FenPrincipal(){
        BoutonForme boutonforme=new BoutonForme();
        //Création de la zonne du haut
        JPanel zone_haut=new JPanel();
        //Création de boutons et libéllés
        btn_triangle=new JButton("Triangle");
        btn_carre=new JButton("Carré");
        btn_cercle=new JButton("Cercle");
        JButton btn_raz=new JButton("RAZ");
        JLabel lbl_forme=new JLabel("Forme : X");
        JLabel lbl_couleur=new JLabel("Coul. : X");
        //Ajout des boutons et libellés à la zonne du haut
        zone_haut.add(btn_triangle);
        zone_haut.add(btn_carre);
        zone_haut.add(btn_cercle);
        zone_haut.add(btn_raz);
        zone_haut.add(lbl_forme);
        zone_haut.add(lbl_couleur);
        //Enregistrement du recepteur d'action
        btn_triangle.addActionListener(boutonforme);
        btn_carre.addActionListener(boutonforme);
        btn_cercle.addActionListener(boutonforme);
        btn_raz.addActionListener(boutonforme);
        //Placement de la zone du haut
        getContentPane().add(BorderLayout.NORTH, zone_haut);
 
        //Création de la zone du bas
        JPanel zone_bas=new JPanel();
        //Création des boutons de la zone du bas
        JButton btn_bleu=new JButton("Bleu");
        btn_bleu.setBackground(Color.blue);
        JButton btn_jaune=new JButton("Jaune");
        btn_jaune.setBackground(Color.yellow);
        JButton btn_rouge=new JButton("Rouge");
        btn_rouge.setBackground(Color.red);
        JButton btn_vert=new JButton("Vert");
        btn_vert.setBackground(Color.green);
        JButton btn_noir=new JButton("Noir");
        btn_noir.setBackground(Color.black);
        //Ajout des boutons à la zone du bas
        zone_bas.add(btn_bleu);
        zone_bas.add(btn_jaune);
        zone_bas.add(btn_rouge);
        zone_bas.add(btn_vert);
        zone_bas.add(btn_noir);
        //Placement de la zone du bas
        getContentPane().add(BorderLayout.SOUTH, zone_bas);
 
        //Création de la zone du centre
        JPanel zone_centre=new JPanel();
        zone_centre.setBackground(Color.white);
        //Placement de la zone du centre
        getContentPane().add(BorderLayout.CENTER, zone_centre);
 
    }
 
 
    public class BoutonForme extends JButton implements ActionListener{
 
        @Override
        public void actionPerformed(ActionEvent e) {
            //Action à effectuer sur click du bouton
            if (e.getSource()== btn_triangle){
                System.out.println("triangle");
            }else if (e.getSource()== btn_carre){
                System.out.println("carre");
            }else if (e.getSource()== btn_cercle){
                System.out.println("cercle");
            }
        }
 
    }
    }

Par ailleurs, ta classe BoutonForme n'a pas besoin d'étendre JButton ; c'est un actionlistener, pas un bouton.

**pinps** · 16/11/2013, 17h17

Effectivement tu a raison, cela marche mieux comme sa.
Je te remercie de ton aide

**pinps** · 17/11/2013, 21h12

Bonjour,
J'ai maintenant un autre problème.
Mon but final est de faire une fenêtre dans laquelle je peut dessiner avec des carrés, des triangles et des cercles qui peuvent avoir la couleur bleu, jaune, rouge, vert ou noir.
Le problème est que j'ai l'erreur suivante et je ne comprend pas.

Exception in thread "AWT-EventQueue-0" java.lang.NullPointerException
at tp.dessinons.FenPrincipal$Dessin.paintComponent(FenPrincipal.java:84)
at javax.swing.JComponent.paint(JComponent.java:1054)
at javax.swing.JComponent.paintChildren(JComponent.java:887)
at javax.swing.JComponent.paint(JComponent.java:1063)
at javax.swing.JComponent.paintChildren(JComponent.java:887)
at javax.swing.JComponent.paint(JComponent.java:1063)
at javax.swing.JLayeredPane.paint(JLayeredPane.java:585)
at javax.swing.JComponent.paintChildren(JComponent.java:887)
at javax.swing.JComponent.paintToOffscreen(JComponent.java:5228)
at javax.swing.RepaintManager$PaintManager.paintDoubleBuffered(RepaintManager.java:1508)
at javax.swing.RepaintManager$PaintManager.paint(RepaintManager.java:1439)
at javax.swing.RepaintManager.paint(RepaintManager.java:1236)
at javax.swing.JComponent.paint(JComponent.java:1040)
at java.awt.GraphicsCallback$PaintCallback.run(GraphicsCallback.java:39)
at sun.awt.SunGraphicsCallback.runOneComponent(SunGraphicsCallback.java:78)
at sun.awt.SunGraphicsCallback.runComponents(SunGraphicsCallback.java:115)
at java.awt.Container.paint(Container.java:1967)
at java.awt.Window.paint(Window.java:3877)
at javax.swing.RepaintManager$3.run(RepaintManager.java:807)
at javax.swing.RepaintManager$3.run(RepaintManager.java:784)
at java.security.AccessController.doPrivileged(Native Method)
at java.security.ProtectionDomain$1.doIntersectionPrivilege(ProtectionDomain.java:76)
at javax.swing.RepaintManager.paintDirtyRegions(RepaintManager.java:784)
at javax.swing.RepaintManager.paintDirtyRegions(RepaintManager.java:757)
at javax.swing.RepaintManager.prePaintDirtyRegions(RepaintManager.java:706)
at javax.swing.RepaintManager.access$1000(RepaintManager.java:62)
at javax.swing.RepaintManager$ProcessingRunnable.run(RepaintManager.java:1647)
at java.awt.event.InvocationEvent.dispatch(InvocationEvent.java:251)
at java.awt.EventQueue.dispatchEventImpl(EventQueue.java:733)
at java.awt.EventQueue.access$200(EventQueue.java:103)
at java.awt.EventQueue$3.run(EventQueue.java:694)
at java.awt.EventQueue$3.run(EventQueue.java:692)
at java.security.AccessController.doPrivileged(Native Method)
at java.security.ProtectionDomain$1.doIntersectionPrivilege(ProtectionDomain.java:76)
at java.awt.EventQueue.dispatchEvent(EventQueue.java:703)
at java.awt.EventDispatchThread.pumpOneEventForFilters(EventDispatchThread.java:242)
at java.awt.EventDispatchThread.pumpEventsForFilter(EventDispatchThread.java:161)
at java.awt.EventDispatchThread.pumpEventsForHierarchy(EventDispatchThread.java:150)
at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:146)
at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:138)
at java.awt.EventDispatchThread.run(EventDispatchThread.java:91)

De plus je ne sait pas comment mi prendre pour afficher les formes.
Voila mon 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
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
package tp.dessinons;
 
import java.awt.BorderLayout;
import java.awt.Color;
import java.awt.Graphics;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.awt.event.MouseEvent;
import java.awt.event.MouseListener;
import java.util.ArrayList;
import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JPanel;
 
 
/**
 *
 * @author pinps
 */
public class FenPrincipal extends JFrame {
 
    private JButton btn_triangle;
    private JButton btn_carre;
    private JButton btn_cercle;
    private JButton btn_raz;
    private JLabel lbl_forme;
    private JLabel lbl_couleur;
    private JButton btn_bleu;
    private JButton btn_jaune;
    private JButton btn_rouge;
    private JButton btn_vert;
    private JButton btn_noir;
 
    public static void main(String[] args) {
        //Description de la fenetre principal
        FenPrincipal fen=new FenPrincipal();
        fen.setTitle("Dessinons");
        fen.setSize(520,300);
        fen.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        fen.setVisible(true);
 
    }
    public class Dessin extends JPanel implements MouseListener{
        ArrayList<Forme> liste=new ArrayList<>();
        public Dessin(){
            liste=null;
        }
        @Override
        public void mouseClicked(MouseEvent e){
            System.out.println("Click panel");
            e.getX();
            e.getY();
            Forme obj = new Forme();
            liste.add(obj);
 
        };
        @Override
        public void mouseEntered(MouseEvent e){};
        @Override
        public void mouseExited(MouseEvent e){};
        @Override
        public void mousePressed(MouseEvent e){};
        @Override
        public void mouseReleased(MouseEvent e){};
        public class Forme{
            String forme;
            Color couleur;
            Short x,y;
            public Forme(){
            forme="triangle";
            couleur=new Color(0,0,255);
            x=260;
            y=150;
            }
        }
        @Override
        public void paintComponent(Graphics g){
            super.paintComponent(g);
            for (Forme item : liste){
            g.setColor(Color.BLUE);
            g.fillRect(260,150,20,20);
            }
        }
    }
    public FenPrincipal(){
        //String forme="";
        BoutonForme boutonforme=new BoutonForme();
        //Création de la zonne du haut
        JPanel zone_haut=new JPanel();
        //Création de boutons et libéllés
        btn_triangle=new JButton("Triangle");
        btn_carre=new JButton("Carré");
        btn_cercle=new JButton("Cercle");
        btn_raz=new JButton("RAZ");
        JLabel lbl_formetxt=new JLabel("Forme : ");
        lbl_forme=new JLabel();
        JLabel lbl_couleurtxt=new JLabel("Couleur : ");
        lbl_couleur=new JLabel();
        //Ajout des boutons et libellés à la zonne du haut
        zone_haut.add(btn_triangle);
        zone_haut.add(btn_carre);
        zone_haut.add(btn_cercle);
        zone_haut.add(btn_raz);
        zone_haut.add(lbl_formetxt);
        zone_haut.add(lbl_forme);
        zone_haut.add(lbl_couleurtxt);
        zone_haut.add(lbl_couleur);
        //Enregistrement du recepteur d'action
        btn_triangle.addActionListener(boutonforme);
        btn_carre.addActionListener(boutonforme);
        btn_cercle.addActionListener(boutonforme);
        btn_raz.addActionListener(boutonforme);
        //Placement de la zone du haut
        getContentPane().add(BorderLayout.NORTH, zone_haut);
 
        //Création de la zone du bas
        JPanel zone_bas=new JPanel();
        //Création des boutons de la zone du bas
        btn_bleu=new JButton("Bleu");
        btn_bleu.setBackground(Color.blue);
        btn_jaune=new JButton("Jaune");
        btn_jaune.setBackground(Color.yellow);
        btn_rouge=new JButton("Rouge");
        btn_rouge.setBackground(Color.red);
        btn_vert=new JButton("Vert");
        btn_vert.setBackground(Color.green);
        btn_noir=new JButton("Noir");
        btn_noir.setBackground(Color.black);
        //Ajout des boutons à la zone du bas
        zone_bas.add(btn_bleu);
        zone_bas.add(btn_jaune);
        zone_bas.add(btn_rouge);
        zone_bas.add(btn_vert);
        zone_bas.add(btn_noir);
        //Enregistrement des recepteurs d'action
        btn_bleu.addActionListener(boutonforme);
        btn_jaune.addActionListener(boutonforme);
        btn_rouge.addActionListener(boutonforme);
        btn_vert.addActionListener(boutonforme);
        btn_noir.addActionListener(boutonforme);
        //Placement de la zone du bas
        getContentPane().add(BorderLayout.SOUTH, zone_bas);
 
        //Création de la zone du centre
        Dessin zone_centre=new Dessin();
        zone_centre.addMouseListener(zone_centre);
        zone_centre.setBackground(Color.white);
        //Placement de la zone du centre
        getContentPane().add(BorderLayout.CENTER, zone_centre);
 
 
    }
 
 
    public class BoutonForme extends JButton implements ActionListener{
 
        @Override
        public void actionPerformed(ActionEvent e) {
            //Action à effectuer sur click du bouton
            if (e.getSource()== btn_triangle){
                lbl_forme.setText("Triangle");
            }else if (e.getSource()== btn_carre){
                lbl_forme.setText("Carre");
            }else if (e.getSource()== btn_cercle){
                lbl_forme.setText("Cercle");
            }
            if (e.getSource()== btn_bleu){
                lbl_couleur.setText("Bleu");
            }else if (e.getSource()== btn_jaune){
                lbl_couleur.setText("Jaune");
            }else if (e.getSource()== btn_rouge){
                lbl_couleur.setText("Rouge");
            }else if (e.getSource()== btn_vert){
                lbl_couleur.setText("Vert");
            }else if (e.getSource()== btn_noir){
                lbl_couleur.setText("Noir");
            }
        }
 
    }
    }

Merci d'avance pour votre aide.

**joel.drigo** · 17/11/2013, 21h24

Dans ce bout de code :

Code :

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

1
2
3
4
ArrayList<Forme> liste=new ArrayList<>();
        public Dessin(){
            liste=null;
        }

tu déclares liste, tu lui affectes une instances de liste, et tu la vires dans le constructeur : du coup ta liste est null,

et donc lors du dessin l'appel de cette méthode va faire une NPE lors du for

Code :

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

1
2
3
4
5
6
7
8
@Override
        public void paintComponent(Graphics g){
            super.paintComponent(g);
            for (Forme item : liste){
            g.setColor(Color.BLUE);
            g.fillRect(260,150,20,20);
            }
        }

Pour dessiner des formes, autre que rectangles horizontaux ou verticaux, cercles et ellipses horizontales et verticales, du peux soit ;

- dessiner des polygones directement (Graphics.drawPolygon(int[], int[], int)
ce qui est suffisant pour tout polygone fermé (triangle par exemple)
- soit peut être plus compliqué pour toi, mais beaucoup plus libre, utiliser les shape

**pinps** · 17/11/2013, 21h48

Effectivement j'ai fait comme cela et plus d'erreur.

Code :

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

1
2
3
4
ArrayList<Forme> liste;
        public Dessin(){
            liste=new ArrayList<>();
        }

Merci

Action Listener qui ne fonctionne pas

avec Java

Vue hybride

Discussions similaires

Partager

Partager