Add/Remove 鼠标点击

Question

我编写了这个程序，它允许用户在矩形中添加和删除圆圈，它在用户左键单击时添加并在 he/she 右键单击时删除，虽然它的添加工作正常但它的删除没有'效果不佳，例如：当我添加四个圆圈并尝试删除所有圆圈时，其中一些没有删除，我不知道为什么，我很困惑。请帮忙

public class BoundingRectangle extends Application {
@Override
public void start(Stage primaryStage) {
    Pane pane = new Pane();
    ArrayList<Circle> list = new ArrayList<>();

    Rectangle rectangle1 = new Rectangle(10, 10, 175, 80);
    rectangle1.setStroke(Color.BLACK);
    rectangle1.setFill(Color.WHITE);
    Rectangle rectangle2 = new Rectangle(250, 75, 300, 200);
    rectangle2.setStroke(Color.BLACK);
    rectangle2.setFill(Color.WHITE);

    Text text = new Text(20, 33, "INSTRUCTION\n" +
            "Add: Left Click\nRemove: Right Click");

    pane.setOnMouseClicked(e -> {
        if (e.getButton() == MouseButton.PRIMARY) {
            Circle circle = new Circle(e.getX(), e.getY(), 10);
            list.add(circle);
            circle.setStroke(Color.BLACK);
            circle.setFill(Color.WHITE);

           if (circle.getCenterX() - rectangle2.getX() < 10) {
                circle.setCenterX(rectangle2.getX() + 10);
            } 
           else if (circle.getCenterY() - rectangle2.getY() < 10) {
                circle.setCenterY(rectangle2.getY() + 10);
            } 
           else if (rectangle2.getX() + rectangle2.getWidth() -
                    circle.getCenterX() < 10) {
                circle.setCenterX(rectangle2.getX() +  rectangle2.getWidth()  - 10);
            } 
           else if (rectangle2.getY() + rectangle2.getHeight() -
                    circle.getCenterY() < 10) {
                circle.setCenterY(rectangle2.getY() + rectangle2.getHeight() - 10);
            }
                pane.getChildren().add(circle);
        }

        else if (e.getButton() == MouseButton.SECONDARY) {
                for (int i = 0; i < list.size(); i++) {
                    if (list.get(i).contains(e.getX(), e.getY())) {
                        pane.getChildren().remove(list.get(i));
                        list.remove(i);
                        break;
                    }
                }
            }

        });
        pane.getChildren().addAll(rectangle1, rectangle2, text);

        Scene scene = new Scene(pane, 600, 300);
        primaryStage.setScene(scene);
        primaryStage.setTitle("BoundingRectangle");
        primaryStage.show();
    }
}

Answer 1

我认为这看起来像一个错误：如果您在右键单击失败后调整 window 的大小，调整大小强制重绘会删除圆圈。一些简单的调试显示圆圈在它们应该从窗格的子列表中删除（因此这是一个重绘问题）。

解决方法是将第二个矩形替换为窗格。这是直接替换且没有其他更改的代码：您实际上可以重构它以简化它（在第二个窗格而不是第一个窗格中注册鼠标侦听器以添加，并在每个圆圈中注册鼠标侦听器以删除, 例如):

import java.util.ArrayList;

import javafx.application.Application;
import javafx.scene.Scene;
import javafx.scene.input.MouseButton;
import javafx.scene.layout.Pane;
import javafx.scene.paint.Color;
import javafx.scene.shape.Circle;
import javafx.scene.shape.Rectangle;
import javafx.scene.text.Text;
import javafx.stage.Stage;

