質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

91.04%

  • JavaFX

    358questions

    JavaFXとは、Java仮想マシン上で動作するリッチインターネットアプリケーション (RIA) のGUIライブラリです。Swingとは異なり、FXMLと呼ばれる XMLとCSSを併用してデザインを記述します。

TableViewで特定の属性を持つ行のみCheckBoxを表示したい

解決済

回答 1

投稿

  • 評価
  • クリップ 0
  • VIEW 184

XCUBE

score 72

表題の通りなのですが、TableViewにCheckBox付の列を表示するようにしていますが、データを構築する際に特定の属性を持つものだけCheckBoxを表示したいとやっているのですが、思うようにうまくいきません。

一部抜粋していますが、やり方として次のようなコードを書いています。

Callback<TableColumn<Parson, Boolean>, TableCell<Parson, Boolean>> callBackCheck = new Callback<TableColumn<Parson, Boolean>, TableCell<Parson, Boolean>>(){
    @Override
    public TableCell<Parson,Boolean> call( TableColumn<Parson,Boolean> param ) {
        return new CheckBoxTableCell<Parson, Boolean>() {
            @Override
            public void updateItem(Boolean item, boolean empty) {
                super.updateItem(item, empty);
                if (!empty && item != null) {
                    TableRow  row = getTableRow();
                    Parson selitem = (Parson)row.getItem();
                    setVisible((selitem.getDataType() != 1)); // 属性により非表示にする
                }
            }
        };
    }
};

   ・
   ・
   ・
ObservableList<TableColumn<Parson, ?>> cols = tableView.getColumns();
TableColumn checkCol = cols.get(0);
checkCol.setCellValueFactory(new PropertyValueFactory<>("view"));
checkCol.setCellFactory(CheckBoxTableCell.forTableColumn((TableColumn<Parson, Boolean>)checkCol));
checkCol.setCellFactory(callBackCheck);

表示するTableViewのデータがスクロールしないほど少ない場合は問題ないのですが、データを増やしてスクロールさせてみるとCheckBoxが表示されているところとされてないところはあるのですが、属性と関係なく表示されたりされなかったりとなってうまくいきません。

そもそもやり方が間違っているのかもしれないと思い質問させていただきました。
よろしくお願いします。

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • KSwordOfHaste

    2017/12/28 12:12

    アイテムのdataTypeプロパティーはテーブル表示中に編集したり変化することがあるでしょうか?

    キャンセル

  • XCUBE

    2017/12/28 12:15

    ありがとうございます。dataTypeは最初に表示してから変化することはないです。

    キャンセル

  • KSwordOfHaste

    2017/12/28 13:12

    あ・・・すみません、トンチンカンな質問してしまいました。

    キャンセル

回答 1

checkベストアンサー

0

訂正:最初の回答は間違っていたようです。申し訳ありません。
CellのvisibilityはTableViewやCellの基底クラスの方の制御の影響を強く受けるようで、せっかくsetVisible(false)にしても直後にsetVisible(true)にされてしまうことがあるようです。多分Cell領域がTableViewの可視領域に入るとそのような動きになるのではないでしょうか(あくまで想像です)。

本件はどのような対処にするのがよいのか自分には少々難しいです。CheckBoxTableCellのようなカスタムセルの場合、setGraphic()やsetVisible()によって外観を制御しているのですが実際にどのような制御を行っているかは必ずしも明文化されてないと思います。下手に派生クラス側で制御しようとすると基底クラスに予期せぬ制御をされてしまう可能性がある以上、基底クラスの実装まで踏み込んでみないと確実な方法がわかりません。しかし本来隠蔽されているはずの実装に基づき実装するのは危険ですよね・・・。公開されている仕様のみでうまく制御できる方法が思いつけばそれはそれでよいのですが・・・

