Проблемы с JApplets и вызовом paint / paintComponent

LineDrawer - это компонент, вы должны добавить его в свой апплет, и он будет отрисован автоматически.

Похоже, вы столкнулись с проблемой коллизии имен.

JComponent определяет два метода рисования (на самом деле он определяет еще несколько, но давайте оставим это простым), paint и paintComponent.

Проблема, с которой вы столкнулись, связана с тем фактом, что вы унаследовали от JComponent, но, похоже, выполняете свою собственную картину. Это нормально, если вы хотите разработать собственные классы для рисования, но не подходят для компонентов Swing. Компонент Swing - это нечто большее, чем просто его рисование.

Другая проблема, с которой вы столкнулись, заключается в том, что вы определили метод «рисования» в побочном MouseHandler методе и, похоже, предполагаете, что он вызывается из класса LineDraw. Проблема в том, что вы фактически вызываете метод JComponent paint, а не тот, который вы хотите / думаете.

Попробуйте удалить extends JComponent и переместить метод paint в класс LineDraw.

Как указано в MouseEvent, в большинстве случаев вы никогда не должны вызывать paint или paintComponent напрямую.

Вы можете найти:

• Рисование в AWT и Swing
• Выполнение пользовательской раскраски

Полезный

ОБНОВЛЕНО

Рисование как компонент Swing

public class AppletWin extends JApplet {

    private LineDraw lineDrawer;

    public void init() {
        EventQueue.invokeLater(new Runnable() {
            @Override
            public void run() {
                setBackground(Color.white);
                lineDrawer = new LineDraw(AppletWin.this);
                setLayout(new BorderLayout());
                add(lineDrawer);
            }
        });
    }

    public class LineDraw extends JComponent {

        private final Point p1;
        private final Point p2;
        private final JApplet callingWindow;
        private final ArrayList<Line2D.Double> lines;

        public LineDraw(JApplet callingWindow) {
            this.p1 = new Point();
            this.p2 = new Point();
            this.callingWindow = callingWindow;
            this.lines = new ArrayList<Line2D.Double>();
            MouseListener mouse = new MouseHandler();
            addMouseListener(mouse);
        }

        public void paintComponent(Graphics g) {
            super.paintComponent(g);
            Graphics2D g2 = (Graphics2D) g;
            g2.setColor(Color.RED);
            for (Line2D.Double e : lines) {
                g2.draw(e);
            }
        }

        public class MouseHandler extends MouseAdapter {

            private boolean firstClick;

            public MouseHandler() {
                firstClick = true;
            }

            public void mouseClicked(MouseEvent e) {
                if (firstClick) {
                    p1.setLocation(e.getPoint());
                    firstClick = false;
                } else {
                    p2.setLocation(e.getPoint());
                    lines.add(new Line2D.Double(p1, p2));
                    p2.setLocation(p1.getLocation());
                }
                repaint();
            }
        }
    }
}

Как художник

public class AppletWin extends JApplet {

    private LineDraw lineDrawer;

    public void init() {
        setBackground(Color.white);
        lineDrawer = new LineDraw(this);
    }

    public void paint(Graphics h) {
        // The the applet clear the background
        super.paint(h);
        Graphics2D g = (Graphics2D) h;
        lineDrawer.paintComponent(g);
    }
    public class LineDraw {

        private final Point p1;
        private final Point p2;
        private final JApplet callingWindow;
        private final ArrayList<Line2D.Double> lines;

        public LineDraw(JApplet callingWindow) {
            this.p1 = new Point();
            this.p2 = new Point();
            this.callingWindow = callingWindow;
            this.lines = new ArrayList<Line2D.Double>();
            MouseListener mouse = new MouseHandler();
            callingWindow.addMouseListener(mouse);
        }

        public void paintComponent(Graphics2D g2) {
            System.out.println("...");
            for (Line2D.Double e : lines) {
                g2.draw(e);
            }
        }

        public class MouseHandler extends MouseAdapter {

            private boolean firstClick;

            public MouseHandler() {
                firstClick = true;
            }

            public void mouseClicked(MouseEvent e) {
                if (firstClick) {
                    p1.setLocation(e.getPoint());
                    firstClick = false;
                } else {
                    p2.setLocation(e.getPoint());
                    lines.add(new Line2D.Double(p1, p2));
                    p2.setLocation(p1.getLocation());
                }
                callingWindow.repaint();
            }
        }
    }
}

Вам действительно следует избегать переопределения paint метода контейнеров верхнего уровня. Метод paint - очень сложный метод, и его обход только вызовет проблемы в долгосрочной перспективе, кроме того, он не имеет двойной буферизации;)

Проблема с этим кодом заключается в том, что метод paintComponent в LineDraw находится внутри внутреннего класса MouseHandler.