Проблемы с получением ширины и высоты панели из объекта класса - Javafx

Итак, я понимаю принцип привязки при изменении размера объектов на панели по мере изменения ее размера, эта проблема немного отличается. Я запутался, я пытаюсь создать класс, который расширяет панель, которая создает в моей основной строке координаты startX и startY, привязанные к центру панели. Проблема в том, что при использовании getWidth()/2 или getHeight()/2 координата помещается где-то вверх и слева от начальных координат (0, 0), когда я нажимаю клавишу со стрелкой, которая при нажатии создает другую строку, которая рисуется в заданном направлении и начинается с конца последней нарисованной линии.

Как я уже сказал, когда я использую getWidth() / 2 и getHeight / 2 в качестве координат startX и startY моей новой строки, в ответ линия помещается в отрицательную координату, помещая ее за пределы экрана выше и слева от начальной (0, 0) координаты панели.

Ниже приведена часть моего кода, которая содержит конструктор по умолчанию, с которым у меня возникла проблема, в конструкторе не по умолчанию я даю возможность вручную вводить начальные координаты, и когда я это делаю, строка размещается именно там, где я хочу. .

public class LineDrawingObject extends Pane {
    // ArrayList to store the Line Object's
    ArrayList<Line> lines = new ArrayList<>();
    Line line;
    private Color lineColor;
    private double lineLength;
    private int lineCount = 0;
    private double startX;
    private double startY;
    private double endX;
    private double endY;

    /** Default Constructor */
    public LineDrawingObject() {
        this.lineLength = 20;
        line = new Line(this.getWidth() / 2, this.getHeight() / 2,
            (this.getWidth() / 2), (this.getHeight() / 2) - this.lineLength);
        this.lineColor = Color.BLACK;
        line.setStroke(this.lineColor);
        this.lineCount++;
        this.lines.add(line);
        getChildren().add(line);
    }

Изменить: я подумал, что мне может понадобиться добавить дополнительную информацию

также я хотел добавить, что размер моей панели установлен в новой сцене (панель, 250, 250), поэтому координаты центра будут (125, 125).... Будет ли использование методов getWidth и getHeight на панели возвращать недопустимый размер если он еще не нарисован? Я попытался установить предпочтительный размер в моем методе запуска, но, похоже, это не сработало. Если это так, как я могу решить эту проблему?

import javafx.application.Application;
import javafx.scene.Scene;
import javafx.scene.layout.Pane;
import javafx.scene.paint.Color;
import javafx.stage.Stage;

/**
 * Created by John on 7/24/2014.
 */
public class DrawLines extends Application {
    @Override // Override the start method in the Application class
    public void start(Stage primaryStage) {
        // Create a pane
        Pane pane = new Pane();

        // Create object to draw lines upon KeyEvent
        LineDrawingObject lineDrawingObject = new LineDrawingObject(20, Color.BLACK,
            pane.getWidth() / 2, pane.getWidth() / 2);
        pane.getChildren().add(lineDrawingObject);
        lineDrawingObject.setOnKeyPressed(e -> {
            lineDrawingObject.paintLine(e.getCode());
        });

        // Create a scene and place it in the pane
        Scene scene = new Scene(pane, 250, 250);
        primaryStage.setTitle("DrawLines"); // Set the stage title
        primaryStage.setScene(scene); // Place the scene in the stage
        primaryStage.show(); // Display the stage

        // Allow object to receive key input
        lineDrawingObject.requestFocus();
    }
}

и вот объект LineDrawing:

import javafx.scene.input.KeyCode;
import javafx.scene.layout.Pane;
import javafx.scene.paint.Color;
import javafx.scene.shape.Line;

import java.util.ArrayList;

/** This object will draw lines inside of a Pane when an arrow key is
 *  pressed and will draw it in that direction from the current line */
public class LineDrawingObject extends Pane {
    // ArrayList to store the Line Object's
    ArrayList<Line> lines = new ArrayList<>();
    Line line;
    private Color lineColor;
    private double lineLength;
    private int lineCount = 0;
    private double startX;
    private double startY;
    private double endX;
    private double endY;

    /** Default Constructor */
    public LineDrawingObject() {
        this.lineLength = 20;
        line = new Line(this.getWidth() / 2, this.getHeight() / 2,
            (this.getWidth() / 2), (this.getHeight() / 2) - this.lineLength);
        this.lineColor = Color.BLACK;
        line.setStroke(this.lineColor);
        this.lineCount++;
        this.lines.add(line);
        getChildren().add(line);
    }

    /** Secondary Constructor, allows you to control the line length and color */
    public LineDrawingObject(double lineLength, Color lineColor, double startX, double startY) {
        this.lineLength = lineLength;
        line = new Line(startX, startY,
                startX, startY - this.lineLength);
        this.lineColor = lineColor;
        line.setStroke(this.lineColor);
        this.lineCount++;
        this.lines.add(line);
        getChildren().add(line);
    }

