【javafxに関する疑問】FXML内において、オブジェクトのコントロール分割は可能か？

Question

実現したいこと

javafxにて、画面内の短形（以降操作キャラと呼びます）を動かし、中央にある枠に触れた時と、枠の内部に入った時にテキストが変化するコードを作成しました。
作成後、「オブジェクトコントロールを個別に設定出来たら、操作キャラは操作キャラのクラスに処理を格納でき、現在のコントロールクラスの可読性が上がるのでは？」と思いました。
そこで質問です。FXMLにおいて、コントロール設定されたFXMLの、特定のオブジェクトのコントロールを他クラスに譲渡するというのは可能なのでしょうか？

発生している問題・分からないこと

実現可能か分からない
・FXMLのオブジェクト（今回の場合は短形を）のコントロールを他のクラスに譲渡できるのか？
→Screen2に記述している、「private Rectangle shape1;//操作キャラ」に関連する移動制御の処理は個別に用意したり、中央枠等の操作キャラ以外を種類ごとに分けて機能を拡張していきたい。

該当のソースコード

package application;

import java.net.URL;
import java.util.ResourceBundle;

import javafx.animation.AnimationTimer;
import javafx.beans.binding.BooleanBinding;
import javafx.beans.property.BooleanProperty;
import javafx.beans.property.SimpleBooleanProperty;
import javafx.fxml.FXML;
import javafx.fxml.Initializable;
import javafx.geometry.Bounds;
import javafx.scene.input.KeyCode;
import javafx.scene.layout.AnchorPane;
import javafx.scene.paint.Color;
import javafx.scene.shape.Rectangle;
import javafx.scene.text.Text;

public class Screen2 implements Initializable{
	
	@FXML
    private AnchorPane screen2;//画面
	
	@FXML
    private Rectangle boundary1;//境界(図形)
    private Bounds boundboundary1;//境界の境界線
    
    @FXML
    private Rectangle shape1;//操作キャラ
    private Bounds boundshape1;//操作キャラ境界線
    
    boolean tutch = false;
    boolean included= false;
    
    
    @FXML
    private Text text;
    
	//キー押下変数
    private BooleanProperty wPressed = new SimpleBooleanProperty();
    private BooleanProperty aPressed = new SimpleBooleanProperty();
    private BooleanProperty sPressed = new SimpleBooleanProperty();
    private BooleanProperty dPressed = new SimpleBooleanProperty();
    private BooleanProperty shiftPressed = new SimpleBooleanProperty();
    
    private BooleanBinding keyPressed = wPressed.or(aPressed).or(sPressed).or(dPressed).or(shiftPressed);
    
    
    
    //移動速度
    private int movementVariable = 2;
    
    //スピードアップ
    private int speedUp = 3;
    
    //短形の高さ:
    private double shape1Height;
    
    AnimationTimer timer = new AnimationTimer() {
        @Override
        public void handle(long timestamp) {
        	
        	boundshape1 = shape1.getBoundsInParent();
        	boundboundary1 = boundary1.getBoundsInParent();
        	contains(boundshape1,boundboundary1);
        	
            if(wPressed.get()) {
            	if(shiftPressed.get()) {
            		shape1.setLayoutY(shape1.getLayoutY() - (movementVariable + speedUp));
            		
            	}
            	shape1.setLayoutY(shape1.getLayoutY() - movementVariable);
                
            }

            if(sPressed.get()){
            	if(shiftPressed.get()) {
            		shape1.setLayoutY(shape1.getLayoutY() + (movementVariable + speedUp));
            	}
                shape1.setLayoutY(shape1.getLayoutY() + movementVariable);
            }

            if(aPressed.get()){
            	if(shiftPressed.get()) {
            		shape1.setLayoutX(shape1.getLayoutX() - (movementVariable + speedUp));
            	}
                shape1.setLayoutX(shape1.getLayoutX() - movementVariable);
            }

            if(dPressed.get()){
            	if(shiftPressed.get()) {
            		shape1.setLayoutX(shape1.getLayoutX() + (movementVariable + speedUp));
            	}
                shape1.setLayoutX(shape1.getLayoutX() + movementVariable);
            }
            
            if(tutch == true) {
            	text.setText("当たっています。");
    			text.setFill(Color.RED);
            }
            if(included == true){
            	text.setText("内部に入っています。");
    			text.setFill(Color.RED);
            }
            if(tutch != true && included != true) {
            	text.setText("何にも接触していません。");
            }
            
            
            checkBorder();
            
        }
    };
    

