时间戳

首页 > 解决方案 > 我可以使用 getComponents() 而不是保留组件的 ArrayList 吗？

java - 我可以使用 getComponents() 而不是保留组件的 ArrayList 吗？

问题描述

我有一个应用程序，用户可以通过单击位置 (x,y) 来动态添加形状，比如圆形（为简单起见）。我一直在保留一个圆的 ArrayList，因此它们通过 paintComponent() 方法重新添加到 JPanel，但是我怀疑这可能是多余的，因为保留了一个幕后数组，可以通过内置 getComponents() 方法。我对吗？我的代码的关键部分如下：

public class DrawingPanel extends JPanel implements Constants {
Point point;
Figure figure;

public DrawingPanel() {
    point = new Point(0, 0);
    figure = new Circle(point, defaultSize);
    setLayout(null);
    setBackground(Color.white);
    setPreferredSize(new Dimension(450, 450));
    addMouseListener(new ActionHandler());
}

public void paintComponent(Graphics page) {
    super.paintComponent(page);
    add(figure);
    }

private class ActionHandler extends MouseAdapter {

    @Override
    public void mousePressed(MouseEvent e) {
        super.mousePressed(e);
        Point point = e.getPoint();
        System.out.println("Mouse pressed at (" + e.getX() + ", " + e.getY() + ")");

        String figureState = itemHandler.getFigureState();
        String figureActionState = itemHandler.getFigureActionState();

        if (figureActionState.equals("None")) {
            switch (figureState) {
                case "Circle" -> {
                    figure = new Circle(point, defaultSize);
                    figure.setBounds((int) point.getX(), (int) point.getY(), 50, 50);
                    figure.addMouseListener(new ActionHandler());
                }
                case "Square" -> {
                    figure = new Rect(point, defaultSize);
                    figure.setBounds((int) point.getX(), (int) point.getY(), 50, 50);
                    figure.addMouseListener(new ActionHandler());
                }
                case "Cross" -> {
                    figure = new Cross(point, defaultSize);
                    figure.setBounds((int) point.getX(), (int) point.getY(), 50, 50);
                    figure.addMouseListener(new ActionHandler());
                }
            }
        }
        repaint();
    }
}

}

public class Figure extends JComponent {
Point position;
Dimension size;

public Figure(Point position, Dimension size) {
    this.position = position;
    this.size = size;
}

public void paintComponent(Graphics page) {
    super.paintComponent(page);
}

}

public interface Constants {
ItemHandler itemHandler = new ItemHandler();
Dimension defaultSize = new Dimension(50, 50);

}

标签： javaswingdynamicjpaneljcomponent

解决方案

您有整体设计问题：

  1. 绘画方法不应该改变类的状态。绘制方法应该只绘制类的当前状态。因此，您不应该在 paintComponent() 方法中向面板添加组件。

  2. 我看不出保留这两个 ArrayList 的理由。组件应该知道如何绘制自身并包含绘制自身所需的所有信息。

也许您可以只绘制一个Shape. 查看Drag a Painted Shape。它保留了所有Shapes要绘制的 ArrayList。此 ArrayList 用于标识Shape要拖动的 a。您需要用现有的鼠标处理逻辑替换拖动逻辑。

推荐阅读