    public ArrayList<Line> getLines() {
        return lines;
    }

    public void setLines(ArrayList<Line> lines) {
        this.lines = lines;
    }

    public Line getLine() {
        return this.line;
    }

    public void setLine(Line line) {
        this.line = line;
    }

    public Color getLineColor() {
        return this.lineColor;
    }

    public void setLineColor(Color lineColor) {
        this.lineColor = lineColor;
    }

    public double getLineLength() {
        return this.lineLength;
    }

    public void setLineLength(double lineLength) {
        this.lineLength = lineLength;
    }

    public int getLineCount() {
        return this.lineCount;
    }

    public void setLineCount(int lineCount) {
        this.lineCount = lineCount;
    }

    public double getStartX() {
        return this.startX;
    }

    public void setStartX(double startX) {
        this.startX = startX;
    }

    public double getStartY() {
        return this.startY;
    }

    public void setStartY(double startY) {
        this.startY = startY;
    }

    public double getEndX() {
        return this.endX;
    }

    public void setEndX(double endX) {
        this.endX = endX;
    }

    public double getEndY() {
        return this.endY;
    }

    public void setEndY(double endY) {
        this.endY = endY;
    }

    public void paintLine(KeyCode keyCode) {
        // Set line start coordinates to the end of the last line
        setStartX(line.getEndX());
        setStartY(line.getEndY());

        // Set line end coordinates
        switch (keyCode) {
            case UP: goUp(); break;
            case LEFT: goLeft(); break;
            case DOWN: goDown(); break;
            case RIGHT: goRight(); break;
        }

        // Create line
        line = new Line(getStartX(), getStartY(), getEndX(), getEndY());
        line.setStroke(lineColor);
        this.lines.add(line);
        getChildren().add(line);
    }

    public void goLeft() {
        setEndX(getStartX() - this.lineLength);
        setEndY(getStartY());
    }

    public void goRight() {
        setEndX(getStartX() + this.lineLength);
        setEndY(getStartY());
    }

    public void goUp() {
        setEndX(getStartX());
        setEndY(getStartY() - this.lineLength);
    }

    public void goDown() {
        setEndX(getStartX());
        setEndY(getStartY() + this.lineLength);
    }
}

Использование конструктора по умолчанию Использование конструктора по умолчанию

Использование пользовательских координат Использование пользовательских координат


person John Conner    schedule 25.07.2014    source источник


Ответы (2)


Вы вызываете getWidth() и getHeight() в конструкторе, который обязательно выполняется до того, как Pane будет добавлен в любую живую сцену. Следовательно, они вернут 0 (потому что панель еще не выложена).

Вместо этого привяжите координаты линии к значениям на основе widthProperty и heightProperty:

line = new Line();
line.startXProperty().bind(widthProperty().divide(2));
line.startYProperty().bind(heightProperty().divide(2));
line.endXProperty().bind(widthProperty().divide(2));
line.endYProperty().bind(heightProperty().divide(2).subtract(lineLength));
person James_D    schedule 25.07.2014
comment
Я пробовал это и был уверен, что это сработает, но проблема в том, что по мере добавления новых строк панель растет, чтобы соответствовать ей, и поэтому перемещает центральную линию каждый раз, когда ширина или высота увеличивается из добавленной строки, и теперь она начинается строки из (0,0) панели - person John Conner; 25.07.2014
comment
Может ли проблема заключаться в том, что я все еще выполняю это действие до того, как панель будет добавлена ​​на сцену? - person John Conner; 25.07.2014

Чтобы решить эту проблему, я решил сделать это проще и установить предустановленный атрибут размера для конструктора и дополнительные конструкторы, позволяющие вам устанавливать размер самостоятельно.

введите здесь описание изображения

import javafx.scene.input.KeyCode;
import javafx.scene.layout.Pane;
import javafx.scene.paint.Color;
import javafx.scene.shape.Line;

import java.util.ArrayList;

/** This object will draw lines inside of a Pane when an arrow key is
 *  pressed and will draw it in that direction from the current line */
public class LineDrawingObject extends Pane {
    // ArrayList to store the Line Object's
    ArrayList<Line> lines = new ArrayList<>();
    Line line;
    private Color lineColor;
    private double lineLength;
    private double startX;
    private double startY;
    private double endX;
    private double endY;

    /** Default Constructor */
    public LineDrawingObject() {
        // Set a default size for this Pane
        this.setWidth(350);
        this.setHeight(350);
        // Set Line properties
        this.startX = getWidth() / 2;
        this.startY = getHeight() / 2;
        this.lineLength = 20;
        this.lineColor = Color.BLACK;
        // Create line and set it's Stroke
        line = new Line(this.startX, this.startY,
                this.startX, this.startY - this.lineLength);
        line.setStroke(this.lineColor);
        // Add line to ArrayList and Pane
        lines.add(line);
        getChildren().add(line);
    }

