Shapes disappearing

Question

Alright, I figured out everything that I got but now I am really stuck. Every time you choose a different shape the previously selected one disappears. How do I make it so they don't disappear and stay on the screen until you exit?

import java.awt.*;
import java.awt.event.*;
import java.awt.geom.*;
import java.util.*;
import javax.swing.*;
public class ShapeStamper extends JFrame{
    Random rand = new Random();
    public int x;
    public int y;
    private JPanel panel1, panel2;
    private JButton button1, button2, button3, button4;
    private int option = 0;

public ShapeStamper(){
    super("Shape Stamper!");
    panel1 = new JPanel();
    button1 = new JButton("Circle");
    button2 = new JButton("Square");
    button3 = new JButton("Rectangle");
    button4 = new JButton("Oval");

    button1.addActionListener(
            new ActionListener(){
        public void actionPerformed(ActionEvent ae){
            option = 1;
                }
            }
    );
     button2.addActionListener(
            new ActionListener(){
        public void actionPerformed(ActionEvent ae){
            option = 2;
                }
            }
    );
     button3.addActionListener(
            new ActionListener(){
        public void actionPerformed(ActionEvent ae){
            option = 3;
                }
            }
    );
     button4.addActionListener(
            new ActionListener(){
        public void actionPerformed(ActionEvent ae){
            option = 4;
                }
            }
    );
  panel2 = new JPanel();
  panel2.setBackground(Color.WHITE);
  MouseHandler mouse = new MouseHandler();
  setVisible(true);
  addMouseListener(mouse);
  addMouseMotionListener(mouse);
  add(panel2);
  panel1.add(button1);
  panel1.add(button2);
  panel1.add(button3);
  panel1.add(button4);

  add(panel1, BorderLayout.SOUTH);
  setSize(500,500);
  setVisible(true);
          }
private class MouseHandler extends MouseAdapter implements MouseMotionListener{
        @Override
    public void mousePressed(MouseEvent e){
        x = e.getX();
        y = e.getY();

        repaint();
    }
}
public void paint(Graphics g){
    super.paintComponents(g);
    Graphics2D g2d = (Graphics2D) g;
    if(option == 0){
        g.setFont(new Font("Serif", Font.BOLD, 32));
        g.drawString("Shape Stamper!", 150, 220);
        g.setFont(new Font("Serif", Font.ITALIC, 16));
        g.drawString("Programmed by: Chris", 150, 230);
    }
    if(option == 1){
        Color randColor1 = new Color(rand.nextInt(256), rand.nextInt(256), rand.nextInt(256));
        g2d.setPaint(randColor1);
        g2d.drawOval(50, 50, 100, 100);
            }
    if(option == 2){
        Color randColor2 = new Color(rand.nextInt(256), rand.nextInt(256), rand.nextInt(256));
        g2d.setPaint(randColor2);
        g2d.drawRect(50, 50, 100, 100);
            }
    if(option == 3){
        Color randColor3 = new Color(rand.nextInt(256), rand.nextInt(256), rand.nextInt(256));
        g2d.setPaint(randColor3);
        g2d.draw(new Rectangle2D.Double(75,50,150,100));
            }
    if(option == 4){
        Color randColor4 = new Color(rand.nextInt(256), rand.nextInt(256), rand.nextInt(256));
        g2d.setPaint(randColor4);
        g2d.draw(new Ellipse2D.Double(50, 25, 100, 50));
            }
}
    public static void main(String[] args) {
        ShapeStamper application = new ShapeStamper();
        application.setDefaultCloseOperation(EXIT_ON_CLOSE);
    }
    }

Answer 1

You should not be overriding paint of a top level container and then trying to painting over the top of the components you have already added.

The basic problem you will encounter is, the paint system is clever enough that it may not ever call paint of the frame, but simply update the child components directly instead.

Instead, create yourself a custom component, extending from something like JPanel and override it's paintComponent method and perform your custom painting there. Then add this component to your frame.

You will also find the the paint updates are cleaner and won't flicker when updated.

You should also make sure you are calling repaint on this custom component when ever you change one it's options to ensure that changes are painted back to the component

Take a look at Performing Custom Painting for more details.

Also, just to be clear, you should not be calling super.paintComponents from paint (or in fact anywhere except for when you override paintComponents...which there really should be a need to do...)