拖放填充形状vs未填充形状[作业]

Question

我正在为一个家庭作业制作一个ms绘画风格的程序。在这个家庭作业中，你应该能够右键点击一个形状并在屏幕上移动它。填充的形状可以在形状内的任何地方右键单击，而未填充的形状则应在形状的边框上右键单击并选择。

我遇到的问题是，当我最初启动程序，绘制和未填充的形状，并尝试拖动它，它会拖动，但一旦我再次放下它，它的行为正确，变得不可拖动，所有未来未填充的形状都相应地行动。

TLDR：我希望所有填充的形状都是可拖动的，所有未填充的形状都是不可拖动的。未填充的形状（出于某种原因）是可拖动的，如果它是放置的第一个形状。拖拽一次未填充的形状后，它的行为就会正确。

下面是相关代码：

private boolean isFilled = false;
private JCheckBox fill = new JCheckBox("Filled");
private int indexToMove;

涂料组件

public void paintComponent(Graphics g) {
super.paintComponent(g);
g.setColor(Color.WHITE);
g.clearRect(0, 0, getWidth(), getHeight());

for (int i = 0; i < shapes.size(); i++) {
    shapes.get(i).draw(g);
}

repaint();

}

在“我的填充”按钮上执行的操作

else if (e.getSource() == fill) {
    if (fill.isSelected()) {
    isFilled = true;
    } else {
    isFilled = false;
    }
}

这是我开始拖放的部分

按下鼠标

public void mousePressed(MouseEvent e) {

if (SwingUtilities.isRightMouseButton(e)) {
    for (int i = 0; i < shapes.size(); i++) {
    if (shapes.get(i).isSelected(e.getX(), e.getY())) {
        System.out.println("Shape contained in: " + i);
        indexToMove = i;
    }
    }
}
}

鼠标拖动

public void mouseDragged(MouseEvent e) {

if (SwingUtilities.isRightMouseButton(e)) {
    if (indexToMove >= 0) {
    shapes.get(indexToMove).move(e.getX(), e.getY());
    }
}
}

鼠标已释放

public void mouseReleased(MouseEvent e) {
if (SwingUtilities.isRightMouseButton(e)) {
    indexToMove = -1;
}
}

下面是相关的SHAPE类代码

public class MyRectangle extends Rectangle implements MyShape {
private Color color;
private boolean filled = false;
private int x, y, dx, dy, height, width;

public MyRectangle(Color color, boolean filled, int x, int y, int endX, int endY) {
this.color = color;
this.filled = filled;
this.x = x;
this.y = y;
this.dx = endX;
this.dy = endY;

}

@Override
public void draw(Graphics g) {

...

if (!filled) {

    g.drawRect(x, y, width, height);
}

if (filled) {
    g.fillRect(x, y, width, height);
}
}

这是我检查它是否填充/未填充的地方，系统输出只发生在我拖放一次未填充的形状之后

@Override
public boolean isSelected(int clickX, int clickY) {
boolean selected = false;
if(filled == false){
    System.out.println("unfilled shape, do not move");
    selected = false;
}

else if (filled == true) {
    if ((clickX > x) && (clickX < dx) && (clickY > y) && (clickY < dy)) {

    selected = true;
    } else {
    selected = false;
    }
}


return selected;
}


public void move(int offsetX, int offsetY) {

int centerX = (dx + x) / 2;
int centerY = (dy + y) / 2;
System.out.println("CenterX: " + centerX + " CenterY: " + centerY);

if (offsetX > centerX) {

    offsetX = offsetX - centerX;
    x += offsetX;
    dx += offsetX;
} else if (offsetX < centerX) {

    offsetX = centerX - offsetX;
    x -= offsetX;
    dx -= offsetX;
}

if (offsetY > centerY) {

    offsetY = offsetY - centerY;

    y += offsetY;
    dy += offsetY;
}

else if (offsetY < centerY) {

    offsetY = centerY - offsetY;
    y -= offsetY;
    dy -= offsetY;
}

}

我知道这是相当多的代码，我试着删掉任何看起来不相关的东西。如果你需要看更多的话让我知道！

Answer 1

回答了我自己的问题。

右键单击时，indexToMove没有分配给任何东西，因为直到形状移动后，它才会注册它不应该移动。

我给indexToMove的值是-1，所以当您启动时，形状将不可拖动。