Trying to use Graphics draw methods

[Lang]

I'm having trouble drawing simple graphics.

This is in my Shuffleboard class

Code:

Board board = new Board("Shuffleboard");
board.setLocation(100, 100);
board.setResizable(false);
board.setVisible(true);

This is in my Board class

Code:

boardCanvas = new BoardCanvas();
this.add(boardCanvas);

This is in my BoardCanvas class

Code:

super.paintComponent(g);
		
g.setColor(Color.black);
for (Weight w : weights){
	g.fillOval(200, 200, 50, 50);
	//g.drawString("Test", 100, 100);
	System.out.println("Bleh");
}

I know its executing because it is printing Bleh to the console. But no oval is being drawn. There appears to be no error.

My BoardCanvas class extends JPanel - and my Shuffleboard class extends JFrame. Would this be an issue?

Any help is appreciated,
Lang

[Lang]

Ok - I fixed the graphics issues.

However, I can't get the graphics to clear with dispose()

Am I doing something wrong? Is it a different function?

[hardwired]

Code:

public class ShuffleBoard extends JFrame {
    // If Board is a model class and not part
    // of the componeent hierarchy.
    Board board = new Board("Shuffleboard");
    BoardCanvas boardCanvas;

    ShuffleTest() {
        boardCanvas = new BoardCanvas();
        this.add(boardCanvas);
        board.seLocation(100, 100);
        board.setResizable(false);
        board.setVisible(true);
    }
}

class Board {
    // This could be some kind of model class.
    // If so you may not need a reference to
    // it in the ShuffleBoard class.
    // If it is designed as part of the component
    // hierarchy see below.
}

class BoardCanvas extends JPanel {

    protected void paintComponent(Graphics g) {
        super.paintComponent(g);
        g.setColor(Color.black);
        for (Weight w : weights) {
            g.fillOval(200, 200, 50, 50);
//            g.drawString("Test", 100, 100);
            System.out.println("Bleh");
        }
    }
}

If Board is part of the component hierarchy:

Code:

public class ShuffleTest extends JFrame {
    Board board = new Board("Shuffleboard");

    ShuffleTest() {
        this.add(board);
        board.seLocation(100, 100);
        board.setResizable(false);
        board.setVisible(true);
    }
}

class Board extends Some_JComponent {
    BoardCanvas boardCanvas;

    Board() {
        boardCanvas = new BoardCanvas();
        this.add(boardCanvas);
    }   
}

[hardwired]

can't get the graphics to clear with dispose()
Don't use dispose to clear graphics.
Set up your painting methods to draw the current state of its enclosing class (the graphic component). Control the state of this class from your event code in this and/or other classes.

[Lang]

I'm having trouble understanding what you said, hardwired.

When the mouse is released I'd like all the graphics to disappear.

So I tried this:

Code:

public void mouseReleased(MouseEvent e){		
	this.remove(boardCanvas);
				
	startShot = null;
	endShot = null;
}

startShot and endShot are two irrelevant Point variables.

[hardwired]

Then try adding a call to revalidate

Code:

this.remove(boardCanvas);
revalidate()

which tells the parent component to do a new layout.