public class BoundingRectangle extends Application {
    @Override
    public void start(Stage primaryStage) {
        Pane pane = new Pane();
        ArrayList<Circle> list = new ArrayList<>();

        Rectangle rectangle1 = new Rectangle(10, 10, 175, 80);
        rectangle1.setStroke(Color.BLACK);
        rectangle1.setFill(Color.WHITE);
        Pane rectangle2 = new Pane();
        rectangle2.relocate(250, 75);
        rectangle2.setPrefSize(300,  200);
        rectangle2.setStyle("-fx-border-color: black;");

        Text text = new Text(20, 33, "INSTRUCTION\n"
                + "Add: Left Click\nRemove: Right Click");

        pane.setOnMouseClicked(e -> {
            if (e.getButton() == MouseButton.PRIMARY) {
                Circle circle = new Circle(e.getX(), e.getY(), 10);
                list.add(circle);
                circle.setStroke(Color.BLACK);
                circle.setFill(Color.WHITE);

                if (circle.getCenterX() - rectangle2.getBoundsInParent().getMinX() < 10) {
                    circle.setCenterX(rectangle2.getBoundsInParent().getMinX() + 10);
                } else if (circle.getCenterY() - rectangle2.getBoundsInParent().getMinY() < 10) {
                    circle.setCenterY(rectangle2.getBoundsInParent().getMinY() + 10);
                } else if (rectangle2.getBoundsInParent().getMaxX()
                        - circle.getCenterX() < 10) {
                    circle.setCenterX(rectangle2.getBoundsInParent().getMaxX() - 10);
                } else if (rectangle2.getBoundsInParent().getMaxY()
                        - circle.getCenterY() < 10) {
                    circle.setCenterY(rectangle2.getBoundsInParent().getMaxY() - 10);
                }
                pane.getChildren().add(circle);
            }

            else if (e.getButton() == MouseButton.SECONDARY) {
                for (int i = 0; i < list.size(); i++) {
                    if (list.get(i).contains(e.getX(), e.getY())) {
                        pane.getChildren().remove(list.get(i));
                        list.remove(i);
                        break;
                    }
                }
            }

        });
        pane.getChildren().addAll(rectangle1, rectangle2, text);

        Scene scene = new Scene(pane, 600, 300);
        primaryStage.setScene(scene);
        primaryStage.setTitle("BoundingRectangle");
        primaryStage.show();
    }

    public static void main(String[] args) {
        launch(args);
    }
}

这是重构后的版本。由于 rectangle2 现在是 Pane，您可以直接将圆圈添加到其中，而不必担心调整它们的位置。右键单击是在圆圈本身上进行的，因此不需要代码来检查单击是否在圆圈上，因此，不需要单独的圆圈列表（尽管您的实际应用程序可能需要这样做其他原因）。

import javafx.application.Application;
import javafx.scene.Scene;
import javafx.scene.input.MouseButton;
import javafx.scene.layout.Pane;
import javafx.scene.paint.Color;
import javafx.scene.shape.Circle;
import javafx.scene.shape.Rectangle;
import javafx.scene.text.Text;
import javafx.stage.Stage;

public class BoundingRectangle extends Application {
    @Override
    public void start(Stage primaryStage) {
        Pane pane = new Pane();

        Rectangle rectangle1 = new Rectangle(10, 10, 175, 80);
        rectangle1.setStroke(Color.BLACK);
        rectangle1.setFill(Color.WHITE);

        Pane rectangle2 = new Pane();
        rectangle2.relocate(250, 75);
        rectangle2.setPrefSize(300,  200);
        rectangle2.setStyle("-fx-border-color: black;");

        Text text = new Text(20, 33, "INSTRUCTION\n"
                + "Add: Left Click\nRemove: Right Click");

        rectangle2.setOnMouseClicked(e -> {
            if (e.getButton() == MouseButton.PRIMARY) {
                Circle circle = new Circle(e.getX(), e.getY(), 10);

                circle.setOnMouseClicked(evt -> {
                    if (evt.getButton() == MouseButton.SECONDARY) {
                        rectangle2.getChildren().remove(circle);
                    }
                });

                circle.setStroke(Color.BLACK);
                circle.setFill(Color.WHITE);

                rectangle2.getChildren().add(circle);
            }


        });
        pane.getChildren().addAll(rectangle1, rectangle2, text);

        Scene scene = new Scene(pane, 600, 300);
        primaryStage.setScene(scene);
        primaryStage.setTitle("BoundingRectangle");
        primaryStage.show();
    }

    public static void main(String[] args) {
        launch(args);
    }
}

Add/Remove 鼠标点击

Add/Remove by mouse click

java

javafx

javafx-8