Discussion :

[mohamed996]

Bonsoir ,
Je viens juste de commencer avec le dessin 2D et donc besoin d'aide !
Mon but c'est que le programme dessine une seule ligne (x) à chaque fois que je clique sur jPanel2 (la même ligne sera redessinée ailleurs celons les cordonnées).
J'ai donc écrit ceci:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
java.awt.event.MouseListener l ;
java.awt.event.MouseEvent e ;
    Line2D x = new Line2D.Double(200, 150, l.mouseClicked(e.getX()), l.mouseClicked(e.getY()));
 
private void jPanel2MouseClicked(java.awt.event.MouseEvent evt) {                                     
 
        Graphics2D a = (Graphics2D) jPanel2.getGraphics();
        a.setColor(Color.BLACK);
        a.setStroke(new BasicStroke(2));
        a.draw(x);
}

Ceci dit , j'ai une erreur à la 3eme ligne du code , le programme souligne e.getX() et e.getY() en étant des erreurs.
Merci d'avance et bonne nuit à tout le monde :3

[plegat]

Bonjour,

Ceci dit , j'ai une erreur à la 3eme ligne du code , le programme souligne e.getX() et e.getY() en étant des erreurs.

Bah qu'est-ce qu'elle fait là cette ligne????

Le premier réflexe à avoir quand tu as une erreur, c'est de lire le message d'erreur... souvent il te dit ce qui cloche!

[mohamed996]

Bonjour,



Bah qu'est-ce qu'elle fait là cette ligne????

Le premier réflexe à avoir quand tu as une erreur, c'est de lire le message d'erreur... souvent il te dit ce qui cloche!

Re;
Ça dit qu'un Mouse Event est requit à la place de getX()/getY() qui sont des int(Pièce Jointe pour plus d'infos).
Enfin d'après ce que j'ai comprit , je suis qu'un débutant :p
Bonne journée !

[Pecose]

Moi pour avoir les coordonnées de ma souris je fait dans "public class Panel extends JPanel()":
private static int mX = 0, mY = 0;

puis dans paintComponent:
mX = MouseInfo.getPointerInfo().getLocation().x-getLocationOnScreen().x;
mY = MouseInfo.getPointerInfo().getLocation().y-getLocationOnScreen().y;

Avec le "import java.awt.MouseInfo;" qui va bien.
Et avec les getMX et getMY pour la beauté des yeux mais pas que.
Parce que je suis pas sur que ca marche le coup de e. getX() et e. getY().
Bon courage.

[joel.drigo]

Salut,

@Pecose, ça ne répond pas vraiment à la question posée. Déjà ton code est non évènementiel, et complètement hors contexte : le problème posé n'a pas vraiment de lien avec la souris mais avec le scope des variables et la façon d'organiser son code.

Reprenons le code de @Mohamed996

Code :

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

1
2
3
4
5
6
7
8
9
10
11
java.awt.event.MouseListener l ;
java.awt.event.MouseEvent e ;
    Line2D x = new Line2D.Double(200, 150, l.mouseClicked(e.getX()), l.mouseClicked(e.getY()));
 
private void jPanel2MouseClicked(java.awt.event.MouseEvent evt) {                                     
 
        Graphics2D a = (Graphics2D) jPanel2.getGraphics();
        a.setColor(Color.BLACK);
        a.setStroke(new BasicStroke(2));
        a.draw(x);
}

On voit que dans ce code, il y a une méthode d'écouteur de clic de souris, et qu'on créé la ligne à afficher en dehors de cette méthode, en espérant accéder aux coordonnées fournis par l'évènement. Vu que la variable evt n'est pas accessible en dehors de la méthode, on a créé une variable bidon non initialisée et on essaye d'invoquer sur une autre variable, pas plus initialisée, dont on invoque la méthode mouseClicked avec les hypothétiques coordonnées de l'évenement e (qui n'existe pas, puisque non initialisée) : or X comme Y sont des int, et la méthode mouseClicked a un argument de type MouseEvent : deux types totalement différents, d'où le message d'erreur précisé dans le second post.

Le bon code, remis dans le bon ordre :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
 
private void jPanel2MouseClicked(java.awt.event.MouseEvent evt) {                                     
 
        Graphics2D a = (Graphics2D) jPanel2.getGraphics();
 
        Line2D x = new Line2D.Double(200, 150, evt.getX(),  evt.getY());
        a.setColor(Color.BLACK);
        a.setStroke(new BasicStroke(2));
        a.draw(x);
 
}

Bon, ce code ne réalise pas vraiment ce qui est demandée dans la question initiale (on dessine en direct, mais au redessin la ligne sera perdue, donc non redessinée).

Il faudrait faire plutôt comme ça :

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
 public class Machin extends JPanel {
 
private Line2D x;
 
public Machin {
        x = new Line2D.Double(200, 150, 200,150);
        a.setColor(Color.BLACK);
        a.setStroke(new BasicStroke(2));
        addMouseListener((evt)-> jPanel2MouseClicked(evt)); // ou en java avant 8, addMouseListener(new MouseAdapter(){ public void mouseClicked(java.awt.event.MouseEvent evt){ jPanel2MouseClicked(evt);} ... });
}
 
public void paint(Graphics2D g) {
 
        g.draw(x);
}
 
 
private void jPanel2MouseClicked(java.awt.event.MouseEvent evt) {                                     
 
        x.setLine(200, 150, evt.getX(),  evt.getY());
        repaint();
 
}

[mohamed996]

Merci de votre réponse, malgré que c'est trop tard :p , je reconnais que mon code de la dernière fois n'avait aucun sens en faite