Controller同士の参照について

Question

###前提のソースコード

// Main.java
import javafx.application.Application;
import javafx.fxml.FXMLLoader;
import javafx.scene.Parent;
import javafx.scene.Scene;
import javafx.stage.Stage;

public class Main extends Application {

    @Override
    public void start(Stage primaryStage) throws Exception{
        Parent root = FXMLLoader.load(getClass().getResource("sample.fxml"));
        primaryStage.setTitle("Hello World");
        primaryStage.setScene(new Scene(root, 300, 275));
        primaryStage.show();
    }

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

// Pane1Controller.java
import javafx.event.ActionEvent;
import javafx.fxml.FXML;
import javafx.scene.control.Button;
import javafx.scene.control.Label;

public class Pane1Controller {
	@FXML
	private Label label;
	@FXML
	private Button button;

	private SampleController sampleController;

	public SampleController getSampleController() {
		return sampleController;
	}

	public void setSampleController(SampleController sampleController) {
		this.sampleController = sampleController;
	}

	@FXML
	private void onButton(ActionEvent event) {
		label.setText(sampleController.getPane2Controller().getTextField().getText());
	}
}

// Pane2Controller.java
import javafx.fxml.FXML;
import javafx.scene.control.TextField;

public class Pane2Controller {
	@FXML
	private TextField textField;

	private SampleController sampleController;

	public SampleController getSampleController() {
		return sampleController;
	}

	public void setSampleController(SampleController sampleController) {
		this.sampleController = sampleController;
	}

	public TextField getTextField() {
		return textField;
	}
}

// SampleController.java
import javafx.fxml.FXML;
import javafx.fxml.Initializable;

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

public class SampleController implements Initializable {
	@FXML
	private Pane1Controller pane1Controller;

	@FXML
	private Pane2Controller pane2Controller;

	@Override
	public void initialize(URL location, ResourceBundle resources) {
		pane1Controller.setSampleController(this);
		pane2Controller.setSampleController(this);
	}

	public Pane1Controller getPane1Controller() {
		return pane1Controller;
	}

	public Pane2Controller getPane2Controller() {
		return pane2Controller;
	}
}

<!-- pane1.fxml -->
<?xml version="1.0" encoding="UTF-8"?>

<?import javafx.scene.control.Button?>
<?import javafx.scene.control.Label?>
<?import javafx.scene.layout.HBox?>
<HBox maxHeight="-Infinity" maxWidth="-Infinity" minHeight="-Infinity" minWidth="-Infinity" prefHeight="400.0"
      prefWidth="600.0" xmlns="http://javafx.com/javafx/8.0.111" xmlns:fx="http://javafx.com/fxml/1"
      fx:controller="Pane1Controller">
    <children>
        <Label fx:id="label"/>
        <Button fx:id="button" mnemonicParsing="false" onAction="#onButton" text="Button"/>
    </children>
</HBox>


<!-- pane2.fxml -->
<?xml version="1.0" encoding="UTF-8"?>

<?import javafx.scene.control.TextField?>
<?import javafx.scene.layout.Pane?>
<Pane maxHeight="-Infinity" maxWidth="-Infinity" minHeight="-Infinity" minWidth="-Infinity" prefHeight="400.0"
      prefWidth="600.0" xmlns="http://javafx.com/javafx/8.0.111" xmlns:fx="http://javafx.com/fxml/1"
        fx:controller="Pane2Controller">
    <children>
        <TextField fx:id="textField"/>
    </children>
</Pane>

<!-- sample.fxml -->
<?xml version="1.0" encoding="UTF-8"?>

<?import javafx.scene.control.Tab?>
<?import javafx.scene.control.TabPane?>
<TabPane maxHeight="-Infinity" maxWidth="-Infinity" minHeight="-Infinity" minWidth="-Infinity" prefHeight="400.0"
         prefWidth="600.0" tabClosingPolicy="UNAVAILABLE" xmlns:fx="http://javafx.com/fxml/1"
         xmlns="http://javafx.com/javafx/8.0.111"
        fx:controller="SampleController">
    <tabs>
        <Tab text="Pane1">
            <content>
                <fx:include source="pane1.fxml" fx:id="pane1"/>
            </content>
        </Tab>
        <Tab text="Pane2">
            <content>
                <fx:include source="pane2.fxml" fx:id="pane2"/>
            </content>
        </Tab>
    </tabs>
</TabPane>

上記ソースでPane1ControllerからPane2Controllerを参照する上記以外の良い, 汎用性のある書き方は無いでしょうか？

動作環境: JDK8 - 111

Answer 1

Pane1/Pane2での情報の共有はそのままにすると仮定しますと、例えば以下のようにしてPane1/Pane2の結合を無くしておくというのはどうでしょう。SampleとPane1の間に新たにインターフェースが生じるので単純に結合度が下がる訳ではないのですが、この文字列情報がPane2ではなく他の場所に置くような変更になったとしてもPane1に影響がない点に価値が見いだせるならありかもしれません。

public class Sample ... {
  ...
  public ObjectProperty<String> somethingProperty() {
    return pane2.somethingProperty();
  }
}

public class Pane1Controller ... {
  ...
  void foo() {
    // Pane2と直接やりとりする代わりにParent(Sample)とのやりとりとしてみる
    ... sampleController.somethingProperty().get() ...
    ...
  }
}

public class Pane2Controller ... {
  ...
  public ObjectProperty<String> somethingProperty() {
    return textField.textProperty();
  }
}

もしこうしたことが多いならTabPaneで分ける情報が充分独立していない点に問題があるのかも知れません。ご質問のコードは実際より単純化してあると思いますので実際の複雑さはわかりませんが、例えばFXMLは分けてControllerは単一にするといった道もあるかも知れません。

Sample, Pane1, Pane2でどんな情報をどう操作したいかにより何に注目して設計したいかが変わってくる気がします。