My GUI's catch block won't compile

Hi! This seems like a simple question, though maybe it belongs in the intermediate forum, but my code has a problem. When I try to compile, I get this message:

Java:37: paintBetterComponent(java.awt.Graphics) in CoolerSmear.MyCoolPanel cannot be applied to () youbrokeit.paintBetterComponent();
1 error

I was trying to modify one of the ready-bake-code classes from the book (the first animation) to make it re-smear when you click a button. Here's my code. I think the problem is in my catch block, ironically enough (serves me right for writing "System.out.println("Error at " + Math.random());", doesn't it?). Thanks in advance for your help!!

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

public class CoolerSmear implements ActionListener{
    int x = 70;
    int y = 70;
    public static void main (String[] Args){
        CoolerSmear gui = new CoolerSmear();
        gui.go();
    }
    public void go(){
        JFrame frame = new JFrame();
        JButton button = new JButton("gui.go();");
        button.addActionListener(this);
        frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        MyCoolPanel drawPanel = new MyCoolPanel();
        frame.getContentPane().add(BorderLayout.CENTER, drawPanel);
        frame.getContentPane().add(BorderLayout.SOUTH, button);
        frame.setSize(300,300);
        frame.setVisible(true);
        for (int i = 0; i < 130; i++) {
            x++;
            y++;
            drawPanel.repaint();
            try{
                Thread.sleep(50);
            } catch(Exception ex){                             
                ex.printStackTrace();
                System.out.println("Error at " + Math.random());
                MyCoolPanel youbrokeit = new MyCoolPanel();
                youbrokeit.paintBetterComponent();
            }
        }
    }
    class MyCoolPanel extends JPanel{
        public void paintComponent(Graphics g){
            g.setColor(Color.green);
            g.fillOval(x,y,40,40);
        }
        public void paintBetterComponent(Graphics g){
            Image image = new ImageIcon("upside.gif").getImage();
            g.drawImage(image,0,0,this);
        }
    }
    public void actionPerformed(ActionEvent event){
        CoolerSmear guii = new CoolerSmear();
        guii.go();
    }
}

it tells you the problem

public void paintBetterComponent(Graphics g){

and when called
youbrokeit.paintBetterComponent();

which needs a Graphics object

(and don't even think about using getGraphics())

Please always tell us which bookand use the CODE button, or your posting becomes illegible.

It needs a Graphics object, okay, but how do I do that? (And what's getGraphics()?)

And I didn't know that about the formatting, thanks. The book I referenced is Head First Java, second edition.

EJ Amber wrote:
And I didn't know that about the formatting, thanks..

You can use code tags for your existing post by clicking on the

button

I looked for that but I didn't see an edit button.

just to show you a major problem in your code, lets use getGraphics() so it will compile and run

change
youbrokeit.paintBetterComponent();
to
youbrokeit.paintBetterComponent(youbrokeit.getGraphics());

run the program and at any time, while the animation is on, or over, click the go button.

the app will freeze until the animation is finished - even try closing the app while the go button is still depressed.

hopefully this should prove why you should never have Thread.sleep() in the Swing thread (EDT)

and don't use getGraphics for other things - it also has major problems

Thread now more suitable for the Swing forum. Moving.

So, if I can't use Thread.sleep(), how do I keep the animation from moving too fast?

Just briefly, because I can't remember the details:

Start a new Thread (maybe with the SwingWorker class) and invoke Thread.sleep() in that Thread.

use a Swing.Timer, it'll do all the background threading for you