MouseListener and coordinate display

[nick woodward]

I was just wondering if anyone could help me check this code please.

It's supposed to display the coordinates of the mouse on a JLabel when clicked, but it doesn't register if the mouse is moved even slightly while clicking. I've written vaguely similar code (but storing the dimensions locally and then calling repaint on the JFrame) with no such problem. Is there any reason why setting the text of a label should behave differently, or have I made a mistake somewhere?

import javax.swing.*;
import java.awt.event.*;
import java.awt.GridLayout;

public class BannerCoordinates {

	public static void main(String[] args){
		GUI gui = new GUI();
	}
}

class GUI extends JFrame{

	private JLabel lab = new JLabel("Where is your mouse?", SwingConstants.CENTER);
	private JLabel lab2 = new JLabel("", SwingConstants.CENTER);

	public GUI(){
		super("Your Mouse Is At:");
		setSize(300, 300);
		setLayout(new GridLayout(2,1));
		add(lab);
		add(lab2);
		setVisible(true);

		addMouseListener(new MListener(this));
	}

	public void setLab(String s){
		lab.setText(s);
	}

	public void setLab2(String s){
		lab2.setText(s);
	}
}

class MListener extends MouseAdapter{
	GUI frame;

	public MListener(GUI f){
		frame = f;
	}

	public void mouseClicked(MouseEvent e){
		frame.setLab("Your mouse is at:");
		frame.setLab2("" + e.getX() + ", " + e.getY());
	}
}

am happy to paste the other code too if needed.

thanks,

Nick

[Piet Souris]

hi Nick,

you are using 'mouseClicked'. That means you only get to hear
of a mouse click when the mouse button is released. Now,
if there is some distance between the location when the mouse
button was pressed and the location where it is released, then
mouseClicked is not invoked at all.

So, if you want some accuracy, use the methods 'mousePressed'
and 'mouseReleased'.

[nick woodward]

Piet Souris wrote:hi Nick,

you are using 'mouseClicked'. That means you only get to hear
of a mouse click when the mouse button is released. Now,
if there is some distance between the location when the mouse
button was pressed and the location where it is released, then
mouseClicked is not invoked at all.

So, if you want some accuracy, use the methods 'mousePressed'
and 'mouseReleased'.

thanks a lot! - i had completely missed that i'd used mousePressed in the other code!

Nick