    @Override
    public void initialize(URL url, ResourceBundle resourceBundle) {
    	//shape1X座標
    	shape1Height = shape1.getHeight();
    	//shape1四隅
    	boundshape1 = shape1.getBoundsInParent();
    	boundboundary1 = boundary1.getBoundsInParent();
    	movementSetup();

        keyPressed.addListener(((observableValue, aBoolean, t1) -> {
            if(!aBoolean){
                timer.start();
            } else {
                timer.stop();
            }
        }));
    }

    public void movementSetup(){
    	screen2.setOnKeyPressed(e -> {
            if(e.getCode() == KeyCode.W) {
                wPressed.set(true);
            }

            if(e.getCode() == KeyCode.A) {
                aPressed.set(true);
            }

            if(e.getCode() == KeyCode.S) {
                sPressed.set(true);
            }

            if(e.getCode() == KeyCode.D) {
                dPressed.set(true);
            }
            
            if(e.getCode() == KeyCode.SHIFT) {
            	shiftPressed.set(true);
            }
            
        });

    	screen2.setOnKeyReleased(e ->{
            if(e.getCode() == KeyCode.W) {
                wPressed.set(false);
            }

            if(e.getCode() == KeyCode.A) {
                aPressed.set(false);
            }

            if(e.getCode() == KeyCode.S) {
                sPressed.set(false);
            }

            if(e.getCode() == KeyCode.D) {
                dPressed.set(false);
            }
            if(e.getCode() == KeyCode.SHIFT) {
            	shiftPressed.set(false);
            }
        });
    }
    
    //当たり判定左がチェック対象、右があたる可能性のあるオブジェクト
    public void contains(Bounds smallShape, Bounds largeShape) {
    	
    	//接触判定
    	if ( smallShape.intersects(largeShape)) {
    		tutch = true;
		}else {
			tutch = false;
		}
    	
    	//内部判定
    	if(largeShape.getMinX() < smallShape.getMinX() && largeShape.getMinY() < smallShape.getMinY() && 
    			largeShape.getMaxX() > smallShape.getMaxX() && largeShape.getMaxY() > smallShape.getMaxY()){
			included = true;
		}
		else {
			included = false;
		}
    	
		
    }
    
    public void checkBorder() {
    	//上からwsad変数
    	double topbound = 0;
    	double bottombound = screen2.getHeight() - shape1Height;
    	double leftbound = 0;
    	double lightbound = screen2.getWidth() - shape1Height;
    	
    	//w移動制御
    	if(shape1.getLayoutY() <= topbound) {
    		shape1.setLayoutY(topbound);
    	}
    	//s移動制御
    	if(shape1.getLayoutY() >= bottombound) {
    		shape1.setLayoutY(bottombound);
    	}
    	//a移動制御
    	if(shape1.getLayoutX() <= leftbound) {
    		shape1.setLayoutX(leftbound);
    	}
    	//d移動制御
    	if(shape1.getLayoutX() >= lightbound) {
    		shape1.setLayoutY(lightbound);
    	}
    	
    }
	
	
}

<?xml version="1.0" encoding="UTF-8"?>

<?import javafx.scene.layout.AnchorPane?>
<?import javafx.scene.shape.Rectangle?>
<?import javafx.scene.text.Text?>

