CheckBoxTreeCell に対応するセル実装はありません。これは、CheckBoxTreeItem の selected/indeterminate プロパティにバインドされた checkBox を持つセルです。明らかに対応する CheckBoxTreeTableCell は、セル データにバインドされた、チェックボックスを含む単純なセルです。
必要なのは CheckBoxTreeTableRow です。これは、TreeItem にアクセスでき、checkBox と treeItem の間のバインディングを管理できるセル層です。以下は、CheckBoxTreeCell の簡単な実装、単純化および調整されたコピーです。un/binding は updateItem で処理されます。
更新：クリーンなソリューション（長い！）
TableRowSkinBase は、カスタムの行グラフィックを処理する準備ができているようです。これには、行スキン内のすべてのレイアウト コードで使用されるメソッド graphicsProperty() があります。
/**
* Returns the graphic to draw on the inside of the disclosure node. Null
* is acceptable when no graphic should be shown. Commonly this is the
* graphic associated with a TreeItem (i.e. treeItem.getGraphic()), rather
* than a graphic associated with a cell.
*/
protected abstract ObjectProperty<Node> graphicProperty();
TreeTableRowSkin は TreeItem のグラフィックを返すように実装しているため、tableRow のグラフィックを返すようにオーバーライドする必要があります。ただし...そうではありません-以下の元のハッキーな回答に記載されているように、レイアウトが曲がっています。掘り下げると、犯人が明らかになりました。それは、パディング内のグラフィックを考慮して、独自のレイアウト コードをハードコードする TreeTableCellSkin です ... treeItem のグラフィック。
したがって、完全なソリューションが必要です
- treeItem グラフィックスをハードコードしない Tree/TableCellSkin (以下の例はまだ完全にクリーンではありません。スーパーがグラフィックス幅を追加し、それを再度減算することに依存しています)
- 拡張スキンをインストールする Tree/TableCell
- 必要に応じて graphicsProperty をオーバーライドする Tree/TableRowSkin。行のグラフィックを返します。
- 必要に応じてグラフィックを更新する Tree/TableRow。以下で、そのグラフィックを checkBox に設定します。これには、treeItem のグラフィックが含まれる可能性があります。
最初のペアは DefaultTreeTableCell/Skin という名前で、2 番目のペアはその下の CheckBoxTreeTableRow/Skin という名前です。
使用法 (OP の例に挿入するスニペット)
// just for fun, have root items with some graphic
final CheckBoxTreeItem<Employee> root = new CheckBoxTreeItem<>(
new Employee("Sales Department", 0.0), new Circle(10, Color.RED));
final CheckBoxTreeItem<Employee> root2 = new CheckBoxTreeItem<>(
new Employee("Departments", 0.0), new Circle(10, Color.BLUE));
// configure treeTableView to use the extended tableRow
treeTableView.setRowFactory(item -> new CheckBoxTreeTableRow<>());
// configure table columns to use the extended table cell
empColumn.setCellFactory(p -> new DefaultTreeTableCell<>());
// all cell types must have a skin that copes with row graphics
salaryColumn.setCellFactory(e -> {
TreeTableCell cell = new ProgressBarTreeTableCell() {
@Override
protected Skin<?> createDefaultSkin() {
return new DefaultTreeTableCell.DefaultTreeTableCellSkin<>(this);
}
};
return cell;
});
セル/行の実装:
/**
* TreeTableCell actually showing something. This is copied from TreeTableColumn plus
* installs DefaultTreeTableCellSkin which handles row graphic width.
*/
public class DefaultTreeTableCell<S, T> extends TreeTableCell<S, T> {
@Override
protected void updateItem(T item, boolean empty) {
if (item == getItem()) return;
super.updateItem(item, empty);
if (item == null) {
super.setText(null);
super.setGraphic(null);
} else if (item instanceof Node) {
super.setText(null);
super.setGraphic((Node)item);
} else {
super.setText(item.toString());
super.setGraphic(null);
}
}
@Override
protected Skin<?> createDefaultSkin() {
return new DefaultTreeTableCellSkin<>(this);
}
/**
* TreeTableCellSkin that handles row graphic in its leftPadding, if
* it is in the treeColumn of the associated TreeTableView.
* <p>
* It assumes that per-row graphics - including the graphic of the TreeItem, if any -
* is folded into the TreeTableRow graphic and patches its leftLabelPadding
* to account for the graphic width.
* <p>
*
* Note: TableRowSkinBase seems to be designed to cope with variations of row
* graphic - it has a method <code>graphicProperty()</code> that's always used
* internally when calculating offsets in the treeColumn.
* Subclasses override as needed, the layout code remains constant. The real
* problem is the TreeTableCell hard-codes the TreeItem as the only graphic
* owner.
*
*/
public static class DefaultTreeTableCellSkin<S, T> extends TreeTableCellSkin<S, T> {
/**
* @param treeTableCell
*/
public DefaultTreeTableCellSkin(TreeTableCell<S, T> treeTableCell) {
super(treeTableCell);
}
/**
* Overridden to adjust the padding returned by super for row graphic.
*/
@Override
protected double leftLabelPadding() {
double padding = super.leftLabelPadding();
padding += getRowGraphicPatch();
return padding;
}
/**
* Returns the patch for leftPadding if the tableRow has a graphic of
* its own.<p>
*
* Note: this implemenation is a bit whacky as it relies on super's
* handling of treeItems graphics offset. A cleaner
* implementation would override leftLabelPadding from scratch.
* <p>
* PENDING JW: doooooo it!
*
* @return
*/
protected double getRowGraphicPatch() {
if (!isTreeColumn()) return 0;
Node graphic = getSkinnable().getTreeTableRow().getGraphic();
if (graphic != null) {
double height = getCellSize();
// start with row's graphic
double patch = graphic.prefWidth(height);
// correct for super's having added treeItem's graphic
TreeItem<S> item = getSkinnable().getTreeTableRow().getTreeItem();
if (item.getGraphic() != null) {
double correct = item.getGraphic().prefWidth(height);
patch -= correct;
}
return patch;
}
return 0;
}
/**
* Checks and returns whether our cell is attached to a treeTableView/column
* and actually has a TreeItem.
* @return
*/
protected boolean isTreeColumn() {
if (getSkinnable().isEmpty()) return false;
TreeTableColumn<S, T> column = getSkinnable().getTableColumn();
TreeTableView<S> view = getSkinnable().getTreeTableView();
if (column.equals(view.getTreeColumn())) return true;
return view.getVisibleLeafColumns().indexOf(column) == 0;
}
}
}
/**
* Support custom graphic for Tree/TableRow. Here in particular a checkBox.
* http://stackoverflow.com/q/29300551/203657
* <p>
* Basic idea: implement custom TreeTableRow that set's its graphic to the
* graphic/checkBox. Doesn't work: layout is broken, graphic appears
* over the text. All fine if we set the graphic to the TreeItem that's
* shown. Possible as long as the treeItem doesn't have a graphic of
* its own.
* <p>
* Basic problem:
* <li> TableRowSkinBase seems to be able to cope: has protected method
* graphicsProperty that should be implemented to return the graphic
* if any. That graphic is added to the children list and sized/located
* in layoutChildren.
* <li> are added the graphic/disclosureNode as needed before
* calling super.layoutChildren,
* <li> graphic/disclosure are placed inside the leftPadding of the tableCell
* that is the treeColumn
* <li> TreeTableCellSkin must cooperate in taking into account the graphic/disclosure
* when calculating its leftPadding
* <li> cellSkin is hard-coded to use the TreeItem's graphic (vs. the rowCell's)
*
* PENDING JW:
* <li>- would expect to not alter the scenegraph during layout (might lead to
* endless loops or not) but done frequently in core code
* <p>
*
* Outline of the solution as implemented:
* <li> need a TreeTableCell with a custom skin
* <li> override leftPadding in skin to add row graphic if available
* <li> need CheckBoxTreeTableRow that sets its graphic to checkBox (or a combination
* of checkBox and treeItem's)
* <li> need custom rowSkin that implements graphicProperty to return the row graphic
*
* @author Jeanette Winzenburg, Berlin
*
* @see DefaultTreeTableCell
* @see DefaultTreeTableCellSkin
*
*/
public class CheckBoxTreeTableRow<T> extends TreeTableRow<T> {
private CheckBox checkBox;
private ObservableValue<Boolean> booleanProperty;
private BooleanProperty indeterminateProperty;
public CheckBoxTreeTableRow() {
this(item -> {
if (item instanceof CheckBoxTreeItem<?>) {
return ((CheckBoxTreeItem<?>)item).selectedProperty();
}
return null;
});
}
public CheckBoxTreeTableRow(
final Callback<TreeItem<T>, ObservableValue<Boolean>> getSelectedProperty) {
this.getStyleClass().add("check-box-tree-cell");
setSelectedStateCallback(getSelectedProperty);
checkBox = new CheckBox();
checkBox.setAlignment(Pos.TOP_LEFT);
}
// --- selected state callback property
private ObjectProperty<Callback<TreeItem<T>, ObservableValue<Boolean>>>
selectedStateCallback =
new SimpleObjectProperty<Callback<TreeItem<T>, ObservableValue<Boolean>>>(
this, "selectedStateCallback");
/**
* Property representing the {@link Callback} that is bound to by the
* CheckBox shown on screen.
*/
public final ObjectProperty<Callback<TreeItem<T>, ObservableValue<Boolean>>> selectedStateCallbackProperty() {
return selectedStateCallback;
}
/**
* Sets the {@link Callback} that is bound to by the CheckBox shown on screen.
*/
public final void setSelectedStateCallback(Callback<TreeItem<T>, ObservableValue<Boolean>> value) {
selectedStateCallbackProperty().set(value);
}
/**
* Returns the {@link Callback} that is bound to by the CheckBox shown on screen.
*/
public final Callback<TreeItem<T>, ObservableValue<Boolean>> getSelectedStateCallback() {
return selectedStateCallbackProperty().get();
}
/** {@inheritDoc} */
@Override
protected void updateItem(T item, boolean empty) {
super.updateItem(item, empty);
if (empty) {
setText(null);
setGraphic(null);
} else {
TreeItem<T> treeItem = getTreeItem();
checkBox.setGraphic(treeItem == null ? null : treeItem.getGraphic());
setGraphic(checkBox);
// uninstall bindings
if (booleanProperty != null) {
checkBox.selectedProperty().unbindBidirectional((BooleanProperty)booleanProperty);
}
if (indeterminateProperty != null) {
checkBox.indeterminateProperty().unbindBidirectional(indeterminateProperty);
}
// install new bindings.
// this can only handle TreeItems of type CheckBoxTreeItem
if (treeItem instanceof CheckBoxTreeItem) {
CheckBoxTreeItem<T> cbti = (CheckBoxTreeItem<T>) treeItem;
booleanProperty = cbti.selectedProperty();
checkBox.selectedProperty().bindBidirectional((BooleanProperty)booleanProperty);
indeterminateProperty = cbti.indeterminateProperty();
checkBox.indeterminateProperty().bindBidirectional(indeterminateProperty);
} else {
throw new IllegalStateException("item must be CheckBoxTreeItem");
}
}
}
@Override
protected Skin<?> createDefaultSkin() {
return new CheckBoxTreeTableRowSkin<>(this);
}
public static class CheckBoxTreeTableRowSkin<S> extends TreeTableRowSkin<S> {
protected ObjectProperty<Node> checkGraphic;
/**
* @param control
*/
public CheckBoxTreeTableRowSkin(TreeTableRow<S> control) {
super(control);
}
/**
* Note: this is implicitly called from the constructor of LabeledSkinBase.
* At that time, checkGraphic is not yet instantiated. So we do it here,
* still having to create it at least twice. That'll be a problem if
* anybody would listen to changes ...
*/
@Override
protected ObjectProperty<Node> graphicProperty() {
if (checkGraphic == null) {
checkGraphic = new SimpleObjectProperty<Node>(this, "checkGraphic");
}
CheckBoxTreeTableRow<S> treeTableRow = getTableRow();
if (treeTableRow.getTreeItem() == null) {
checkGraphic.set(null);
} else {
checkGraphic.set(treeTableRow.getGraphic());
}
return checkGraphic;
}
protected CheckBoxTreeTableRow<S> getTableRow() {
return (CheckBoxTreeTableRow<S>) super.getSkinnable();
}
}
@SuppressWarnings("unused")
private static final Logger LOG = Logger
.getLogger(CheckBoxTreeTableRow.class.getName());
}
元の答え: ハック!
その中にはおかしなコード行があります:
treeItem.setGraphics(checkBox);
それは本当に奇抜で、おそらく最終的に大混乱を引き起こすでしょう.これは、TreeTableRowSkinのレイアウトの不具合に関するハックであり、何らかの理由で(私は掘り下げることができませんでした)、グラフィックセットをセルに配置できません. checkBox を直接返すカスタム CheckBoxTreeTableRowSkin で動作させることができませんでしたgraphicProperty()。
/**
* @author Jeanette Winzenburg, Berlin
*/
public class CheckBoxTreeTableRowHack<T> extends TreeTableRow<T> {
private CheckBox checkBox;
private ObservableValue<Boolean> booleanProperty;
private BooleanProperty indeterminateProperty;
public CheckBoxTreeTableRowHack() {
setSelectedStateCallback(item -> {
if (item instanceof CheckBoxTreeItem<?>) {
return ((CheckBoxTreeItem<?>)item).selectedProperty();
}
return null;
});
this.checkBox = new CheckBox();
// something weird going on with layout
checkBox.setAlignment(Pos.TOP_LEFT);
}
// --- selected state callback property
private ObjectProperty<Callback<TreeItem<T>, ObservableValue<Boolean>>>
selectedStateCallback =
new SimpleObjectProperty<Callback<TreeItem<T>, ObservableValue<Boolean>>>(
this, "selectedStateCallback");
/**
* Property representing the {@link Callback} that is bound to by the
* CheckBox shown on screen.
*/
public final ObjectProperty<Callback<TreeItem<T>, ObservableValue<Boolean>>> selectedStateCallbackProperty() {
return selectedStateCallback;
}
/**
* Sets the {@link Callback} that is bound to by the CheckBox shown on screen.
*/
public final void setSelectedStateCallback(Callback<TreeItem<T>, ObservableValue<Boolean>> value) {
selectedStateCallbackProperty().set(value);
}
/**
* Returns the {@link Callback} that is bound to by the CheckBox shown on screen.
*/
public final Callback<TreeItem<T>, ObservableValue<Boolean>> getSelectedStateCallback() {
return selectedStateCallbackProperty().get();
}
/** {@inheritDoc} */
@Override
public void updateItem(T item, boolean empty) {
super.updateItem(item, empty);
if (empty) {
setText(null);
setGraphic(null);
} else {
//
TreeItem<T> treeItem = getTreeItem();
// PENDING JW: this is nuts but working .. certainly will pose problems
// when re-using the cell
treeItem.setGraphic(checkBox);
// this is what CheckBoxTreeCell does, setting the graphic
// of the tableRow confuses the layout
// checkBox.setGraphic(treeItem == null ? null : treeItem.getGraphic());
// setGraphic(checkBox);
// uninstall bindings
if (booleanProperty != null) {
checkBox.selectedProperty().unbindBidirectional((BooleanProperty)booleanProperty);
}
if (indeterminateProperty != null) {
checkBox.indeterminateProperty().unbindBidirectional(indeterminateProperty);
}
// install new bindings.
// We special case things when the TreeItem is a CheckBoxTreeItem
if (treeItem instanceof CheckBoxTreeItem) {
CheckBoxTreeItem<T> cbti = (CheckBoxTreeItem<T>) treeItem;
booleanProperty = cbti.selectedProperty();
checkBox.selectedProperty().bindBidirectional((BooleanProperty)booleanProperty);
indeterminateProperty = cbti.indeterminateProperty();
checkBox.indeterminateProperty().bindBidirectional(indeterminateProperty);
} else {
throw new IllegalStateException("item must be CheckBoxTreeItem");
}
}
}
}
// usage: in the example add
treeTableView.setRowFactory(f -> new CheckBoxTreeTableRowHack<>());