Thread: Why isn't the MouseListener showing the X and Y cords like I want it to?

You can get the mouse position using this code - http://www.javaprogrammingforums.com...-position.html

Originally Posted by javapenguin

Why are all of these component's on the editor weird?

Also, the smiles are limited and there is a message popping up in yellow every once in while while I type that says "Auto-Saved" that I don't ever recall having seen before in the past.

I upgraded the forum software last night. The new editor is part of this upgrade.
There was a slight problem originally but this has been fixed.

As far as im aware, the "Auto-Saved" feature is for just incase the browser crashes or there is a problem when you are about to post. It saves your message as a draft.

I tried the code JavaPF suggested but it's not working.

In fact, it's not displaying anything at all now.

What I'm trying to do is use a Timer on the FountainAnimator class to keep repainting the JLabel in the ShowMouseLocation class to the position of the mouse inside the FountainAnimator class.

This was because I'm trying to make an analog clock and it'd help to figure out the positions instead of having to spend a long time guessing.

To ensure that the code will compile, I've given the entire program that I've had.

However, I'm convinced that the problem is within either the ShowMouseLocation class and/or is some glitch with the Timer in the FountainAnimator class or something else like calling the wrong repaint() somewhere inside the ShowMouseLocation class or something.

   import java.awt.*;
   import java.awt.event.*;
   import javax.swing.*;
   import javax.swing.JOptionPane;
   import java.util.*;
   import java.io.*;
   import javax.swing.Timer;
   import java.awt.MouseInfo;
   public class PaintDemo1 extends JFrame implements ActionListener
   {
      static final int HOUR = 0;
      static final int TWELVE = 12;
      static final int TWENTY_FOUR = 24;
      static final int THIRTY_SIX = 36;
      static final int FORTY_EIGHT = 48;
 
 
      private FountainAnimator animator;
      private Timer timer;
      private JScrollPane jsp;
      private int xLoc;
      private int xLoc2, xLoc3, xLoc4;
      private int yLoc,yLoc2,yLoc3,yLoc4;
      private int timesCalledCounter;
      private String time;
      private int hour, minute, second;
      private Timer clockTimer;
      private ClockListener cl;
     //  private int a;
      private double xLocation;
      private double yLocation;
      private AnalogClockListener acli;
 
 
 
      private class AnalogClockListener implements ActionListener
      {
 
         public void actionPerformed(ActionEvent e)
         {
 
 
         }
 
 
 
 
      }
 
      public class ShowMouseLocation extends JFrame implements ActionListener
      {
 
 
         private JLabel positions;
         private String str;
         public ShowMouseLocation()
         {
 
 
            positions = new JLabel();
            add(positions);
 
         }
 
 
         public void actionPerformed(ActionEvent e)
         {
 
            xLocation = MouseInfo.getPointerInfo().getLocation().x;
            yLocation = MouseInfo.getPointerInfo().getLocation().y;
 
 
 
            str = "<br>"+xLocation + "</br>" + "<br>" + yLocation + "</br>";
            System.out.println(str);
 
            repaint();
 
         }
 
         public void paint(Graphics g)
         {
            super.paint(g);
            positions.setText(str);
 
 
 
         }
      }
 
      private class ClockListener implements ActionListener
      {
 
 
 
         public void actionPerformed(ActionEvent e)
         {
            Calendar cal = Calendar.getInstance();
 
            int hourOfDay = cal.get(Calendar.HOUR_OF_DAY);
 
            minute = cal.get(Calendar.MINUTE);
            second = cal.get(Calendar.SECOND);
 
            if (hourOfDay < 12)
            { // beginning of if
               if (minute < 10)
               { // beginning of if
 
                  if (second < 10)
                  { // beginning of if
                     hour = hourOfDay;
                     time = hour + ":0" + minute + ":0" + second + " A.M.";
 
 
                  } // end if
                  else
                  { // begin else
                     hour = hourOfDay;
                     time = hour + ":0" + minute + ":" + second + " A.M.";
                  } // end else
 
               } // end if
 
               else
               { // begin else
                  if (second < 10)
                  { // begin if 
                     hour = hourOfDay;
                     time = hour + ":" + minute + ":0" + second + " A.M.";
                  } // end if
 
                  else
                  { // begin else
                     hour = hourOfDay;
                     time = hour + ":" + minute + ":" + second + " A.M.";
                  } // end else
 
               } // end else
 
            } // end if
 
            else if (hourOfDay == 12)
            {  // begin else if 
               if (minute < 10)
               { // beginning of if
 
                  if (second < 10)
                  { // beginning of if
                     hour = hourOfDay;
                     time = hour + ":0" + minute + ":0" + second + " P.M.";
 
 
                  } // end if
                  else
                  { // begin else
                     hour = hourOfDay;
                     time = hour + ":0" + minute + ":" + second + " P.M.";
                  } // end else
 
               } // end if
 
               else
               { // begin else
                  if (second < 10)
                  { // begin if 
                     hour = hourOfDay;
                     time = hour + ":" + minute + ":0" + second + " P.M.";
                  } // end if
 
                  else
                  { // begin else
                     hour = hourOfDay;
                     time = hour + ":" + minute + ":" + second + " P.M.";
                  } // end else
 
               } // end else
 
 
            } // end else if
 
            else
            { // begin else
               if (minute < 10)
               { // beginning of if
 
                  if (second < 10)
                  { // beginning of if
                     hour = hourOfDay - 12;
                     time = hour + ":0" + minute + ":0" + second + " P.M.";
 
 
                  } // end if
                  else
                  { // begin else
                     hour = hourOfDay - 12;
                     time = hour + ":0" + minute + ":" + second + " P.M.";
                  } // end else
 
               } // end if
 
               else
               { // begin else
                  if (second < 10)
                  { // begin if 
                     hour = hourOfDay - 12;
                     time = hour + ":" + minute + ":0" + second + " P.M.";
                  } // end if
 
                  else
                  { // begin else
                     hour = hourOfDay - 12;
                     time = hour + ":" + minute + ":" + second + " P.M.";
                  } // end else
 
               } // end else
            } // end else
 
            animator.repaint();
         } // end method
 
      } // end class
      public PaintDemo1()
      {
 
 
         timesCalledCounter = 0;
         setVisible(true);
         animator = new FountainAnimator();
         animator.setVisible(true);
 
         timer = new Timer(0, animator);
         timer.setDelay(1000);
         timer.start();
 
         cl = new ClockListener();
         clockTimer = new Timer(0,cl);
 
         clockTimer.setDelay(1000);
         clockTimer.start();
 
 
 
 
 
         jsp = new JScrollPane(animator, JScrollPane.VERTICAL_SCROLLBAR_ALWAYS  , JScrollPane.HORIZONTAL_SCROLLBAR_ALWAYS);
         Dimension d = new Dimension(9000,9000);
         animator.setPreferredSize(d);
         animator.revalidate();
         animator.setSize(d);
 
         animator.setAutoscrolls(true);
 
 
         setContentPane(jsp);
 
 
      }
 
 
 
      public static void main(String[] args)
      {
         PaintDemo1 pd1 = new PaintDemo1();
         pd1.setVisible(true);
 
      }
 
 
 
      public class FountainAnimator extends JPanel implements ActionListener, MouseListener 
      {
         private ShowMouseLocation sml;
         private Timer aTimer;
 
 
         public FountainAnimator()
         {
            super(new BorderLayout());
            setBackground(Color.GREEN);
            sml = new ShowMouseLocation();
            sml.setVisible(true);
 
            aTimer = new Timer(0,sml);
            aTimer.setDelay(100);
            System.out.println(xLocation);
            System.out.println(yLocation);
 
 
 
         }
 
         public void mouseClicked(MouseEvent e)
         {
 
 
         }
 
         public void mouseExited(MouseEvent e){}
         public void mouseEntered(MouseEvent e){}
         public void mouseReleased(MouseEvent e){}
         public void mousePressed(MouseEvent e){
            xLocation = e.getXOnScreen();
            yLocation = e.getYOnScreen();
            System.out.println(xLocation);
            sml.repaint();
         }
 
         private int getTimeInterval()
         {
            if (minute == HOUR)
               return HOUR;
 
 
            else if (minute == TWELVE)
               return TWELVE;
 
            else if (minute == TWENTY_FOUR)
               return TWENTY_FOUR;
 
            else if (minute == THIRTY_SIX)
               return THIRTY_SIX;
 
            else 
               return FORTY_EIGHT;
 
         }
 
         protected void paintComponent(Graphics g)
         {
            super.paintComponent(g);
 
            drawSidewalk(g);
            g.setColor(new Color(88,88,88));
            Polygon poly = new Polygon();
 
            poly.addPoint(200, 200);
            poly.addPoint(400, 200);
            poly.addPoint(400, 300);
            poly.addPoint(100, 300);
            g.fillPolygon(poly);
 
            g.setColor(Color.BLACK);
 
            g.drawPolygon(poly);
 
            Polygon poly2 = new Polygon();
            poly2.addPoint(200, 200);
            poly2.addPoint(250, 100);
            poly2.addPoint(450, 110);
            poly2.addPoint(400, 200);
            g.setColor(Color.BLUE);
            g.fillPolygon(poly2);
 
            g.setColor(Color.BLACK);
            g.drawLine(200, 200, 250, 100);
            g.drawLine(250, 100, 450, 110);
            g.drawLine(450, 110, 400, 200);
 
 
            Polygon poly3 = new Polygon();
 
            poly3.addPoint(100, 300);
            poly3.addPoint(100, 200);
            poly3.addPoint(250, 100);
            poly3.addPoint(200, 200);
 
            g.setColor(new Color (88, 88,88));
            g.fillPolygon(poly3);
 
            g.setColor(Color.BLACK);
            g.drawLine(100, 300, 100, 200);
            g.drawLine(100, 200, 250, 100);
 
            Polygon poly4 = new Polygon();
            g.setColor(new Color(88,88,88));
 
            poly4.addPoint(450, 110);
            poly4.addPoint(510, 270);
            poly4.addPoint(400,300);
            poly4.addPoint(400, 200);
            g.fillPolygon(poly4);
            g.setColor(Color.BLACK);
            g.drawPolygon(poly4);
 
 
 
 
            drawFountain(g);
 
 
            animateFountain(g);
 
            drawLake(g);
 
            drawDigitalClock(g);
 
            drawRegularClock(g);
 
         }
 
         public void drawParkShelter(Graphics g)
         {
 
         }
 
 
         public void drawFountain(Graphics g)
         {
            g.setColor(new Color(95,95,95));
            g.drawOval(300, 120, 65, 65);
            g.setColor(Color.BLACK);
            g.drawOval(320, 140, 25, 25);
            g.setColor(new Color(95,95,95));
            g.fillOval(320, 140, 25, 25);
 
            g.setColor(new Color(50,50,50));
            Polygon poly = new Polygon();
            poly.addPoint(325, 153);
            poly.addPoint(325, 125);
            poly.addPoint(330, 115);
            poly.addPoint(340, 125);
            poly.addPoint(340, 153);
            g.fillPolygon(poly);
 
            g.setColor(Color.BLACK);
            g.drawPolygon(poly);
 
 
         }
 
         public void drawSidewalk(Graphics g)
         {
 
            g.setColor(new Color(183,183,183));
            g.fill3DRect(100,50,420,300,true);
            g.setColor(Color.BLACK);
            g.draw3DRect(100,50,420,300, true);
 
 
 
 
         }
 
 
         public void drawRegularClock(Graphics g)
         {
            g.setColor(Color.BLACK);
            g.drawOval(3000,3000, 150,150);
            g.fillOval(2990,2990,170,170);
            g.setColor(Color.WHITE);
            g.fillOval(3000,3000,150,150);
            g.setColor(Color.BLACK);
            g.setFont(new Font("Arial Bold", Font.BOLD, 17));
            g.drawString("12", 3065, 3015);
            g.setColor(Color.DARK_GRAY);
            g.drawLine(3072, 3020, 3072, 3005);
            g.setColor(Color.BLACK);
            g.drawString("6", 3071, 3150);
            g.setColor(Color.DARK_GRAY);
            g.drawLine(3073,3135, 3073, 3150);
 
 
 
 
 
         }
         public void drawDigitalClock(Graphics g)
         {
 
 
            Polygon poly = new Polygon();
 
            poly.addPoint(1000,2485);
            poly.addPoint(982, 2525);
            poly.addPoint(1189,2525);
            poly.addPoint(1172, 2485);
 
            g.setColor(Color.BLACK);
            g.fillPolygon(poly);
            g.setColor(Color.LIGHT_GRAY);
            g.drawPolygon(poly);
            g.setColor(Color.YELLOW);
            g.setFont(new Font("Arial Bold Italic", Font.BOLD + Font.ITALIC, 27));
            g.drawString(time, 1003,2510);
 
 
         }
 
 
 
 
 
         public void animateFountain(Graphics g)
         {
 
 
 
 
            g.setColor(new Color(0,159,206));
            g.fillOval(xLoc,yLoc, 10, 10);
            g.fillOval(xLoc2,yLoc2,10,10);
            g.fillOval(xLoc3,yLoc3, 10,10);
            g.fillOval(xLoc4, yLoc4, 10,10);
            g.setColor(Color.BLACK);
            g.drawOval(xLoc, yLoc, 10,10);
            g.drawOval(xLoc2, yLoc2, 10,10);
            g.drawOval(xLoc3, yLoc3, 10,10);
            g.drawOval(xLoc4, yLoc4, 10,10);
 
 
 
 
         // repaint();
         }
 
 
         public void drawLake(Graphics g)
         {
            g.setColor(new Color(9,34, 136));
            g.fillOval(500,500, 300, 200);
            g.setColor(Color.BLACK);
            g.drawOval(500,500,300,200);
 
 
 
 
 
         }
 
 
         public void actionPerformed(ActionEvent e)
         {
 
 
 
 
            if (timesCalledCounter ==0)
            {
 
               xLoc =  325;
               yLoc = 125;
 
 
 
 
               timesCalledCounter++;
            }
 
            else if (timesCalledCounter ==1)
            {
 
 
               xLoc = 350;
               yLoc = 138;
               timesCalledCounter++;
            }
 
            else if (timesCalledCounter ==2)
            {
 
 
               xLoc =373;
               yLoc =  150;
 
 
 
 
               timesCalledCounter++;
            }
 
            else
            {
 
 
               xLoc = 385;
               yLoc =  170;
               timesCalledCounter = 0;
            }
 
 
 
            animator.repaint();
 
 
 
         }
 
      }
 
      public void actionPerformed(ActionEvent e)
      {
        // animator.repaint();
      	// Do I even need this listener?
      }
 
 
 
 
   }