<AnchorPane fx:id="screen2" maxHeight="-Infinity" maxWidth="-Infinity" minHeight="-Infinity" minWidth="-Infinity" prefHeight="500.0" prefWidth="1000.0" xmlns="http://javafx.com/javafx/24.0.1" xmlns:fx="http://javafx.com/fxml/1" fx:controller="application.Screen2">
   <children>
      <Rectangle fx:id="shape1" arcHeight="5.0" arcWidth="5.0" fill="#ff8c1f" height="100.0" layoutX="450.0" layoutY="400.0" stroke="BLACK" strokeType="INSIDE" width="100.0" />
      <Rectangle fx:id="boundary1" arcHeight="5.0" arcWidth="5.0" fill="TRANSPARENT" height="200.0" layoutX="400.0" layoutY="150.0" stroke="BLACK" strokeType="INSIDE" width="200.0" AnchorPane.bottomAnchor="150.0" AnchorPane.leftAnchor="400.0" AnchorPane.rightAnchor="400.0" AnchorPane.topAnchor="150.0" />
      <Text fx:id="text" layoutX="417.0" layoutY="63.0" strokeType="OUTSIDE" strokeWidth="0.0" wrappingWidth="166.80340576171875" />
   </children>
</AnchorPane>

試したこと・調べたこと

• teratailやGoogle等で検索した
• ソースコードを自分なりに変更した
• 知人に聞いた
• その他

上記の詳細・結果

試してみたこと。
・操作キャラを別のFXMLに作成。Screen2.FXMLで呼び出し（include）、Screen2クラスの操作キャラの初期値設定箇所に割り込ませてみた。
→対象の短形でNullpoint発生

補足

特になし

Answer 1

「ランダムに動く円」追加版（大して変わらないが1万時に収まらなかったので^^;

<?xml version="1.0" encoding="UTF-8"?>

<?import application.Shape1?>
<?import javafx.scene.layout.AnchorPane?>
<?import javafx.scene.shape.Rectangle?>
<?import javafx.scene.text.Text?>
<AnchorPane fx:id="screen2" prefHeight="500.0" prefWidth="1000.0" xmlns:fx="http://javafx.com/fxml"
            fx:controller="application.Screen2Controller">
    <children>
        <Shape1 fx:id="shape1" arcHeight="5.0" arcWidth="5.0" fill="#ff8c1f" height="100.0" layoutX="450.0"
                layoutY="400.0"
                focusTraversable="true" stroke="BLACK" strokeType="INSIDE" width="100.0"/>
        <!-- <Shape1 fill="RED" layoutX="150.0" layoutY="150.0" width="100.0" height="100.0" focusTraversable="true"/> -->

        <Rectangle fx:id="boundary1" arcHeight="5.0" arcWidth="5.0" fill="TRANSPARENT" height="200.0" layoutX="400.0"
                   layoutY="150.0" stroke="BLACK" strokeType="INSIDE" width="200.0"/>
        <Text fx:id="text" layoutX="417.0" layoutY="63.0" text="何にも接触していません。"/>
    </children>
</AnchorPane>

package application;

import javafx.fxml.FXML;
import javafx.fxml.Initializable;
import javafx.scene.layout.AnchorPane;
import javafx.scene.shape.Rectangle;
import javafx.scene.text.Text;

import java.net.URL;
import java.util.ArrayList;
import java.util.ResourceBundle;

public class Screen2Controller implements Initializable {
    @FXML private AnchorPane screen2;
    @FXML private Rectangle boundary1;
    @FXML private Rectangle shape1;
    @FXML private Text text;

    @Override public void initialize(URL url, ResourceBundle resourceBundle) {
        shape1.layoutXProperty().addListener((_, _, _) -> checkShape1());
        shape1.layoutYProperty().addListener((_, _, _) -> checkShape1());

        screen2.setOnKeyPressed(e -> {
            switch (e.getCode()) {
                case E -> screen2.getChildren().add(new RandomCircle());
                case SPACE -> {
                    shape1.setLayoutX(450);
                    shape1.setLayoutY(400);

                    var circles = new ArrayList<RandomCircle>();
                    for (var child : screen2.getChildren()) {
                        if (child instanceof RandomCircle circle) {
                            circle.stop();
                            circles.add(circle);
                        }
                    }
                    screen2.getChildren().removeAll(circles);
                }
            }
        });
    }

