Paint method is covering other layouts

[Michael Carol]

Hello, I have just started learning AWT and Swing so I've done my first application. It works fine except one part. Application draws shapes which user choose by clicking button. The problem is that paint method is covering all the rest. Moreover at the start of the application there is a transparent background and nothing is visible until I move mouse cursor above one of the hidden buttons and click it. Also neither labels nor panels are visible. If you cut this method from code menu is visible and looks like it should:

public void paint(Graphics g) {
        if(rectangle){
        g.setColor(Color.YELLOW);
        g.fillRect(100, 350 ,180, 110);
        }
        if(oval){
        g.setColor(Color.BLUE);
        g.fillOval(350, 400 ,140, 140);
        }
        if(line){
        g.setColor(Color.GREEN);
        g.drawLine(650, 450 ,140, 125);
        }
        }

This is strange because it shouldn't even work at start of the application because flags are set on false. Please take a look at this and give me a hint

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

public class Shapes extends JFrame implements ActionListener{
    private JLabel label1;
    private JButton b1, b2, b3;
    private JPanel p = new JPanel(new BorderLayout());
    
    public Shapes(){
    setSize(800,600);
    setTitle("Shapes");
    setLayout(null);
    
    b1 = new JButton("RECTANGLE");
    b2 = new JButton("OVAL");
    b3 = new JButton("LINE");
    
    b1.setBounds(40, 25, 130, 20);
    add(b1);
    b1.addActionListener(this);
    b2.setBounds(250, 25, 100, 20);
    add(b2);
    b2.addActionListener(this);
    b3.setBounds(440, 25, 100, 20);
    add(b3);
    b3.addActionListener(this);
    
    p.setLayout(null);
    p.setLocation(50, 30);
    p.setSize(600, 70);
    p.setBackground(Color.red);
    add(p);
    p.add(b1);
    p.add(b2);
    p.add(b3);
    
    label1 = new JLabel("Choose the shape");
    label1.setBounds(300, 100, 500, 20);
    add(label1);
    }
    private boolean rectangle = false;
    private boolean oval = false;
    private boolean line = false;
        
        public void paint(Graphics g) {
        if(rectangle){
        g.setColor(Color.YELLOW);
        g.fillRect(100, 350 ,180, 110);
        }
        if(oval){
        g.setColor(Color.BLUE);
        g.fillOval(350, 400 ,140, 140);
        }
        if(line){
        g.setColor(Color.GREEN);
        g.drawLine(650, 450 ,140, 125);
        }
        }
        public void actionPerformed(ActionEvent e){
            if(e.getSource() == b1){
            rectangle = true;
            label1.setText("Rectangle");
            repaint();
            }
            if(e.getSource() == b2){
            oval = true;
            label1.setText("Oval");
            repaint();
            }
            if(e.getSource() == b3){
            line = true;
            label1.setText("Line");
            repaint();
            }
         }
    public static void main(String[] args){
        Shapes frame = new Shapes();
        frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        frame.setVisible(true);
    }
}

[Greg Brannon]

I don't see a print() method. What print() method?

Your program seems to work fine. How about adding a Clear button?

[Michael Carol]

I'm sorry it was a typo. I was talking about paint method above.

So when I open my app it looks like this (everything is transparent):



And I must move my mouse cursor above each button to show them.



No labels and panels are available either. However on my other PC with WinXP labels and panels are shown after moving cursor above the button. I really have no idea why this happens :/

Without paint method menu looks fine and app starts ok but when I add paint method it start to be messed up :/

[Greg Brannon]

Try starting your paint() method with:

super.paint( g );

You should find that the accepted/recommended approach in your study material.

[Michael Carol]

Thank you very much for your help - it works perfect now