一応それらしく動くあまり乱暴ではなさそうな方法があったので一応参考までにコードを載せてみます。しかし充分にCheckBoxTableCellの振る舞いを調べた上での実装ではないのでこの実装でよいかどうかはCheckBoxTableCellがどのタイミングでsetGraphicを呼び出すかをよく調べた方がよいと思います。また、こうした対処の常でJDKのバージョンが変わる際に再度検証が必要かも知れません。JavaFXはJDK8時点では充分安定しておらずJDK9で大規模なリファクタリングが行われているようですので要注意と思います。

Callback<TableColumn<Parson, Boolean>, TableCell<Parson, Boolean>> cb2 = tc -> new CheckBoxTableCell<Parson, Boolean>() {
  @Override
  public void updateItem(Boolean item, boolean empty) {
    super.updateItem(item, empty);
    updateVisiblity();
  }

  @Override
  public void updateIndex(int i) {
    super.updateIndex(i);
    updateVisiblity();
  }

  private void updateVisiblity() {
    int index = getIndex();
    boolean visibility = index != -1 &&
        getItem() != null &&
        getTableView().getItems().get(index).getDataType() != 1;
    if (!visibility)
      setGraphic(null);
  }
};
...
checkCol.setCellFactory(cb2);

以下最初の回答

そもそもやり方が間違っているのかもしれない

個人的にはアプローチはよいと思います。ただCellの振る舞いについて注意が必要な点がありそこにはまってしまったのが原因だと思います。

原因

Cellは特定のアイテムに結び付いているのではなく特定のプロパティの「値」に結び付いているだけなので、表示対象のBoolean値が変化した場合ならupdateItemが呼び出されますが、そうでない限りは例え別のインスタンスのプロパティーに再割り当てされたとしてもupdateItemは起動されないのです。

これは自分も全く同じことで過去にはまったのですが、ご質問を拝見したときすぐに気づけませんでした。自分にとってそれほどに勘違いしやすい仕様に思えますが、よーく考えてみると妥当な仕様だと納得した覚えがあります。表示内容は基本的に「表示しようとしている値に依存する」という考え方がベースのため「別のインスタンスに切り替わったからといってupdateItemを呼び出す必要はない」という設計なのでしょう。

対処

updateItemによって状態を変化させるのではなく、表示対象のアイテムが切り替わったタイミングで変化させるという方法が考えられると思います。例えばソート順を変化させないならこのCellが置かれているTableRowの変化タイミングを検知すればそれができると思います。

return new CheckBoxTableCell<Parson, Boolean>() {
  {
    tableRowProperty().addListener((observable, oldRow, newRow) -> {
      Parson item = (Parson)newRow.getItem();
      setVisible(item.getDataType() != 1);
    });
  }
};

しかしソート順が変化した場合はどうなるでしょうか・・・やってみてないので実験してみるとよいと思います。ちゃんと調べてないのですが最新のJDK8ではソート順が変化してもtableRowとアイテムの対応関係は変化しないのではないかと思います。

ただJDK8の途中でTableViewのソートの実装が変わったという情報を見たことがあるので念のためtableRowプロパティーとtableRowプロパティーのitemプロパティーの両方を監視した方がよいのかも知れません。(その点は自信ないので確認した方がよいと思います)
もしそこまで配慮するなら以下のようにできます。

return new CheckBoxTableCell<Parson, Boolean>() {
  ChangeListener<Parson> itemListener = (observable, oldItem, newItem) -> {
    // newItemに従ってセルの状態を変更する処理
      Parson item = (Parson)newRow.getItem();
      setVisible(item.getDataType() != 1);
  };
  { // Cellインスタンスの初期化ブロック(無名クラスにはコンストラクターが書けないので...)
    tableRowProperty().addListener((observable, oldRow, newRow) -> {
      if (oldRow != null) oldRow.itemProperty().removeListener(itemListener);
      if (newRow != null) newRow.itemProperty().addListener(itemListener);
    });
  }
};