    private void checkShape1() {
        var boundshape1 = shape1.getBoundsInParent();
        var boundboundary1 = boundary1.getBoundsInParent();
        var included = boundboundary1.contains(boundshape1);
        var touch = boundboundary1.intersects(boundshape1);

        if (included) text.setText("内部に入っています。");
        else if (touch) text.setText("当たっています。");
        else text.setText("何にも接触していません。");

        var top = 0d;
        var bottom = screen2.getHeight() - shape1.getHeight();
        var left = 0d;
        var right = screen2.getWidth() - shape1.getWidth();

        if (shape1.getLayoutY() <= top) shape1.setLayoutY(top);
        if (shape1.getLayoutY() >= bottom) shape1.setLayoutY(bottom);
        if (shape1.getLayoutX() <= left) shape1.setLayoutX(left);
        if (shape1.getLayoutX() >= right) shape1.setLayoutX(right);
    }
}

　

package application;

import javafx.animation.AnimationTimer;
import javafx.beans.binding.BooleanBinding;
import javafx.beans.property.BooleanProperty;
import javafx.beans.property.SimpleBooleanProperty;
import javafx.scene.shape.Rectangle;

public class Shape1 extends Rectangle {
    private final BooleanProperty wPressed = new SimpleBooleanProperty();
    private final BooleanProperty aPressed = new SimpleBooleanProperty();
    private final BooleanProperty sPressed = new SimpleBooleanProperty();
    private final BooleanProperty dPressed = new SimpleBooleanProperty();
    private final BooleanProperty shiftPressed = new SimpleBooleanProperty();
    private final BooleanBinding keyPressed = wPressed.or(aPressed).or(sPressed).or(dPressed).or(shiftPressed);

    private final AnimationTimer timer = new AnimationTimer() {
        @Override public void handle(long timestamp) {
            var offset = shiftPressed.get() ? 5 : 2;

            if (wPressed.get()) setLayoutY(getLayoutY() - offset);
            if (sPressed.get()) setLayoutY(getLayoutY() + offset);
            if (aPressed.get()) setLayoutX(getLayoutX() - offset);
            if (dPressed.get()) setLayoutX(getLayoutX() + offset);
        }
    };

    public Shape1() {
        setOnKeyPressed(e -> {
            switch (e.getCode()) {
                case W -> wPressed.set(true);
                case A -> aPressed.set(true);
                case S -> sPressed.set(true);
                case D -> dPressed.set(true);
                case SHIFT -> shiftPressed.set(true);
            }
        });

        setOnKeyReleased(e -> {
            switch (e.getCode()) {
                case W -> wPressed.set(false);
                case A -> aPressed.set(false);
                case S -> sPressed.set(false);
                case D -> dPressed.set(false);
                case SHIFT -> shiftPressed.set(false);
            }
        });

        keyPressed.addListener((_, aBoolean, _) -> {
            if (!aBoolean) timer.start();
            else timer.stop();
        });
    }
}

　

package application;

import javafx.animation.AnimationTimer;
import javafx.scene.paint.Color;
import javafx.scene.shape.Ellipse;

public class RandomCircle extends Ellipse {
    private final AnimationTimer timer = new AnimationTimer() {
        @Override public void handle(long timestamp) {
            setCenterX(getCenterX() + Math.random() * 10 - 5);
            setCenterY(getCenterY() + Math.random() * 10 - 5);
        }
    };

    public RandomCircle() {
        setRadiusX(20);
        setRadiusY(20);
        setCenterX(Math.random() * 100 + 100);
        setCenterY(Math.random() * 100 + 100);
        setFill(Color.color(Math.random(), Math.random(), Math.random()));
        timer.start();
    }
    public void stop() { timer.stop(); }
}

　

package application;

import javafx.application.Application;
import javafx.fxml.FXMLLoader;
import javafx.scene.Scene;
import javafx.stage.Stage;

import java.io.IOException;

public class HelloApplication extends Application {
    public static void main(String[] args) { launch(); }