The problem wouldn't be caused by my setting autoScrolls to true for the FountainAnimator object, would it?

I don't know what autoscrolls are but what I had been looking for when I wrote that part was to get the thing to resize withiout adding any new components to it, but I figured out that revalidate() was what I was looking for for that problem.

Is this possibly a contentPane issue for ShowMouseLocation as I didn't ever call setContentPane() ?

The ClockListener and stuff like that are only there because in the earlier, smaller code, I must have either forgotten a bracket and/or not erased a variable that refereed to one of those removed classes.

Also, the MouseListeners in FountainAnimator no longer are needed. They're not the problem, I think, either.

Also, the methods like drawLake() and such are not important to the problem being discussed. They are only there to ensure that the code will compile.

The most likely spots that the problem is in are:

 public class ShowMouseLocation extends JFrame implements ActionListener
      {
 
 
         private JLabel positions;
         private String str;
         public ShowMouseLocation()
         {
 
 
            positions = new JLabel();
            add(positions);
 
         }
 
 
         public void actionPerformed(ActionEvent e)
         {
 
            xLocation = MouseInfo.getPointerInfo().getLocation().x;
            yLocation = MouseInfo.getPointerInfo().getLocation().y;
 
 
 
            str = "<br>"+xLocation + "</br>" + "<br>" + yLocation + "</br>";
            System.out.println(str);
 
            repaint();
 
         }
 
         public void paint(Graphics g)
         {
            super.paint(g);
            positions.setText(str);
 
 
 
         }
      }

