yakidaiko
@yakidaiko (Yakiny)

Are you sure you want to delete the question?

Leaving a resolved question undeleted may help others!

Android Studio のチェックリストでスクロールするとチェックの状態が変わる

解決したいこと

Android Studio を使ってチェックリストを作成しようと、
サイトを参考にしてコードを書きました。
が、スクロールが必要になるほど、チェックの量を増やしたときに
問題が発生しました。

このようにアイテム1にチェックを入れて

下にスクロールし、

戻したら、アイテム1のチェックが外れた状態に戻ってしまいました。

どのようにコードを修正すればいいのか、
回答お願いします。

ソースコード

Listレイアウトファイル list.xml

<?xml version="1.0" encoding="utf-8"?>
<LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"
    xmlns:app="http://schemas.android.com/apk/res-auto"
    xmlns:tools="http://schemas.android.com/tools"
    android:orientation="vertical" android:layout_width="match_parent"
    android:layout_height="match_parent"
    android:descendantFocusability="blocksDescendants">

    <LinearLayout
        android:layout_width="match_parent"
        android:layout_height="wrap_content"
        android:orientation="horizontal">

        <CheckBox
            android:id="@+id/checkBox"
            android:layout_width="wrap_content"
            android:layout_height="wrap_content"
            android:layout_marginBottom="6dp"
            android:layout_marginLeft="20dp"
            android:layout_marginTop="6dp" />

        <TextView
            android:id="@+id/textView2"
            android:layout_width="wrap_content"
            android:layout_height="wrap_content"
            android:layout_marginLeft="20dp"
            android:layout_marginTop="6dp"
            android:layout_weight="1"
            android:text="TextView" />
    </LinearLayout>

</LinearLayout>

MainActivity.java

package com.test.checklist;

import androidx.appcompat.app.AppCompatActivity;

import android.os.Bundle;

import android.content.Context;

import android.util.Log;
import android.view.View;
import android.view.ViewGroup;
import android.widget.AdapterView;
import android.widget.CheckBox;
import android.widget.CompoundButton;
import android.widget.ListView;
import android.widget.SimpleAdapter;
import android.widget.TextView;

import java.util.ArrayList;
import java.util.HashMap;
import java.util.List;
import java.util.Map;

public class MainActivity extends AppCompatActivity {

    // Mapのキー
    private final String[] FROM = {"name","check"};
    // リソースのコントロールID
    private final int[] TO = { R.id.textView2,R.id.checkBox};

    // カスタムアダプター
    private class MyAdapter extends  SimpleAdapter{

        // 外部から呼び出し可能なマップ
        public Map<Integer,Boolean> checkList = new HashMap<>();

        public MyAdapter(Context context, List<? extends Map<String, ?>> data,
                         int resource, String[] from, int[] to) {
            super(context, data, resource, from, to);

            // 初期値を設定する
            for(int i=0; i<data.size();i++){
                Map map = (Map)data.get(i);
                checkList.put(i,(Boolean)map.get("check"));
            }
        }

        @Override
        public View getView(final int position, View convertView, ViewGroup parent) {
            View view = super.getView(position, convertView, parent);
            CheckBox ch = view.findViewById(R.id.checkBox);

            // チェックの状態が変化した場合はマップに記憶する
            ch.setOnCheckedChangeListener(new CompoundButton.OnCheckedChangeListener() {
                @Override
                public void onCheckedChanged(CompoundButton buttonView, boolean isChecked) {
                    checkList.put(position,isChecked);
                }
            });
            return view;
        }
    }

    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_main);

        ListView lv = findViewById(R.id.listView);

        // リストデータの生成
        List<Map<String,Object>> list = new ArrayList<>();
        Map<String,Object> map = new HashMap<>();

        map.put("name","アイテム1");
        map.put("check",false);
        list.add(map);

        map =  new HashMap<>();
        map.put("name","アイテム2");
        map.put("check",true);
        list.add(map);

        map =  new HashMap<>();
        map.put("name","アイテム3");
        map.put("check",true);
        list.add(map);

        map =  new HashMap<>();
        map.put("name","アイテム4");
        map.put("check",true);
        list.add(map);

        map =  new HashMap<>();
        map.put("name","アイテム5");
        map.put("check",true);
        list.add(map);

        map =  new HashMap<>();
        map.put("name","アイテム6");
        map.put("check",true);
        list.add(map);

        map =  new HashMap<>();
        map.put("name","アイテム7");
        map.put("check",true);
        list.add(map);

        map =  new HashMap<>();
        map.put("name","アイテム8");
        map.put("check",true);
        list.add(map);

        map =  new HashMap<>();
        map.put("name","アイテム9");
        map.put("check",true);
        list.add(map);

        map =  new HashMap<>();
        map.put("name","アイテム10");
        map.put("check",true);
        list.add(map);

        map =  new HashMap<>();
        map.put("name","アイテム11");
        map.put("check",true);
        list.add(map);

        map =  new HashMap<>();
        map.put("name","アイテム12");
        map.put("check",true);
        list.add(map);

        map =  new HashMap<>();
        map.put("name","アイテム13");
        map.put("check",true);
        list.add(map);

        map =  new HashMap<>();
        map.put("name","アイテム14");
        map.put("check",true);
        list.add(map);

        // アダプターの設定
        MyAdapter adapter = new MyAdapter(MainActivity.this,
                list,R.layout.list,FROM,TO);
        lv.setAdapter(adapter);

        // イベント
        lv.setOnItemClickListener(new AdapterView.OnItemClickListener() {
            @Override
            public void onItemClick(AdapterView<?> parent, View view, int position, long id) {
                MainActivity.this.setTitle(String.valueOf(position)+"番目がクリックされました。");
            }
        });

    }
}
0

1Answer

リストをスクロールすると画面外のビューは破棄(正確には再利用)され戻ってきたときに再度生成(正確には略)されるといった挙動になります。ですからビューを生成するときは都度現在の状態を反映してやる必要があります。

0Like

Comments

  1. @yakidaiko

    Questioner

    回答ありがとうございます。
    では、どのようにコードを変更すればよいでしょうか。
  2. ビューを生成するときにチェックボックスのチェック状態も変更してください。

Your answer might help someone💌