    @Override public void start(Stage stage) throws IOException {
        var fxmlLoader = new FXMLLoader(HelloApplication.class.getResource("screen2.fxml"));
        stage.setScene(new Scene(fxmlLoader.load()));
        stage.show();
    }
}

Answer 2

・FXMLのオブジェクト（今回の場合は短形を）のコントロールを他のクラスに譲渡できるのか？

fx:includeでできました^^
Scene Builder で他の FXML ファイルを埋め込む #Java - Qiita

ただ何をどう分けるのかは悩ましいですね...

• 移動自体はShape1Controllerに任せた
• 当たり判定等はScreen2Controller側で

<?xml version="1.0" encoding="UTF-8"?>

<?import application.Shape1?>
<?import javafx.scene.layout.AnchorPane?>
<?import javafx.scene.shape.Rectangle?>
<?import javafx.scene.text.Text?>
<AnchorPane fx:id="screen2"
            prefHeight="500.0" prefWidth="1000.0"
            xmlns:fx="http://javafx.com/fxml" fx:controller="application.Screen2Controller">
    <children>
        <!-- <fx:include fx:id="shape1" source="shape1.fxml"/> -->
        <!-- ↑の代わりに継承クラスを使った場合 -->
        <Shape1 fx:id="shape1" arcHeight="5.0" arcWidth="5.0" fill="#ff8c1f" height="100.0" layoutX="450.0"
                layoutY="400.0"
                focusTraversable="true" stroke="BLACK" strokeType="INSIDE" width="100.0"/>

        <Rectangle fx:id="boundary1" arcHeight="5.0" arcWidth="5.0" fill="TRANSPARENT" height="200.0" layoutX="400.0"
                   layoutY="150.0" stroke="BLACK" strokeType="INSIDE" width="200.0"/>
        <Text fx:id="text" layoutX="417.0" layoutY="63.0" text="何にも接触していません。"/>
    </children>
</AnchorPane>

package application;

import javafx.fxml.FXML;
import javafx.fxml.Initializable;
import javafx.scene.input.KeyCode;
import javafx.scene.layout.AnchorPane;
import javafx.scene.shape.Rectangle;
import javafx.scene.text.Text;

import java.net.URL;
import java.util.ResourceBundle;

public class Screen2Controller implements Initializable {
    @FXML private AnchorPane screen2;
    @FXML private Rectangle boundary1;
    @FXML private Rectangle shape1;
    @FXML private Text text;
//    @FXML private Shape1Controller shape1Controller; // コントローラも取れる

    @Override public void initialize(URL url, ResourceBundle resourceBundle) {
        shape1.layoutXProperty().addListener((_, _, _) -> checkShape1());
        shape1.layoutYProperty().addListener((_, _, _) -> checkShape1());

        screen2.setOnKeyPressed(e -> {
            if (e.getCode() == KeyCode.SPACE) {
                shape1.setLayoutX(450);
                shape1.setLayoutY(400);
            }
        });
    }

    private void checkShape1() {
        var boundshape1 = shape1.getBoundsInParent();
        var boundboundary1 = boundary1.getBoundsInParent();
        var included = boundboundary1.contains(boundshape1);
        var touch = boundboundary1.intersects(boundshape1);

        if (included) text.setText("内部に入っています。");
        else if (touch) text.setText("当たっています。");
        else text.setText("何にも接触していません。");

        var top = 0d;
        var bottom = screen2.getHeight() - shape1.getHeight();
        var left = 0d;
        var right = screen2.getWidth() - shape1.getWidth();

        if (shape1.getLayoutY() <= top) shape1.setLayoutY(top);
        if (shape1.getLayoutY() >= bottom) shape1.setLayoutY(bottom);
        if (shape1.getLayoutX() <= left) shape1.setLayoutX(left);
        if (shape1.getLayoutX() >= right) shape1.setLayoutX(right);
    }
}

　

<?xml version="1.0" encoding="UTF-8"?>