and

 public class FountainAnimator extends JPanel implements ActionListener, MouseListener 
      {
         private ShowMouseLocation sml;
         private Timer aTimer;
 
 
         public FountainAnimator()
         {
            super(new BorderLayout());
            setBackground(Color.GREEN);
            sml = new ShowMouseLocation();
            sml.setVisible(true);
 
            aTimer = new Timer(0,sml);
            aTimer.setDelay(100);
            System.out.println(xLocation);
            System.out.println(yLocation);
 
 
 
         }

Hmmmmm...also thought of something.

It appears that most of my other Timers are inside the PaintDemo1 constructor and the Timer I'm using for the MouseLocation thingy is not. Could that be the problem?

Also, I don't think I'm setting it to track the mouse position of the FountainAnimator thing like I wanted the code to but don't know how to do that exactly.

It is working.

It just starts out minimized.

I don't know what's causing that either.

Can't fix that.

Try enlarging it.

Perhaps it's because I'm using JGrasp to compile this or something like that.

I can't figure out how to start it out maximized.

Ok, but how do I get it to show the position relative to the Mouse location on FountainAnimator?

So far it's just showing the location relative to the screen.

   import java.awt.*;
   import java.awt.event.*;
   import javax.swing.*;
   import javax.swing.JOptionPane;
   import java.util.*;
   import java.io.*;
   import javax.swing.Timer;
   import java.awt.MouseInfo;
   public class PaintDemoX extends JFrame
   {
 
 
 
      private FountainAnimator animator;
      private Timer timer;
      private JScrollPane jsp;
      private int xLoc;
      private int xLoc2, xLoc3, xLoc4;
      private int yLoc,yLoc2,yLoc3,yLoc4;
      private int timesCalledCounter;
      private String time;
      private int hour, minute, second;
   //   private Timer clockTimer;
     // private ClockListener cl;
     //  private int a;
      private double xLocation;
      private double yLocation;
   //   private AnalogClockListener acli;
 
 
 
 
 
      public class ShowMouseLocation extends JFrame implements ActionListener
      {
 
 
         private JLabel positions;
         private String str;
         private String str2;
         public ShowMouseLocation()
         {
 
            Dimension dim2 = new Dimension(50,50);
            setMinimumSize(dim2);
            positions = new JLabel();
            add(positions);
 
         }
 
 
         public void actionPerformed(ActionEvent e)
         {
 
 
         /* I had hoped that these two methods calls for xLocation and yLocation would be getting the value of the mouse x and y
         locations in the FountainAnimator object animator, or any FountainAnimator object perhaps for that matter(at least animator at the very least).
         Anyway, I had hoped that it would show this location in the other JFrame.  However, the JFrame has a hard time staying open at the same time as my main JFrame.  If I minimize them both to really small size, then I can show both at same time, otherwise clicking on one of the JFrames makes the other automatically minimize.  But that's not the real issue here, though it certainly is annoying.  Actually, what I had hoped for was to originally show
         the mouse pointer in a JPanel thing that would be placed below the FountainAnimator object and then combine thse two into a single panel that shows both or something.  Kinda like how in JGrasp and other compilers, it shows the position where the key is ready to type, the vertical bar thingy that shows where you're about to type, anyway, it shows its position below in a smaller JPanel or whatever.  Anyway, I tried that and it wouldn't work quite well as one of the panels appeared too small or  something.  As it is, I am trying it this way unless you could show me how to use the layouts to attach a small JPanel thing that only has the JLabel on it to the bottom of the FountainAnimator object or something like that.  */
 
 
 
            MouseEvent me = new MouseEvent(animator, MouseEvent.MOUSE_CLICKED, 0, MouseEvent.BUTTON1, 0,0, 1, false);
 
            xLocation = animator.getMousePosition().getX();
            yLocation = animator.getMousePosition().getY();
 
            Double xL = xLocation;
            Double yL = yLocation;
            str = xL.toString();
            str2 = yL.toString();
 
            System.out.println(str);
 
            repaint(); // which repaint() am I calling: the PaintDemo1 or the ShowMouseLocation paint method?
 
         }
 
         public void paint(Graphics g)
         {
            super.paint(g);
            positions.setText("<html><br>"+str+ "</br> <br>" + str2 + "</br></html>");
            // why is str blank?
 
 
 
         }
      }
 
 
      public PaintDemoX()
      {
 
 
         timesCalledCounter = 0;
         setVisible(true);
         animator = new FountainAnimator();
         animator.setVisible(true);
         animator.setOpaque(true);
         Dimension dim = new Dimension(200,200);
         animator.setMinimumSize(dim);
         this.setMinimumSize(dim);
 
 
         jsp = new JScrollPane(animator, JScrollPane.VERTICAL_SCROLLBAR_ALWAYS  , JScrollPane.HORIZONTAL_SCROLLBAR_ALWAYS);
         Dimension d = new Dimension(9000,9000);
         animator.setPreferredSize(d);
         animator.revalidate();
         animator.setSize(d);
 
         animator.setAutoscrolls(true);
 
 
         setContentPane(jsp);
 
         setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
      }
 
 
 
      public static void main(String[] args)
      {
         PaintDemoX pd1 = new PaintDemoX();
         pd1.setVisible(true);
 
      }
 
 
 
      public class FountainAnimator extends JPanel 
      {
         private ShowMouseLocation sml;
         private Timer aTimer;
 
 
         public FountainAnimator()
         {
            super(new BorderLayout());
            setBackground(Color.GREEN);
            sml = new ShowMouseLocation();
            sml.setVisible(true);
 
            aTimer = new Timer(0,sml);
            aTimer.setDelay(100);
            aTimer.start();
            System.out.println(xLocation);
            System.out.println(yLocation);
 
 
 
         }
 
 
 
 
 
      }
 
 
 
 
 
 
   }

animator.getMousePosition().getX() is returning null.

I'm trying to get the position of the mouse relative to the pixel position of the mouse inside the animator object.

The FountainAnimator object.

BTW, making a MouseEvent object and trying that route always yielded the Point (0,0).