1

TL;DR: リスナーは他のセルでもアクティブ化されます。

カスタム クラスのデータを表すTreeViewさまざまなが含まれています。TreeItem基にBooleanPropertyなるデータが変更されると、セルの色が変わります。再び変化する場合は、色を削除する必要があります。

リスナーを使用していますが、 をスクロールするTreeViewと、特定のセルのプロパティを変更すると、他のセルの色も変更されます。この動作は、MWE を実行し、いくつかのセルを右クリックし、スクロールし、もう一度クリックするなどして再現できます。TreeView は、スクロールするだけでクリーンアップされる可能性があり、関係するセルが一瞬ビューから外れます。

リスナーを削除することもできますが、スクロールしてセルに戻った後にセルが再表示された場合にのみ、色が変更されます。

問題は、cellFactory でリスナーを適切に使用するにはどうすればよいかということです。

MWE

CellFactoryQuestion.java

package cellfactoryquestion;

import javafx.application.Application;
import javafx.beans.property.BooleanProperty;
import javafx.beans.property.SimpleBooleanProperty;
import javafx.css.PseudoClass;
import javafx.scene.Scene;
import javafx.scene.control.TreeCell;
import javafx.scene.control.TreeItem;
import javafx.scene.control.TreeView;
import javafx.scene.layout.StackPane;
import javafx.stage.Stage;

public class CellFactoryQuestion extends Application {

  /** Custom class used as underlying data of TreeItems */
  class CustomObject {
    String label;
    BooleanProperty state = new SimpleBooleanProperty(false);

    CustomObject(String s) { label = s; }
  }

  /** Cell Factory for CustomObject */
  class CustomTreeCell extends TreeCell<CustomObject>{

    PseudoClass customClass = PseudoClass.getPseudoClass("custom");

    @Override
    protected void updateItem(CustomObject co, boolean empty) {
      super.updateItem(co, empty);

      if (empty || co == null) {
        setText(null);
        setGraphic(null);
        pseudoClassStateChanged(customClass, false);

      } else {
        setText(co.label);
        setGraphic(null);

// BEGIN PROBLEMATIC

        /* define background color of cell according to state */
        pseudoClassStateChanged(customClass, co.state.getValue());
        co.state.addListener((o, ov, nv) -> {
          pseudoClassStateChanged(customClass, nv);
        });

// END PROBLEMATIC

        /* if right click, switch state */
        this.setOnContextMenuRequested(e -> {
          co.state.setValue(co.state.getValue() ^ true);
        });

      }
    }
  }

  @Override
  public void start(Stage primaryStage) {

    /* define TreeView 1/3 */
    TreeView tw = new TreeView();
    TreeItem rootTreeItem = new TreeItem(new CustomObject("Root"));
    rootTreeItem.setExpanded(true);

    /* define TreeView 2/3 */
    for (int c = 0; c != 5; c++) {
      TreeItem ci = new TreeItem(new CustomObject("Cat " + c));
      rootTreeItem.getChildren().add(ci);
      ci.setExpanded(true);
      for (int i = 0; i != 5; i++) {
        TreeItem ii = new TreeItem(new CustomObject("Item " + i));
        ci.getChildren().add(ii);
      }
    }

    /* define TreeView 3/3 */
    tw.setRoot(rootTreeItem);
    tw.setCellFactory(value -> new CustomTreeCell());

    /* define Scene */
    StackPane root = new StackPane();
    root.getChildren().add(tw);
    Scene scene = new Scene(root, 300, 250);
    scene.getStylesheets().add("/styles/Styles.css");
    primaryStage.setScene(scene);
    primaryStage.show();
  }

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

}

スタイル.css

.tree-cell:custom {
  -fx-background-color: salmon;
}
4

1 に答える 1