回答編集履歴

- [JavaFX - JavaFXのTableColumnに中身を追加したい｜teratail](https://teratail.com/questions/331339)

- 過去質問で解決したはずの部分まで壊れているのはなんででしょうか？

- 試行錯誤は大変結構ですが、手あたり次第に勘で書き換えてもうまくいきません。

- `TableView`はもともと難しいので、１行１行どういう意味かザックリでも理解しながらやってください。

-

- まず[TableColumn<S,T>](https://docs.oracle.com/javase/jp/8/javafx/api/javafx/scene/control/TableColumn.html)の型引数が逆です。

- そのため`ScheduleDataCell`も逆になってしまい、ヌルポがでます。

-

- `setCellValueFactory`があったはずですが、丸々抜けています。

- 通常は表示するプロパティを指定するわけですが、セルにまとめて表示したいわけですね？

- 単に改行したいだけなら各プロパティを`concat`するのが簡単です。

-

- あくまで`ScheduleDataCell`を利用したいなら、アイテム自身（`ScheduleData`）をセルに渡す必要がありますが、[ObservableValue<T>](https://docs.oracle.com/javase/jp/8/javafx/api/javafx/beans/value/ObservableValue.html)でなければなりません。

- [TableColumn#setCellValueFactory](https://docs.oracle.com/javase/jp/8/javafx/api/javafx/scene/control/TableColumn.html#setCellValueFactory-javafx.util.Callback-)

-

- 単純に[ReadOnlyObjectWrapper<T>](https://docs.oracle.com/javase/jp/8/javafx/api/javafx/beans/property/ReadOnlyObjectWrapper.html)で包んでみたところ、編集が即時反映されませんでした（まあ当然か）

-

- ちょっと正しいか自信はないのですが、`ScheduleData`を`ObjectBinding`にして内部のプロパティとバインドしたところ想定した動作になりました。

-

- ```fxml

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

-

- <?import javafx.scene.control.TableColumn?>

- <?import javafx.scene.control.TableView?>

- <?import javafx.scene.layout.BorderPane?>

- <BorderPane xmlns="http://javafx.com/javafx/11.0.1" xmlns:fx="http://javafx.com/fxml/1"

- fx:controller="sample.ScheduleTableController">

- <center>

- <TableView fx:id="table" editable="true">

- <columns>

- <TableColumn fx:id="scheduleColumn" prefWidth="100" text="schedule"/>

- <TableColumn fx:id="concatColumn" prefWidth="100" text="concat"/>

- <TableColumn fx:id="titleColumn" prefWidth="100" text="title"/>

- <TableColumn fx:id="timeColumn" prefWidth="100" text="time"/>

- <TableColumn fx:id="detailColumn" prefWidth="100" text="detail"/>

- </columns>

- </TableView>

- </center>

- </BorderPane>

- ```

-

- ```Java

- package sample;

-

- import java.net.URL;

- import java.time.LocalTime;

- import java.util.ResourceBundle;

-

- import javafx.beans.property.ReadOnlyObjectWrapper;

- import javafx.collections.FXCollections;

- import javafx.collections.ObservableList;

- import javafx.fxml.FXML;

- import javafx.fxml.Initializable;

- import javafx.scene.control.TableCell;

- import javafx.scene.control.TableColumn;

- import javafx.scene.control.TableView;

- import javafx.scene.control.cell.PropertyValueFactory;

- import javafx.scene.control.cell.TextFieldTableCell;

- import javafx.scene.layout.VBox;

- import javafx.scene.text.Text;

-

-

- public class ScheduleTableController implements Initializable {

- @FXML

- private TableView<ScheduleData> table;

- @FXML

- private TableColumn<ScheduleData, ScheduleData> scheduleColumn; // アイテム自身をセルに出すため<ScheduleData, ScheduleData>になる

- @FXML

- private TableColumn<ScheduleData, String> concatColumn; // 通常はアイテムの型（TableView<T>のT）, プロパティの型（StringPropertyならString）

- @FXML

- private TableColumn<ScheduleData, String> titleColumn;

- @FXML

- private TableColumn<ScheduleData, String> timeColumn;

- @FXML

- private TableColumn<ScheduleData, String> detailColumn;

-

- @Override public void initialize(URL location, ResourceBundle resources) {

- // ScheduleDataがObjectBindingでなくとも、Wrapperでコード上は通る

- // が、編集した時の挙動が異なる

- // 即時反映されずセルのクリックでupdateItemが走る模様

- // scheduleColumn.setCellValueFactory(x -> new ReadOnlyObjectWrapper<>(x.getValue()));

-

- // これでいいのか自信はないが、とりあえず想定通りに動いている

- scheduleColumn.setCellValueFactory(x -> x.getValue());

- scheduleColumn.setCellFactory(x -> new ScheduleDataCell());

-

-

- // 単にまとめたいだけならつなげればいい

- concatColumn.setCellValueFactory(x ->

- x.getValue().titleProperty()

- .concat("\n")

- .concat(x.getValue().timeProperty())

- .concat("\n")

- .concat(x.getValue().detailProperty()));

-

-

- // titleColumn.setCellValueFactory(new PropertyValueFactory<>("title")); // ↓同じ意味

- titleColumn.setCellValueFactory(x -> x.getValue().titleProperty());

- titleColumn.setCellFactory(TextFieldTableCell.forTableColumn());

-

- timeColumn.setCellValueFactory(x -> x.getValue().timeProperty());

- timeColumn.setCellFactory(TextFieldTableCell.forTableColumn());

-

- detailColumn.setCellValueFactory(x -> x.getValue().detailProperty());

- detailColumn.setCellFactory(TextFieldTableCell.forTableColumn());

-

- ObservableList<ScheduleData> schedules = FXCollections.observableArrayList(

- new ScheduleData("部活", LocalTime.of(12, 30), LocalTime.of(16, 0), "試合"),

- new ScheduleData("部活", LocalTime.of(12, 30), LocalTime.of(16, 0), "試合"));

- table.setItems(schedules);

- }

- }

-

- class ScheduleDataCell extends TableCell<ScheduleData, ScheduleData> {

- private VBox cellContainer = new VBox();

- private Text scheduleTitle = new Text();

- private Text time = new Text();

- private Text txtDetail = new Text();

-

- public ScheduleDataCell() {

- cellContainer.getChildren().addAll(scheduleTitle, time, txtDetail);

- }

-

- @Override protected void updateItem(ScheduleData item, boolean empty) {

- super.updateItem(item, empty);

-

- if (!empty) {

- scheduleTitle.setText(item.titleProperty().get());

- time.setText(item.timeProperty().get());

- txtDetail.setText(item.detailProperty().get());

-

- // これでもよさそうな気もするが、いつunbindすればいいかよくわからない

- // scheduleTitle.textProperty().bind(item.titleProperty());

- // time.textProperty().bind(item.timeProperty());

- // txtDetail.textProperty().bind(item.detailProperty());

-

- setGraphic(cellContainer);

- }

- }

- }

- ```

-

- ```Java

- package sample;

-

- import java.time.LocalTime;

-

- import javafx.beans.binding.ObjectBinding;

- import javafx.beans.property.SimpleStringProperty;

- import javafx.beans.property.StringProperty;

-

-

- public class ScheduleData extends ObjectBinding<ScheduleData> {

- private StringProperty title = new SimpleStringProperty();

- private StringProperty time = new SimpleStringProperty();

- private StringProperty detail = new SimpleStringProperty();

-

- public ScheduleData(String title, LocalTime startTime, LocalTime finishTime, String detail) {

- this.title.set(title);

- time.set(startTime + "～" + finishTime);

- this.detail.set(detail);

-

- bind(this.title, time, this.detail);

- }

-

- public StringProperty titleProperty() { return title; }

-

- public StringProperty timeProperty() { return time; }

-

- public StringProperty detailProperty() { return detail; }

-

- @Override protected ScheduleData computeValue() { return this; }

- }

- ```

-

- ```Java

- package sample;

-

- 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 {

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

-

- @Override public void start(Stage primaryStage) throws Exception {

- Parent root = FXMLLoader.load(getClass().getResource("sample.fxml"));

- primaryStage.setScene(new Scene(root, 600, 300));

- primaryStage.show();

- }

- }

- ```

-

- ---

-

- そもそも`TableView`は表形式で編集するのに便利なコンポーネントだと思うのですが、まとめて表示の意味は何なのでしょう？

-

- ---

-

- [Java - TableViewにデータを追加したい｜teratail](https://teratail.com/questions/332466)

- と全く同じ質問ですよね？

- [推奨していない質問｜teratail（テラテイル）](https://teratail.com/help/avoid-asking)

-