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

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

ただいまの
回答率

89.10%

ListViewでのCheckBoxにチェックが付かない

解決済

回答 2

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 3,695

hiro094

score 26

前提・実現したいこと

現在ListViewで一覧を表示しているのですが、項目のLongClickからチェックボックスが表示され、複数選択出来るようなものを作りたいと思っています。
実装は
ListVIew#setChoiceMode(AbsListView.CHOICE_MODE_MULTIPLE_MODAL);
で長押しから複数選択モードに切り替え、実際に何件を選択済みというふうに実装できました。

発生している問題・エラーメッセージ

しかし、ListView#setMultiChoiceModeListenerの
onItemCheckedStateChanged内にCheckBox#setCheckedメソッドを用いてチェックをしているのですが、実際のチェックボックスにチェックが付きません。
CheckBoxのインスタンス内のmChecked?は正しくFalseからTrueに変わっているのは確認出来たのですが、表示だけが変わらないようなのです。

CheckedTextViewも試しましたが同じ状態でした。

該当のソースコード

        <CheckBox
            android:id="@+id/chk_selected"
            android:layout_width="wrap_content"
            android:layout_height="wrap_content"
            android:clickable="false"
            android:focusable="false" />
    Fragment内

    @Override
    public void onActivityCreated(@Nullable Bundle savedInstanceState) {
        super.onActivityCreated(savedInstanceState);

        //...省略
        lvContacts.setChoiceMode(AbsListView.CHOICE_MODE_MULTIPLE_MODAL);
        lvContacts.setMultiChoiceModeListener(new AbsListView.MultiChoiceModeListener() {

            //...省略
            @Override
            public void onItemCheckedStateChanged(ActionMode mode, int position, long id, boolean checked) {
                    CheckBox test = (CheckBox)lvContacts.getAdapter().getView(position, null, lvContacts).findViewById(R.id.chk_selected);
                    test.setChecked(checked);
                    int count = lvContacts.getCheckedItemCount();
                    mode.setTitle(String.format("%d 件選択中", count));
           }
        }
    }
Adapter内
    @Override
    public View getView(int position, View convertView, ViewGroup parent) {
        convertView = layoutInflater.inflate(R.layout.list_item,parent,false);

        ((TextView)convertView.findViewById(R.id.tv_title)).setText(memoList.get(position).getTitle());
        CheckBox cb = ((CheckBox)convertView.findViewById(R.id.chk_selected));

        return convertView;
    }

追記(2017/06/15 20:53)

ViewHolderを実装しても駄目だったのですが、チェックボックスの値を別に保持する必要があるとの記事を他に見たので、試しにそのような実装をAdapter#getViewのほうに追記した所チェックが反映されるようになりました。ただ、この記述方法がはたして正しいのか分からないので、これで正しいか、他に正しいor綺麗な記述方法があれば引き続きご回答よろしくお願いします。

Adapter内

    @Override
    public View getView(final int position, View convertView, ViewGroup parent) {
        final ReviewMemo item = (ReviewMemo) this.getItem(position);
        ViewHolder holder;

        if(convertView == null) {
            convertView = layoutInflater.inflate(R.layout.list_item, parent, false);
            holder = new ViewHolder(convertView);
            convertView.setTag(holder);
            checked.add(false);
        }else{
            holder = (ViewHolder)convertView.getTag();
        }
        holder.tvTitle.setText(item.getTitle());

        //checkedというListメンバを作り、保持するようにしました。
        holder.chkSelected.setOnCheckedChangeListener(new CompoundButton.OnCheckedChangeListener(){
            @Override
            public void onCheckedChanged(CompoundButton arg0, boolean arg1) {
                checked.set(position, arg1); //チェックが変わるたびに
            }
        });
        holder.chkSelected.setChecked(checked.get(position));

        return convertView;
    }

    class ViewHolder{

        final TextView tvIndex;
        final ImageView ivAvatar;
        final TextView tvTitle;
        final CheckBox chkSelected;

        public ViewHolder(View itemView) {
            tvIndex = (TextView) itemView.findViewById(R.id.tv_title);
            ivAvatar = (ImageView) itemView.findViewById(R.id.iv_avatar);
            tvTitle = (TextView) itemView.findViewById(R.id.tv_title);
            chkSelected = (CheckBox)itemView.findViewById(R.id.chk_selected);
        }
    }

追記2(2017/06/16 14:08)

リストの数が2つ以上の場合、ActionModeが切り替わってからのチェックが2つめからのチェックが付かない事が発覚しました。
何故かsetOnCheckedChangeListenerが2回目から呼ばれていないようです。
おそらくViewの部分だと思うのですが、かなり回りくどいことをしているように思います。

リストに画像を入れない場合、ListFragment#onActivityCreated内に以下だけを書くだけでチェックボックスまわりは全て動作出来ます。

        setListAdapter(new ArrayAdapter<String>(
                getActivity(),
                android.R.layout.simple_list_item_checked,
                mItems));

ここにImageViewを表示させたいだけなのですが、そうなるとArrayAdapterをカスタマイズしないといけない+なぜかチェックボックスまわりが機能しなくなってしまいます。

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 2

check解決した方法

0

内容は追記1で解決しましたが、
onItemCheckedStateChangedの中にアダプターのnotifyDataSetChangedメソッドで更新したら複数行あるものでも問題なく行けるようになりました。

    Fragment内

    @Override
    public void onActivityCreated(@Nullable Bundle savedInstanceState) {
        super.onActivityCreated(savedInstanceState);

        //...省略
        lvContacts.setChoiceMode(AbsListView.CHOICE_MODE_MULTIPLE_MODAL);
        lvContacts.setMultiChoiceModeListener(new AbsListView.MultiChoiceModeListener() {

            //...省略
            @Override
            public void onItemCheckedStateChanged(ActionMode mode, int position, long id, boolean checked) {
                    CheckBox test = (CheckBox)lvContacts.getAdapter().getView(position, null, lvContacts).findViewById(R.id.chk_selected);
                    test.setChecked(checked);
                    int count = lvContacts.getCheckedItemCount();
                    mode.setTitle(String.format("%d 件選択中", count));

                    //追加分
                    mAdapter.notifyDataSetChanged();
           }
        }
    }

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

0

全てのソースコードが読めないため推測での回答になりますが、おそらくビューが使い回されているのではないでしょうか。

僕が昔書いたコードはこんな感じでした。

@Override
public View getView(final int position, View convertView, ViewGroup parent)
{
    final MyClass item = this.getItem(position);
    ViewHolder holder;

    if (convertView == null)
    {
        convertView = myInflater.inflate(R.layout.some_layout, false);
        holder = new ViewHolder();
        holder.one   = (TextView) convertView.findViewById(R.id.one);
        holder.two   = (TextView) convertView.findViewById(R.id.two);
        holder.three = (ImageView)convertView.findViewById(R.id.three);
        convertView.setTag(holder);
    }
    else
    {
        holder = (ViewHolder) convertView.getTag();
    }


    // 回答後に追記しました
    holder.one.setText(item.getTextOne);
    holder.two.setText(item.getTextTwo);
    // 追記ここまで

    return convertView;
}

唐突に出てきた ViewHolder は単純な内部クラスです。

private class ViewHolder
{
    TextView  one;
    TextView  two;
    ImageView therr;
}

具体的な回答ではありませんが、ご参考になれば。

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/06/15 21:01

    回答ありがとうございます。
    RecyclerViewではViewHolderを使っていたのですが、まだ知識が浅いためListViewにはいらないものだと思っていました。
    しかし、このような実装をしてみなたのですが、やはりチェックは付きませんでした。
    正しい解決かわからないのですが、チェックが付くようになりました。
    問題はチェックボックスの値をそれぞれ保持する必要がある?事と、getViewの呼ばれるタイミングがよく分かっていないのですが、getViewでチェックボックスの値を再セットする必要がある?ようです。
    よろしければ、引き続きなぜこれで上手く行ったのかと、他に正しい方法があればご回答のほどよろしくお願いします。

    キャンセル

  • 2017/06/16 23:53

    問題が解決したようで何よりです。本件は満了していると思いますが getView の周りだけ補足しておきます。

    ---

    getView() は各行が画面に表示されるタイミングで一行につき一回ずつ呼び出されます。ログの出力を確認した感じでは画面の大きさ + 数行程度のエリアを読み込んでいるようです。

    各行の値は設定した Tag ID を基に正しく保持されています。

    但し、画面の描画に構造上の難点があります。システムは有限なメモリ量を節約するために ListView で画面外にスクロールアウトした行を新しく表示する行に使いまわそうとします。この時、スクロールアウトした行の要素に値が入っていて、かつ新しく表示する行の要素に値が入っていない場合、(システムは初期値と判断して何も処理しないため)スクロールアウトした時点の外観のまま画面に表示されてしまいます。
    これは行の見た目と実際に持っている値に差異が生じてしまうため、アプリケーション作成者の意図するところではありません。

    この問題を解消するために holder.TextView.setText などを行います。試しにこの処理をコメントアウトしてみると、スクロールアウトした行の値が 2、3 行後くらいにスクロールインしてくる行の一部として表示される動作が確認できるはずです。

    内容は以上です。

    上記の内容は Android 3.x 時の古い認識であるため、もしかすると最新のバージョンでは異なっているかもしれません。その場合はごめんなさい。リスト処理は件数が大きくなってくると思わぬ不具合が出てくると思います。頑張ってください。

    キャンセル

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

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