Affichage d'un point au clic

Version imprimable

Bonsoir,
Je programme une application qui est censée dessiner un point lorsque l'utilisateur clique dans la fenetre. Mais je n'arrive pas à faire afficher mon point.
J'ai une JFrame qui contient un JPanel. Lors du clic le point est ajouté à une liste de points et repaint() est appelée, ce qui appelle la méthode redéfinie paint() du JPanel. J'ai du oublier un truc tout bete mais je n'arrive pas à corriger.
Merci de m'aider.

Code:

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
 
public class DemoGUI extends JFrame implements MouseListener {
 
    private PanelDessin panelDessin;
 
 
    public DemoGUI() {
        super();
        this.setSize(800, 600);
        this.setResizable(false);
 
 
 
        panelDessin = new PanelDessin(this);
        panelDessin.setMinimumSize(new Dimension(700, 600));
        panelDessin.setBorder(new LineBorder(Color.blue));
        this.getContentPane().add(panelDessin, BorderLayout.CENTER);
        this.setDefaultCloseOperation(DISPOSE_ON_CLOSE);
    }
 
 
    public static void main(String args[]) {
        DemoGUI f = new DemoGUI();
        f.setVisible(true);
    }
 
    public void mouseClicked(MouseEvent e) {
        panelDessin.ajouterPoint(e);
        this.repaint();
    }
 
    public void mousePressed(MouseEvent e) {
        throw new UnsupportedOperationException("Not supported yet.");
    }
 
    public void mouseReleased(MouseEvent e) {
        throw new UnsupportedOperationException("Not supported yet.");
    }
 
    public void mouseEntered(MouseEvent e) {
        throw new UnsupportedOperationException("Not supported yet.");
    }
 
    public void mouseExited(MouseEvent e) {
        throw new UnsupportedOperationException("Not supported yet.");
    }
}

Code:

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
 
public class PanelDessin extends JPanel{
 
    private ArrayList<Point> listePoints;
    private JFrame parent;
 
    public PanelDessin(JFrame parent) {
        super();
        this.parent = parent;
        listePoints = new ArrayList<Point>();
        this.setSize(700, 600);
        this.setVisible(true);
 
    }
 
    public void ajouterPoint(MouseEvent evt) {
        // on met dans le bon repere
        Point p = evt.getLocationOnScreen();
        p.setLocation(p.getX(), Math.abs(p.getY()-this.getHeight()));
        listePoints.add(p);
    }
 
 
    @Override
    public void paint(Graphics g) {
        super.paint(g);
        g.setColor(Color.red);
        int i =0;
        for (Point p : listePoints) {
            System.out.println("dessin du point "+i+" : "+p);
            g.drawRect(p.x, p.y+this.getHeight(), 5, 5);
            i++;
        }
    }

11/04/2010, 13h37
Ivelios

Utilises paintComponent(Graphics g) au lieu de paint(Graphics g)
11/04/2010, 18h57
vincbobor

j'avais aussi essayé, même problème...
11/04/2010, 19h10
nnovic

Tu n'aurais pas oublié de faire un

Code:

addMouseListener(this);

quelque part ?...

salut
surtout je ne comprend pas pk tu as fais

Code:

1
2
g.drawRect(p.x, p.y+this.getHeight(), 5, 5);

en plus , tu dois revoir ton cour sur les Listener en Java...

Code:

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
 
import java.awt.BorderLayout;
import java.awt.Color;
import java.awt.Dimension;
import java.awt.event.MouseEvent;
import java.awt.event.MouseListener;
 
import javax.swing.JFrame;
import javax.swing.border.LineBorder;
 
 
public class DemoGUI extends JFrame  {
 
    /**
         * 
         */
 
	private PanelDessin panelDessin;
 
 
    public DemoGUI() {
        super();
        this.setSize(800, 600);
       // this.setResizable(false);
        panelDessin = new PanelDessin(this);
        panelDessin.setMinimumSize(new Dimension(700, 600));
        panelDessin.setBorder(new LineBorder(Color.blue));
        this.getContentPane().add(panelDessin, BorderLayout.CENTER);
        this.setDefaultCloseOperation(DISPOSE_ON_CLOSE);
 
    }
 
 
    public static void main(String args[]) {
        DemoGUI f = new DemoGUI();
        f.setVisible(true);
    }	
}

Code:

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
 
import java.awt.Color;
import java.awt.Graphics;
import java.awt.Graphics2D;
import java.awt.Point;
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.JFrame;
import javax.swing.JPanel;
 
 
public class PanelDessin extends JPanel implements MouseListener{
 
    private ArrayList<Point> listePoints;
    private JFrame parent;
 
    public PanelDessin(JFrame parent) {
        super();
        this.parent = parent;
        listePoints = new ArrayList<Point>();
        this.setSize(700, 600);
        this.setVisible(true);
        addMouseListener(this);
    }
 
    public void paintComponent(Graphics g) {
    	super.paintComponent(g);
        g.setColor(Color.red);
        int i =0;
        for (Point p : listePoints) {
            System.out.println("dessin du point "+i+" : "+p);
            g.drawRect(p.x, p.y, 5, 5);
            i++;
        }
    }
 
 
	@Override
	public void mousePressed(MouseEvent arg0) {
		// TODO Auto-generated method stub
		   Point p = arg0.getPoint();
	        p.setLocation(p.getX(), Math.abs(p.getY()-this.getHeight()));
	        listePoints.add(p);
	        repaint();
 
	}
 
	public void mouseReleased(MouseEvent arg0) {}
	public void mouseExited(MouseEvent arg0) {}
	public void mouseClicked(MouseEvent arg0) {}
	public void mouseEntered(MouseEvent arg0) {}
}

12/04/2010, 14h27
vincbobor

salut,
merci pour ton aide, ça marche maintenant
Pour les coordonnées je changeais de repère car c'est plus facile pour les traitements derrière de travailler dans un repère 0,0 en bas à gauche.
Je me suis peut-être un peu embrouillé avec ça, mais je devrais y arriver :)
12/04/2010, 14h54
sinok

Accessoirement pour la redéfinition de la méthode paintComponent il est préférable d'utiliser sa signature originelle, c'est à dire protected void paintComponent et non public void paintComponent.