<?import javafx.scene.shape.Rectangle?>
<Rectangle fx:id="shape1" arcHeight="5.0" arcWidth="5.0" fill="#ff8c1f" height="100.0" layoutX="450.0" layoutY="400.0"
           focusTraversable="true" stroke="BLACK" strokeType="INSIDE" width="100.0" xmlns:fx="http://javafx.com/fxml"
           fx:controller="application.Shape1Controller"/>

package application;

import javafx.animation.AnimationTimer;
import javafx.beans.binding.BooleanBinding;
import javafx.beans.property.BooleanProperty;
import javafx.beans.property.SimpleBooleanProperty;
import javafx.fxml.FXML;
import javafx.fxml.Initializable;
import javafx.scene.shape.Rectangle;

import java.net.URL;
import java.util.ResourceBundle;

public class Shape1Controller implements Initializable {
    @FXML private Rectangle shape1;

    private final BooleanProperty wPressed = new SimpleBooleanProperty();
    private final BooleanProperty aPressed = new SimpleBooleanProperty();
    private final BooleanProperty sPressed = new SimpleBooleanProperty();
    private final BooleanProperty dPressed = new SimpleBooleanProperty();
    private final BooleanProperty shiftPressed = new SimpleBooleanProperty();
    private final BooleanBinding keyPressed = wPressed.or(aPressed).or(sPressed).or(dPressed).or(shiftPressed);

    private final AnimationTimer timer = new AnimationTimer() {
        @Override public void handle(long timestamp) {
            var offset = shiftPressed.get() ? 5 : 2;

            if (wPressed.get()) shape1.setLayoutY(shape1.getLayoutY() - offset);
            if (sPressed.get()) shape1.setLayoutY(shape1.getLayoutY() + offset);
            if (aPressed.get()) shape1.setLayoutX(shape1.getLayoutX() - offset);
            if (dPressed.get()) shape1.setLayoutX(shape1.getLayoutX() + offset);
        }
    };

    @Override public void initialize(URL url, ResourceBundle resourceBundle) {
        shape1.setOnKeyPressed(e -> {
            switch (e.getCode()) {
                case W -> wPressed.set(true);
                case A -> aPressed.set(true);
                case S -> sPressed.set(true);
                case D -> dPressed.set(true);
                case SHIFT -> shiftPressed.set(true);
            }
        });

        shape1.setOnKeyReleased(e -> {
            switch (e.getCode()) {
                case W -> wPressed.set(false);
                case A -> aPressed.set(false);
                case S -> sPressed.set(false);
                case D -> dPressed.set(false);
                case SHIFT -> shiftPressed.set(false);
            }
        });

        keyPressed.addListener((_, aBoolean, _) -> {
            if (!aBoolean) timer.start();
            else timer.stop();
        });
    }
}

　

package application;

import javafx.animation.AnimationTimer;
import javafx.beans.binding.BooleanBinding;
import javafx.beans.property.BooleanProperty;
import javafx.beans.property.SimpleBooleanProperty;
import javafx.scene.shape.Rectangle;

public class Shape1 extends Rectangle {
    private final BooleanProperty wPressed = new SimpleBooleanProperty();
    private final BooleanProperty aPressed = new SimpleBooleanProperty();
    private final BooleanProperty sPressed = new SimpleBooleanProperty();
    private final BooleanProperty dPressed = new SimpleBooleanProperty();
    private final BooleanProperty shiftPressed = new SimpleBooleanProperty();
    private final BooleanBinding keyPressed = wPressed.or(aPressed).or(sPressed).or(dPressed).or(shiftPressed);

    private final AnimationTimer timer = new AnimationTimer() {
        @Override public void handle(long timestamp) {
            var offset = shiftPressed.get() ? 5 : 2;

            if (wPressed.get()) setLayoutY(getLayoutY() - offset);
            if (sPressed.get()) setLayoutY(getLayoutY() + offset);
            if (aPressed.get()) setLayoutX(getLayoutX() - offset);
            if (dPressed.get()) setLayoutX(getLayoutX() + offset);
        }
    };

