Problème avec du drag'n'drop de JLabel

Version imprimable

Bonjour,

J'essaye de faire un petit test de drag'n'drop d'un JLabel sur un JPanel.
J'ai donc ajouté un MouseMotionListener à mon JLabel, mais lorsque je déplace ce dernier, bizarrement, une seule image sur deux du JLabel est bonne au niveau des coordonnées (l'autre se trouve en bas à droite).

Voici les positions successives de mon JLabel lorsque je le bouge :
java.awt.Point[x=450,y=394]
java.awt.Point[x=250,y=248]
java.awt.Point[x=428,y=370]
java.awt.Point[x=236,y=230]
java.awt.Point[x=414,y=358]
java.awt.Point[x=224,y=220]

Je ne vois pourtant pas ce qui cloche dans mon mouseDragged :
Code:

1 2 3 4 5 6 7 8 9 public void mouseDragged(MouseEvent e) { Component source = (Component) e.getSource(); // mon JLabel, donc Point fLocation = f.getLocation(); // location de la frame sur l'écran int xos = e.getXOnScreen(); int yos = e.getYOnScreen(); source.setLocation(xos - fLocation.x - 4 + e.getX(), yos - fLocation.y - 24 + e.getY()); // 4 et 24 sont la différence entre la frame et le JPanel dp.repaint(); // apparemment inutile }
Qu'est-ce que j'ai oublié de prendre en compte dans mon calcul ? Quelqu'un a une idée ?

Merci d'avance !

19/12/2009, 17h23
Baptiste Wicht

Je suis pas un pro du drag and drop, mais il me semble bizarre que tu utilises getX() et getXOnScreen() pour ton calcul. Est-ce que tu ne pourrais pas utiliser seulement un des deux, ça serait plus clair.

Il me semble qu'un simple :

Code:

source.setLocation(e.getX(), e.getY());

suffit normalement, non ? Pourquoi tu as besoin de calculer la position sur l'écran ?
19/12/2009, 17h41
zzzer

Citation:

Envoyé par Baptiste Wicht

Je suis pas un pro du drag and drop, mais il me semble bizarre que tu utilises getX() et getXOnScreen() pour ton calcul. Est-ce que tu ne pourrais pas utiliser seulement un des deux, ça serait plus clair.

Il me semble qu'un simple :

Code:

source.setLocation(e.getX(), e.getY());

suffit normalement, non ? Pourquoi tu as besoin de calculer la position sur l'écran ?

Je veux en fait tenir compte de l'endroit du JLabel où l'utilisateur a cliqué.
Par exemple, s'il clique dans le quart inférieur droit du JLabel, le comportement ne devra pas être le même que s'il clique dans le quart supérieur gauche, car setLocation() positionne l'élément avec son coin supérieur gauche...

Citation:

Envoyé par zzzer

Je veux en fait tenir compte de l'endroit du JLabel où l'utilisateur a cliqué.

Essaie ceci:

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
import javax.swing.*;
import java.awt.event.*;
import java.awt.*;
public class DragComponentDemo extends JFrame{
    //The component to be dragged:
    private JLabel label = new JLabel("Hello world");
    public DragComponentDemo(){
        super("DragComponentDemo");
        setDefaultCloseOperation(WindowConstants.EXIT_ON_CLOSE);
        setSize(400, 300);
        setLocationRelativeTo(null);
        setLayout(null);
        label.setBounds(50, 50, label.getPreferredSize().width,
                label.getPreferredSize().height);
        label.setCursor(new Cursor(Cursor.MOVE_CURSOR));
        add(label);
        DragHandler dragHandler = new DragHandler();
        label.addMouseListener(dragHandler);
        label.addMouseMotionListener(dragHandler);
    }
    public static void main(final String... args){
        Runnable gui = new Runnable() {
            public void run() {
                new DragComponentDemo().setVisible(true);
            }
        };
        //GUI must start on EventDispatchThread:
        SwingUtilities.invokeLater(gui);
    }
}
class DragHandler extends MouseAdapter{
//We save the "press" point in "mousePressed", so we can translate the
//source location in "mouseDragged" relatively to the "press" point:
    private Point press;
    @Override
    public void mousePressed(final MouseEvent e) {
        press = e.getPoint();
    }
    @Override
    public void mouseDragged(final MouseEvent e) {
        Component source = (Component) e.getSource();
        Point location = source.getLocation();
        location.translate(e.getX() - press.x, e.getY() - press.y);
        source.setLocation(location);
    }
}

Citation:

Envoyé par uhrand

Essaie ceci:

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
import javax.swing.*;
import java.awt.event.*;
import java.awt.*;
public class DragComponentDemo extends JFrame{
    //The component to be dragged:
    private JLabel label = new JLabel("Hello world");
    public DragComponentDemo(){
        super("DragComponentDemo");
        setDefaultCloseOperation(WindowConstants.EXIT_ON_CLOSE);
        setSize(400, 300);
        setLocationRelativeTo(null);
        setLayout(null);
        label.setBounds(50, 50, label.getPreferredSize().width,
                label.getPreferredSize().height);
        label.setCursor(new Cursor(Cursor.MOVE_CURSOR));
        add(label);
        DragHandler dragHandler = new DragHandler();
        label.addMouseListener(dragHandler);
        label.addMouseMotionListener(dragHandler);
    }
    public static void main(final String... args){
        Runnable gui = new Runnable() {
            public void run() {
                new DragComponentDemo().setVisible(true);
            }
        };
        //GUI must start on EventDispatchThread:
        SwingUtilities.invokeLater(gui);
    }
}
class DragHandler extends MouseAdapter{
//We save the "press" point in "mousePressed", so we can translate the
//source location in "mouseDragged" relatively to the "press" point:
    private Point press;
    @Override
    public void mousePressed(final MouseEvent e) {
        press = e.getPoint();
    }
    @Override
    public void mouseDragged(final MouseEvent e) {
        Component source = (Component) e.getSource();
        Point location = source.getLocation();
        location.translate(e.getX() - press.x, e.getY() - press.y);
        source.setLocation(location);
    }
}

Merci beaucoup, ça marche parfaitement !
Problème résolu ;)

21/12/2009, 12h45
zzzer

uhrand, une petite question supplémentaire : y a-t-il une raison particulière pour créer une classe interne plutôt que de faire étendre ma classe principale ?
21/12/2009, 23h32
uhrand

Citation:

Envoyé par zzzer

uhrand, une petite question supplémentaire : y a-t-il une raison particulière pour créer une classe interne plutôt que de faire étendre ma classe principale ?

En réalité, "DragHandler" n'est pas une classe interne: tu peux facilement la placer dans un autre fichier source. Je ne trouve pas que c'est une bonne idée d'en faire une extension de la fenêtre principale "DragComponentDemo", parce qu’il s'agit d'un écouteur de souris et pas d'une fenêtre. C'est une sorte de "contrôleur" qui s'occupe des actions de la souris et des réactions de la vue.