drawing multiple graphics on the same JLabel

Question

I am trying to draw a point, a polygon and a rectangle on the same JLabel but not really sure how to handle the paintComponent method to do so. Currently I am able to draw a point and polygon separately using 2 different classes that extends label, but when I try to put them in the same class because I need both of them on the same label, it starts to draw the polygon and does not allow to draw a point separately.

Here is the code; I need help in debugging the code.

import javax.swing.*;
import java.awt.*;
import java.awt.event.MouseEvent;
import java.awt.event.MouseListener;
import java.awt.event.MouseMotionListener;
import java.util.*;

class Drawing {

    public static void main(String[] args) {
        JFrame frame = new JFrame("Draw Everything");
        frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        frame.add(new Test(new ImageIcon("map.JPG")));
        frame.pack();
        frame.setVisible(true);
    }
}

class Test extends JLabel
        implements MouseListener, MouseMotionListener {

    private boolean polygonDrawn = false;
    private final Point trackPoint = new Point();
    private static ArrayList polygonPoints = new ArrayList();
    private static Point point;

    public Test(ImageIcon imageIcon) {
        super(imageIcon, SwingConstants.LEFT);
        addMouseListener(this);
        addMouseMotionListener(this);

    }

    @Override
    protected void paintComponent(Graphics g) {
        super.paintComponent(g);

        if (point != null) {
            Double x = point.getX();
            Double y = point.getY();
            g.setColor(Color.RED);
            g.fillOval(x.intValue(), y.intValue(), 5, 5);
        }

        int numPoints = polygonPoints.size();
        if (numPoints == 0) {
            return;
        }

        Point prevPoint = (Point) polygonPoints.get(0);

        Iterator it = polygonPoints.iterator();
        while (it.hasNext()) {
            Point curPoint = (Point) it.next();
            draw(g, prevPoint, curPoint);
            prevPoint = curPoint;
        }
        if (polygonDrawn) {
            draw(g, prevPoint, (Point) polygonPoints.get(0));
        } else {
            draw(g, prevPoint, trackPoint);
        }

    }

    @Override
    public Dimension getPreferredSize() {
        return new Dimension(820, 580);
    }

    @Override
    public void mouseClicked(MouseEvent evt) {
        int x = evt.getX();
        int y = evt.getY();

        switch (evt.getClickCount()) {
            case 1: // single-click
                if (polygonDrawn) {
                    polygonPoints.clear();
                    polygonDrawn = false;
                }
                polygonPoints.add(new Point(x, y));
                repaint();
                break;

            case 2: // double-click
                polygonDrawn = true;
                polygonPoints.add(new Point(x, y));
                repaint();
                break;

            default:
                break;
        }
    }

    @Override
    public void mouseMoved(MouseEvent evt) {
        trackPoint.x = evt.getX();
        trackPoint.y = evt.getY();
        repaint();
    }

    private void draw(Graphics g, Point p1, Point p2) {
        int x1 = p1.x;
        int y1 = p1.y;

        int x2 = p2.x;
        int y2 = p2.y;

        g.setColor(Color.RED);
        g.drawLine(x1 + 3, y1 + 3, x2 + 3, y2 + 3);

        g.setColor(Color.RED);
        g.fillOval(x1, y1, 8, 8);

        g.setColor(Color.RED);
        g.fillOval(x2, y2, 8, 8);
    }

    @Override
    public void mousePressed(MouseEvent e) {
        int x = e.getX();
        int y = e.getY();
        point = new Point(x, y);
        repaint();

    }

    @Override
    public void mouseReleased(MouseEvent e) {
    }

    @Override
    public void mouseEntered(MouseEvent e) {

    }

    @Override
    public void mouseExited(MouseEvent e) {

    }

    @Override
    public void mouseDragged(MouseEvent e) {

    }
}

Answer 1

It is not completely clear what you want to achieve, but note that when drawing the polygon, you are drawing the latest fixed polygon-point on exactly the same place as when drawing the point. To see this, increase the size of the point with

g.fillOval(x.intValue(), y.intValue(), 10, 10);

and draw the polygon with a different color in the draw method:

g.setColor(Color.BLUE);