    public Shape1() {
        setOnKeyPressed(e -> {
            switch (e.getCode()) {
                case W -> wPressed.set(true);
                case A -> aPressed.set(true);
                case S -> sPressed.set(true);
                case D -> dPressed.set(true);
                case SHIFT -> shiftPressed.set(true);
            }
        });

        setOnKeyReleased(e -> {
            switch (e.getCode()) {
                case W -> wPressed.set(false);
                case A -> aPressed.set(false);
                case S -> sPressed.set(false);
                case D -> dPressed.set(false);
                case SHIFT -> shiftPressed.set(false);
            }
        });

        keyPressed.addListener((_, aBoolean, _) -> {
            if (!aBoolean) timer.start();
            else timer.stop();
        });
    }
}

　

package application;

import javafx.application.Application;
import javafx.fxml.FXMLLoader;
import javafx.scene.Scene;
import javafx.stage.Stage;

import java.io.IOException;

public class HelloApplication extends Application {
    public static void main(String[] args) { launch(); }

    @Override public void start(Stage stage) throws IOException {
        var fxmlLoader = new FXMLLoader(HelloApplication.class.getResource("screen2.fxml"));
        stage.setScene(new Scene(fxmlLoader.load()));
        stage.show();
    }
}

Comments

[TN8001]

Rectangleを継承したクラスを作ったほうがシンプルな気もする^^;

[Fitz]

回答いただきありがとうございます。いくつか質問させてください。
自身のやり方でNULLが発生した原因
→短形の初期化ができていなかったのでNULLが発生していたと認識していますが、合っているでしょうか？

コードについて
①Screen2Controller.java16行目
@FXML private Rectangle shape1;でScreen2Controller側でshape1を取得できているのは、FXMLでIDを記載したからでしょうか？

