paintComponent function problem

[max1]

Hi guys I'm learing about Graphic in Java and I have a problem , because my paintComponent
doesn't work I mean that complier doesn't see ma function and program is performed without paint function



import java.awt.BorderLayout;
import java.awt.Color;
import java.awt.Graphics;
import javax.swing.JFrame;
import javax.swing.JPanel;

public class Figure extends JPanel{

private JPanel panel;
Figure()
{
setLayout( new BorderLayout() );
panel = new JPanel();
panel.setBackground(Color.YELLOW);
add(panel, BorderLayout.CENTER);
}

public void paintComponent (Graphics g )
{
super.paintComponent(g);
g.setColor(Color.RED);
g.drawRect(45, 45, 45, 45);
}

public static void main( String [] args )
{
JFrame f = new JFrame("Name");

Figure k = new Figure();
f.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
f.setSize(300, 200);
f.setVisible(true);
f.add(k);
}
}

[doWhile]

Hi guys I'm learing about Graphic in Java and I have a problem , because my paintComponent
doesn't work I mean that complier doesn't see ma function and program is performed without paint function

How do you know it "doesn't work"? Add some println's in the paintComponent method and see if the what you add is printed to the console.

[max1]

How do you know it "doesn't work"? Add some println's in the paintComponent method and see if the what you add is printed to the console.

I tried but sill is yellow panel without any drawing text or shapes

[doWhile]

I tried but sill is yellow panel without any drawing text or shapes

Adding a System.out.println statement won't fix it...it lets you know whether paintComponent is being called or not and is an invaluable tool in stepping through and debugging code...in your case you think paintComponent isn't being called - this will tell you whether it is or not. Post back the result. Whether it does or not will indicate what you should do to fix the problem.

[DarrylBurke]

Add components to a GUI hierarchy before, rather than after, making it visible.

// f.setVisible(true);
 f.add(k);
f.setVisible(true);

db

[max1]

"Adding a System.out.println statement won't fix it..."
"Add components to a GUI hierarchy before, rather than after, making it visible.

Java Code:

// f.setVisible(true);
f.add(k);
f.setVisible(true);"

I tried both this methods but it wasn't change anything , but when I changed paintComponent() to paint(), it works correctly drawing this things what I want,
but all this stuff are drawing at default JFrame background rather then yellow JPanel

[doWhile]

I tried both this methods but it wasn't change anything , but when I changed paintComponent() to paint(), it works correctly drawing this things what I want,
but all this stuff are drawing at default JFrame background rather then yellow JPanel

Did you read my post? Adding println' won't fix it, but give you information to help you 'fix' it. And yes, changing paintComponent to paint will 'fix' the problem given the inherent way that paint works (calling super.paint at the start of the paint method will paint the children before you paint over the children - one of the reasons its advised not to override the paint method). I am not sure why you are adding a JPanel to make the background Yellow instead of just painting yellow.

[max1]

I am not sure why you are adding a JPanel to make the background Yellow instead of just painting yellow.

I want to do a simple game which will consist of to parts first - label (JLabel) which will be show a score and second part will be (JPanel) area where when a player click on viewing element they change result on a label score. But now I am not sure if I must put this two part together to JFrame or it is anther way to do this.

[doWhile]

I want to do a simple game which will consist of to parts first - label (JLabel) which will be show a score and second part will be (JPanel) area where when a player click on viewing element they change result on a label score. But now I am not sure if I must put this two part together to JFrame or it is anther way to do this.

If I understand correctly, then combine the components using an appropriate LayoutManager. It should not be necessary to rely on Components painting over one another as the example above demonstrates, rather create each one and rely on a LayoutManager to place them accordingly.