I'm trying to change the Rectangle colour to Black but its not working

Question

I'm new to Java and don't know exactly what the cause.Let me explain the issue

I created a Rectangle Shape and its working, then i thought about changing its color to black for some testing but it seems not working below is my code. When i call the method from paintComponent itself then its working but if i do the same from any other method then its not changing the color. I tried calling the method repaint also but still the same

public class Meme extends JPanel {

Rectangle2D.Float myRect = new Rectangle2D.Float(90, 90, 90, 90);
Graphics2D graphics2d;

public void DRAW() {
    graphics2d.setColor(new Color(0, 0, 200));
    graphics2d.fill(myRect);
}

public void ChangeColour() {
    System.out.println("Called");
    graphics2d.setPaint(Color.BLACK);
    System.out.println("Called2");
    graphics2d.fill(myRect);
    System.out.println("Called3");
}

@Override
protected void paintComponent(Graphics g) {
    super.paintComponent(g);
    graphics2d = (Graphics2D) g;
    graphics2d.setRenderingHint(RenderingHints.KEY_ANTIALIASING, RenderingHints.VALUE_ANTIALIAS_ON);
    graphics2d.setRenderingHint(RenderingHints.KEY_TEXT_ANTIALIASING, RenderingHints.VALUE_TEXT_ANTIALIAS_ON);
    DRAW();
}

}

Button click listener method

private void jButton1ActionPerformed(java.awt.event.ActionEvent evt) {                                         
    meme1.ChangeColour();
} 

Answer 1

As far as I can remember, whenever you change some properties (color in this case), you have to call repaint. This will invoke a call to paintComponent and the frame will be drawn once again.

In your case, I am guessing even if you call repaint after changing color, the DRAW method gets called again in paintComponent which resets the changed color back to (0, 0, 200). Therefore, you don't see any change in the screen. But when you call changeColor in paintComponent method (assuming after the call to DRAW), the change of color persists and does not get overridden.

POSSIBLE SOLUTION

Just keep the color stored somewhere else. Like

Color myColor = new Color(0,0,200);

then in DRAW:

private void DRAW() {
    graphics2d.setColor(myColor);
    graphics2d.fill(myRect);
}

and in ChangeColor:

private void ChangeColour() {
    myColor = Color.BLACK;
}

Hope it helps.