My moving square leaving the boundaries of the panel is not noticed by Java

Question

Could someone please tell me why neither System.out.println("Out of bounds!") nor JOptionPane.showMessageDialog(null, "Out of bounds!") get reached?
Here is the code:
import java.awt.Graphics;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import javax.swing.JFrame;
import javax.swing.JOptionPane;
import javax.swing.JPanel;
import javax.swing.Timer;
public class Bounds extends JPanel
    int counter = 0, x = 50, y = 50, width = 20, height = 20;
    public Bounds()
        JFrame frame = new JFrame();
        frame.setVisible(true);
        frame.setSize(800,600);
        frame.setLocationRelativeTo(null);
        frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        frame.add(this);
        ActionListener listener = new graphicsListener();
        Timer timer = new Timer(2000, listener);
        timer.start();
    protected void paintComponent(Graphics g)
        super.paintComponent(g); // so the background gets refreshed
        g.drawRect(x+counter , y+counter, width+counter, height+counter);
        counter+=10;
        System.out.println("Panel width: " + getWidth() + " Panel height: " + getHeight());
        if(x+width >= getWidth() || y+height >= getHeight())
            System.out.println("Out of bounds!");
            JOptionPane.showMessageDialog(null, "Out of bounds!");
    public class graphicsListener implements ActionListener
        public void actionPerformed(ActionEvent event)
            repaint();
    public static void main(String[] args)
        new Bounds();
}Any help identifying the problem would be greatly appreciated!
Thanks in advance!

Answer 1

I changed it to this but still don't see why it's launching infinite JOptionPane windows:
import java.awt.Graphics;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import javax.swing.JFrame;
import javax.swing.JOptionPane;
import javax.swing.JPanel;
import javax.swing.Timer;
public class Bounds extends JPanel
    int x = 50, y = 50, width = 20, height = 20;
    Timer timer;
    public Bounds()
        JFrame frame = new JFrame();
        frame.setVisible(true);
        frame.setSize(800,600);
        frame.setLocationRelativeTo(null);
        frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        frame.add(this);
        ActionListener listener = new graphicsListener();
        timer = new Timer(200, listener);
        timer.start();
    protected void paintComponent(Graphics g)
        super.paintComponent(g); // so that the background gets refreshed
        g.drawRect(x, y, width, height);
        System.out.println("Panel width: " + getWidth() + " Panel height: " + getHeight());
        System.out.println("x+width = " + (x+width));
        System.out.println("y+height = " + (y+height));
        if(x+width >= getWidth() || y+height >= getHeight())
            System.out.println("Out of bounds!");
            JOptionPane.showMessageDialog(null, "Out of bounds!");
            timer.stop(); // Why isn't this having an effect?
        x+=10;
        y+=10;
        width+=20;
        height+=20;
    public class graphicsListener implements ActionListener
        public void actionPerformed(ActionEvent event)
            repaint();
    public static void main(String[] args)
        new Bounds();
}As indicated in the line that says: timer.stop(), I suspected that the timer kept invoking graphicsListener but I can't actually find the bug.