更改形状列表的颜色

Question

我被这个问题卡住了：

当我点击一个形状（有一个矩形和圆形列表）时，它会改变它的颜色。但当我点击外面时，它不会变回来。

public void mouseClicked(MouseEvent me) {
    Color colorAux;
    for (int i = 0; i < images.size(); i++) {
        colorAux = images.get(i).getColor();
        if (images.get(i).getShape() == "Rectangle") {
            if ((images.get(i).getLocation().getX() < me.getX() && images.get(i).getLocation().getY() < me.getY() && images.get(i).getX() + images.get(i).getWidth() > me.getX() && images.get(i).getLocation().getY() + images.get(i).getHeight() > me.getY())) {
                images.get(i).setColor(Color.BLUE);
                repaint();
                JOptionPane.showMessageDialog(null, colorAux); //Debug
            } else if (!(images.get(i).getLocation().getX() < me.getX() && images.get(i).getLocation().getY() < me.getY() && images.get(i).getX() + images.get(i).getWidth() > me.getX() && images.get(i).getLocation().getY() + images.get(i).getHeight() > me.getY()) && (images.get(i).getColor() == Color.BLUE)) {
                images.get(i).setColor(colorAux);           
                repaint();
            }
        }
    }

更具体地说，当我只画1个形状时，它工作！但是，例如，我绘制一个蓝色矩形、一个紫色圆和一个红色矩形，并单击其中一些形状，如红色矩形，每个形状都将其颜色改为蓝色。当我再次单击外部时，它会将每个形状的颜色更改为默认颜色（黑色）。

public void mouseClicked(MouseEvent me) {
    List<Color> colors = new ArrayList<Color>();
    for (int j = 0; j < images.size(); j++) {
        colors.add(images.get(j).getColor());
    }
    for (int i = 0; i < images.size(); i++) {
        if ((images.get(i).getLocation().getX() < me.getX() && images.get(i).getLocation().getY() < me.getY() && images.get(i).getX() + images.get(i).getWidth() > me.getX() && images.get(i).getLocation().getY() + images.get(i).getHeight() > me.getY())) {
            images.get(i).setColor(Color.BLUE);
            repaint();
        } else {
            images.get(i).setColor(colors.get(i));
            repaint();
        }
    }
}

Answer 1

你在正确的想法中使用一个颜色列表（除非所有对象在开始时都有相同的颜色）。在这个列表中，您存储了所有对象的初始颜色以及您可以执行的

// initialColors is the list holding the initial colors
for (int i=0; i<images.size(); i++) {
    if (images.get(i).getShape() == "Rectangle") {
        if (/*code to check if we are inside the rectangle which you already have*/) {
            images.get(i).setColor(Color.BLUE);
            repaint();
        } else {
            images.get(i).setColor(initialColors.get(i));
            repaint();
        }
    } /* maybe add a case for getShape() == "Circle" */
}

您可以在创建和填充InitialColors列表的同时创建和填充Images列表（因为在那一刻，您就知道每个形状的颜色了）。

关于为什么您的方法不起作用：假设我们在一个矩形内单击，它的颜色更改为蓝色。当我们现在使用coloraux=images.get(i).getcolor()来获取颜色时，我们得到的是蓝色，因为我们改变了矩形的颜色。当我们到达images.get(i).setcolor(colorAux)时，我们将蓝色矩形的颜色设置为蓝色，这意味着什么也没有发生。

编辑：现在您添加到问题中的更改不起作用，因为我们仍然有同样的问题：我们在mouseclicked事件中获得颜色，而不是在形状最初着色时获得颜色。这意味着我们用当前的颜色（可能已经改为蓝色）来填充颜色列表，而不是初始的颜色。您应该将添加的循环移动到最初为形状着色的地方（可能是填充Images列表的地方）。