(ご質問のコードではlambda式ではなく無名クラスが使われていたりしますね。でもJavaFXなのでJava8前提でよい気がします。そこでlambda式を用いたコード例にしました)

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/12/28 14:51 編集

    ご回答ありがとうございます。

    投降後に色々試行錯誤していました。
    質問にも書いていましたか、スクロールしないような恰好で表示すれば問題なかったので、一旦TableViewを非表示にしてHeightを極端に高くしてデータを投入後にまたHeightを元に戻すという姑息な手段で逃げようとしましたが、ご指摘の通りソート時に表示がおかしくなりました。

    初歩的な質問で申し訳ないのですが、ご回答いただいたラムダ式を使用した場合、CheckBoxをどのように参照すればよいかわかりません。
    CheckBox.setVisible(newItem.getDataType() != 1);
    で表示/非表示をしたいのです。

    キャンセル

  • 2017/12/28 15:00 編集

    無名クラス内での書き方と同じです。最初のコード例でtableRowPropertyのChangeListenerをラムダ式で表現していますが{}の中身は同じですよね?後のコード例で

    // newItemに従ってセルの状態を変更する処理

    とコメントになっている部分に同じコードを書けばよいです。
    ---
    はっきりさせるために解答のコードに上記を反映しておきました。

    キャンセル

  • 2017/12/28 15:33

    ほんとうにありかどうございます。

    教えていただいたもので以下のようなコードに変更してみましたが
    thisが親クラス(実行クラス)を指してしまいnewItemもParsonを指すようです。
    何かお気づきの点などご指摘いただいたら幸いです。

    Callback<TableColumn<Parson, Boolean>, TableCell<Parson, Boolean>> callBackCheck
    = new Callback<TableColumn<Parson, Boolean>, TableCell<Parson, Boolean>>(){
    @Override
    public TableCell<Parson,Boolean> call( TableColumn<Parson,Boolean> param ) {
    return new CheckBoxTableCell<Parson, Boolean>() {
    ChangeListener<Parson> itemListener = (observable, oldItem, newItem) -> {
    // newItemに従ってセルの状態を変更する処理
    Parson item = (Parson)newItem.getItem();
    setVisible(item.getDataType() != 1);
    };
    { // Cellインスタンスの初期化ブロック(無名クラスにはコンストラクターが書けないので...)
    tableRowProperty().addListener((observable, oldRow, newRow) -> {
    if (oldRow != null) oldRow.itemProperty().removeListener(itemListener);
    if (newRow != null) newRow.itemProperty().addListener(itemListener);
    });
    }
    };
    }
    };

    キャンセル

  • 2017/12/28 16:06 編集

    thisが親クラスになるのは私の勘違いでした。すいません。

    ご教示いただいたコードを元に再度確認し、ソートをしてみたのですが、非表示にはできませんでした。

    試しにソート時に
    setVisible(false);
    System.out.println(isVisible());
    で属性に関係なくすべて非表示にしたところ非表示にはならず、ログもすべてtrueになっていました。

    一応updateItemでは非表示にできましたが、ソート後の動作がうまくいかず、ソートのイベントでのsetVisible(false)も効かずといった感じです。

    もう少し試行錯誤してみます。

    キャンセル

  • 2017/12/28 16:57 編集

    すみません、実際にやってみたのですが上のコードは間違ってる点がありました。また他に配慮不足の点もあるようで少し試したのですがすんなり動いてくれません><
    自分でも改めて調べてみます。ちなみに自分はJDK9でやっているのですがXCUBEさんはJava1.8.0_152とかですか?できるなら合わせた方がよいかも知れませんので。

    キャンセル

  • 2017/12/28 17:12

    ありがとうございます。
    Javaのバージョンは1.8.0_151の32bit版です(お客様の環境の32bitなもので…)。

    KSwordOfHasteさんの貴重なお時間を費やすのは気が引けます・・・
    気持ちだけでもほんとうにうれしいです。

    キャンセル

  • 2017/12/28 17:34

    JDK9とJDK8ではTableViewまわりでもいくつも変更があるようなのでJDK8でやってみることにします。なんとなく原因がわかりました。visibleはTableViewがセルが見えるようになったときに制御するためのもののようなのでこれを自前で制御するのはよくなさそうな感じです。上の回答が間違っている気がしますので、自分でももう少し調べさせてください。

    ちなみに、お気遣いは無用です。そもそも自分の回答が間違っていたのですし、こういう問題は自分にとっても勉強になります。

    キャンセル

  • 2018/01/05 10:36

    検証いただきましてありがとうございます。
    本件では特定の行のCheckBoxを変更できないようにするのが本来の目的の一つでもありますが、
    setVisible(false);が効かないのであればsetDisable(true);で代用すれば、その目的は一応達成できます。
    これによりㇾのON/OFFはできなくなるのですが、見た目がグレーアウトにならず、利用者にON/OFFができるように見えてしまうという問題があります。
    □の枠線の色を透明もしくは背景色と同じ色に変えることができれば、疑似的にsetVisible(false);にできると思うので、ちょっとその方法を模索してみます。

    大変ありがとうございました。心より感謝いたします。

    キャンセル

  • 2018/01/05 12:32 編集

    上記、コメントでsetDisable(true);を代用するとしましたが、それでも並び替えるとうまくいきませんでした。

    ですが、結果的にsetVisible(false);でもなくsetDisable(true);でもなくsetEditable(false);で解決しました。
    見た目を非表示にするため、非表示にしたいcheckboxにIdを設定し、そのIdに対して外枠、背景などを透明にしてstyleをcssに記述しました。
    その結果、疑似的ではありますが、非表示にすることができましたが、ひとつ疑問ができました。
    以下のように疑似要素として disabled を付加しないとstyleが適用できませんでした。
    setDisable(true);にはしていないのですが、setEditable(false);も内部的にはDisable扱いなんだなということで理解することにしました。

    #Id:disabled {
      -fx-background-color: transparent;
      -fx-border-color: transparent;
      -fx-fill: transparent;
    }

    キャンセル

  • 2018/01/05 12:40

    なるほど!CheckBoxTableCellの実装を気にするよりはXCUBEさんの採られた方法の方がずっと適切だと思いました。なお、そのようにするのでしたら疑似クラスを設定してもよい気がします。Cellに対して

    cell.getStyleClass().add("hidden")
    cell.getStyleClass().remove("hidden")

    とかやるわけです。独自のスタイルクラスなのでJavaFXランタイムがそれを勝手に変更する心配はないですし・・・css上も

    .hidden {
    ...
    }

    といった感じに書けますね。IDを振る必要は特にないのでは?

    キャンセル

  • 2018/01/05 13:51

    アドバイスありがとうございます。
    TableViewクラスの不具合なのか私のやり方が悪いのか.hiddenはうまくいきませんでした。
    私としてもあまりIdは付けたくなかったのですが・・・該当するcellに直接3つのstyleを追加するか迷った結果Idを付けることにしました。
    hiddenという疑似クラスがあるということが判ったことだけでも収穫です。

    キャンセル

  • 2018/01/05 13:56 編集

    いえ、hiddenという疑似クラスは標準にはありません。アプリケーションで独自に定義するものです。それっぽい名前として"hidden"にしたまでです。
    ---
    もとい・・・標準にありました!申し訳ないですがhiddenではなく独自の疑似クラス名を使った方がよいと思います。

    キャンセル

  • 2018/01/05 16:50

    そうやって独自のセレクタを追加できるんですか、知りませんでした。
    今までIDもしくは標準のセレクタで対応していました。

    そういえば暫くやっていないですがHTMLを記述するときもIDとCLASSを使う方法がありましたのを思い出しました。

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 91.04%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る

  • JavaFX

    358questions

    JavaFXとは、Java仮想マシン上で動作するリッチインターネットアプリケーション (RIA) のGUIライブラリです。Swingとは異なり、FXMLと呼ばれる XMLとCSSを併用してデザインを記述します。