②Screen2Controller.java20行目
shape1.layoutXProperty().addListener((_, _, _) -> checkShape1());
→座標の変化を上記のコードで検知、addListener((_, _, _) でcheckShape1()を起動といった動きになるのでしょうか？

➂Shape1Controller.java：25行目
var offset = shiftPressed.get() ? 5 : 2;について、shiftキーでスピードが上がる記述をしているように見えますが、このような記述方法を始めてみました。どういった記述方法なのでしょうか？

➃Rectangleを継承したクラスを作ったほうがシンプル
→「shape1自体FXMLで作成せず、継承クラスで作成、Screen2Controllerに値を渡す」といった意味でしょうか？

[TN8001]

> →短形の初期化ができていなかったのでNULLが発生していたと認識していますが、合っているでしょうか？

どうやったかが分からないので何とも言えませんが、fx:idを書き忘れたとかタイポとかでしょうかね？

> @FXML private Rectangle shape1;でScreen2Controller側でshape1を取得できているのは、FXMLでIDを記載したからでしょうか？

はい。
「fx:id="shape1"」と型・名前ともにあっていれば取得できるはずです。

> →座標の変化を上記のコードで検知、addListener((_, _, _) でcheckShape1()を起動といった動きになるのでしょうか？

そうです。
「Shape1Controllerで当たり判定までするのは違うかな？」と思ったんでこのようにしました。
場合によってはController同士で変更しあって、無限ループしたりするかもしれませんｗ

> var offset = shiftPressed.get() ? 5 : 2;について、shiftキーでスピードが上がる記述をしているように見えますが、このような記述方法を始めてみました。どういった記述方法なのでしょうか？

「三項演算子」です。↓と同じ意味です^^
int offset;
if (shiftPressed.get()) offset = 5;
else offset = 2;

[java 三項演算子 - Google 検索](https://www.google.com/search?q=java+%E4%B8%89%E9%A0%85%E6%BC%94%E7%AE%97%E5%AD%90)

> →「shape1自体FXMLで作成せず、継承クラスで作成、Screen2Controllerに値を渡す」といった意味でしょうか？

Shape1Controllerに書いたコードを「public class Shape1 extends Rectangle」に移せば、shape1.fxmlがいらなくなります。

ただ実際はもっと凝った内容なんですかね？だとしたら今の構成のほうがいいんかな？？

[Fitz]

ベスト選択後に申し訳ありません。
「Shape1Controllerに書いたコードを「public class Shape1 extends Rectangle」に移せば、shape1.fxmlがいらなくなります。」
とのことですが、コードのイメージがつかめておりません。
Rectangleを継承→＠FXML shape1とshape1.fxmlを削除→コンストラクタでRectangle(x,y,w,h)を呼び出し→Screen2Controllerで図形を作成→画面に表示
という流れでしょうか？
また、表示のために再度stageとsceneの取得が必要ということでしょうか？

[TN8001]

> ベスト選択後に申し訳ありません。

回答に関する疑問はいつでもだれでもいいですよ^^

> とのことですが、コードのイメージがつかめておりません。

回答を修正しました（Shape1.javaを追加・screen2.fxmlを変更）
伝わりましたでしょうか。

[Fitz]

追記いただきありがとうございます。
<?import application.Shape1?>でコンポーネントの紐づけを行い、動かすことができました。
この記述の重要箇所は、ラベルをRectangleから継承したShape1に変更することでしょうか？
Rectangle要素＋移動要素を持ったShape1にすることで、クラスから直接短形を動かすことができるという認識です。
また、Shape1クラスのAnimationTimer についてですが、こちらは疑問が2つあります。
①「Shape1.this.setLayoutY(Shape1.this.getLayoutY() - offset)」といったような記述になるのは、自クラス（継承したRectangleクラス）のメソッドを呼び出したいからでしょうか？
②AnimationTimer ですが、今後、他のクラスで別途定義した場合でも、匿名クラスで作成していれば問題なく動作するのでしょうか？例えば、ランダムに動く別の短形を動かす場合等々。
リンク先の資料見た感じその場で定義・インスタンス化しているので、他クラスで再びAnimationTimerを匿名クラスで作成しても別インスタンス扱いで動作できるのかな？と思いました。

[TN8001]

> <?import application.Shape1?>でコンポーネントの紐づけを行い、動かすことができました。

すいません。追記忘れました^^;

> この記述の重要箇所は、ラベルをRectangleから継承したShape1に変更することでしょうか？

「ラベル」というのは<Rectangleとかのことですか？（HTMLで言うところのタグ、XMLだと要素）
「重要」というかShape1をどこにも使わなかったら意味がないので^^;

> Rectangle要素＋移動要素を持ったShape1にすることで、クラスから直接短形を動かすことができるという認識です。

まあそうですね。
でも例えば「円も動かしたくなったらどうすんの？ほとんど同じなclass Shape2 extends Ellipseクラスも作りますか？」みたいなことは思います（「よろしくないよね」という意味で）

回答コードは現状出ている条件で、（わたしが考える）一番シンプルというか素直な実装というだけで、ベストかどうかはわかりません。
十人十色とまでは言いませんが、色々なアプローチが考えられます。

> ①「Shape1.this.setLayoutY(Shape1.this.getLayoutY() - offset)」といったような記述になるのは、自クラス（継承したRectangleクラス）のメソッドを呼び出したいからでしょうか？

すいません。ここは「setLayoutY(getLayoutY() - offset)」でよかったです。
「Shape1.this.」も冗長なだけで間違いではないです（「this.setLayoutY(」とは書けません）

> ②AnimationTimer ですが、今後、他のクラスで別途定義した場合でも、匿名クラスで作成していれば問題なく動作するのでしょうか？例えば、ランダムに動く別の短形を動かす場合等々。

例えば↓のようなのを追加してみてください（<Shape1 fx:id="shape1"は残したまま）

<Shape1 fill="RED" layoutX="150.0" layoutY="150.0" width="100.0" height="100.0" focusTraversable="true"/>

[Tab]キーを押すとキーボードフォーカスが移動し、もう一方の四角が動くはずです（当たり判定はオレンジしかチェックしていないので赤は当然判定なし）

1000個とかあるとどうなるかわかりませんが、数十個程度なら大丈夫じゃないでしょうか（問題が出たらその時考えましょう）

「ランダムに動く円」を追加するように別回答しました（全然面白くないｗ