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

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

ただいまの
回答率

88.04%

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

解決済

回答 1

投稿

  • 評価
  • クリップ 0
  • VIEW 2,020

score 100

表題の通りなのですが、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式を用いたコード例にしました)

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/01/05 13:51

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

    キャンセル

  • 2018/01/05 13:55 編集

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

    キャンセル

  • 2018/01/05 16:50

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

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

    キャンセル

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

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

関連した質問

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