    /** Second Constructor, allows you to control the line length, color and center it */
    public LineDrawingObject(double lineLength, Color lineColor) {
        // Set a default size for this Pane
        this.setWidth(350);
        this.setHeight(350);
        // Set line properties
        this.startX = getWidth() / 2;
        this.startY = getHeight() / 2;
        this.lineLength = lineLength;
        this.lineColor = lineColor;
        // Create line and set it's Stroke
        line = new Line(this.startX, this.startY,
                this.startX, this.startY - this.lineLength);
        line.setStroke(this.lineColor);
        // Add line to ArrayList and Pane
        lines.add(line);
        getChildren().add(line);
    }

    /** Third Constructor, allows you to control the line length, color, and pane size */
    public LineDrawingObject(double lineLength, Color lineColor,
                             double paneWidth, double paneHeight) {
        // Set a default size for this Pane
        this.setWidth(paneWidth);
        this.setHeight(paneHeight);
        // Set line properties
        this.startX = getWidth() / 2;
        this.startY = getHeight() / 2;
        this.lineLength = lineLength;
        this.lineColor = lineColor;
        // Create line and set it's Stroke
        line = new Line(this.startX, this.startY,
                this.startX, this.startY - this.lineLength);
        line.setStroke(this.lineColor);
        // Add line to ArrayList and Pane
        lines.add(line);
        getChildren().add(line);
    }

    /** Third Constructor, allows you to control the line length and color, startX, and startY */
    public LineDrawingObject(double lineLength, double startX, double startY, Color lineColor) {
        // Set line properties
        this.startX = startX;
        this.startY = startY;
        this.lineLength = lineLength;
        this.lineColor = lineColor;
        // Create line and set it's Stroke
        line = new Line(this.startX, this.startY,
                this.startX, this.startY - this.lineLength);
        line.setStroke(this.lineColor);
        // Add line to ArrayList and Pane
        lines.add(line);
        getChildren().add(line);
    }

    /** Get the list of lines */
    public ArrayList<Line> getLines() {
        return lines;
    }

    /** Get the current line object */
    public Line getLine() {
        return this.line;
    }

    /** Manually set the current line object */
    public void setLine(Line line) {
        this.line = line;
    }

    /** Get the current line object's color */
    public Color getLineColor() {
        return this.lineColor;
    }

    /** Set the current line object's color */
    public void setLineColor(Color lineColor) {
        this.lineColor = lineColor;
    }

    /** Get length of the lines */
    public double getLineLength() {
        return this.lineLength;
    }

    /** Set a new length for the lines */
    public void setLineLength(double lineLength) {
        this.lineLength = lineLength;
    }

    /** Get the count of the number of line's currently in existence */
    public int getLineCount() {
        return this.lines.size();
    }

    /** Get the startX coordinates */
    public double getStartX() {
        return this.startX;
    }

    /** Set the startX coordinates */
    public void setStartX(double startX) {
        this.startX = startX;
    }

    /** Get the startY coordinates */
    public double getStartY() {
        return this.startY;
    }

    /** Set the startY coordinates */
    public void setStartY(double startY) {
        this.startY = startY;
    }

    /** Get the endX coordinates */
    public double getEndX() {
        return this.endX;
    }

    /** Set the endX coordinates */
    public void setEndX(double endX) {
        this.endX = endX;
    }

    /** Get the endY coordinates */
    public double getEndY() {
        return this.endY;
    }

    /** Set the endY coordinates */
    public void setEndY(double endY) {
        this.endY = endY;
    }

    /** Paint the next line based on the key pressed */
    public void paintLine(KeyCode keyCode) {
        // Set line start coordinates to the end of the last line
        setStartX(line.getEndX());
        setStartY(line.getEndY());

        // Set line end coordinates
        switch (keyCode) {
            case UP: goUp(); break;
            case LEFT: goLeft(); break;
            case DOWN: goDown(); break;
            case RIGHT: goRight(); break;
        }

        // Create line
        line = new Line(getStartX(), getStartY(), getEndX(), getEndY());
        line.setStroke(lineColor);
        this.lines.add(line);
        getChildren().add(line);
    }

    /** Set the end coordinates to the left of the last line */
    public void goLeft() {
        setEndX(getStartX() - this.lineLength);
        setEndY(getStartY());
    }

    /** Set the end coordinates to the right of the last line */
    public void goRight() {
        setEndX(getStartX() + this.lineLength);
        setEndY(getStartY());
    }

    /** Set the end coordinates above the last line */
    public void goUp() {
        setEndX(getStartX());
        setEndY(getStartY() - this.lineLength);
    }

    /** Set the end coordinates below the last line */
    public void goDown() {
        setEndX(getStartX());
        setEndY(getStartY() + this.lineLength);
    }
}
person John Conner    schedule 25.07.2014