回答編集履歴

追加

2022/10/28 17:20

投稿

jimbe

スコア13209

test CHANGED Viewed

@@ -3,6 +3,7 @@
 質問の画像を見る限り、まだリストビューがスクロールするほどの件数が無い為に再利用が発生していないようですが、再利用が発生した場合、convertView が null で無い＝チェックボックスのリスナが古いまま新しいデータの表示に使われ、チェックボックスの変更は古いデータに対して処理が行われることになってしまいます。
 追加毎にログが出るのは、追加毎にリストに setAdapter し全ての(再)表示が行われて getView が呼ばれるため、デフォルト false のチェックボックスに setChecked で true を設定しているデータで OnCheckedChangeListener が呼ばれている為でしょう。
+それだけなら常に true と出るはずですが・・・なぜ false になるのかは再現出来ていません。
 ---
 ```java

追加

2022/10/28 17:18

投稿

jimbe

スコア13209

test CHANGED Viewed

@@ -1,6 +1,8 @@
 レイアウトをインフレートした時だけリスナを定義するのでは、問題があります。
 getView で convertView が null かどうかによって infrate を制御するのは、ListView が convertView を再利用するからです。
 質問の画像を見る限り、まだリストビューがスクロールするほどの件数が無い為に再利用が発生していないようですが、再利用が発生した場合、convertView が null で無い＝チェックボックスのリスナが古いまま新しいデータの表示に使われ、チェックボックスの変更は古いデータに対して処理が行われることになってしまいます。
+追加毎にログが出るのは、追加毎にリストに setAdapter し全ての(再)表示が行われて getView が呼ばれるため、デフォルト false のチェックボックスに setChecked で true を設定しているデータで OnCheckedChangeListener が呼ばれている為でしょう。
 ---
 ```java

コード修正

2022/10/28 16:51

投稿

jimbe

スコア13209

test CHANGED Viewed

@@ -82,8 +82,7 @@
       return position;
     }
     @Override
-    public @NonNull
-    View getView(int position, View convertView, @NonNull ViewGroup parent) {
+    public @NonNull View getView(int position, View convertView, @NonNull ViewGroup parent) {
       if(convertView == null) {
         convertView = LayoutInflater.from(parent.getContext()).inflate(R.layout.list, parent, false);
         convertView.setTag(new ViewHolder(convertView));

誤字

2022/10/28 13:30

投稿

jimbe

スコア13209

test CHANGED Viewed

@@ -1,5 +1,5 @@
 レイアウトをインフレートした時だけリスナを定義するのでは、問題があります。
-getView で convertView が null かどうかによって infrate を制御するのは、ListView が convartView を再利用するからです。
+getView で convertView が null かどうかによって infrate を制御するのは、ListView が convertView を再利用するからです。
 質問の画像を見る限り、まだリストビューがスクロールするほどの件数が無い為に再利用が発生していないようですが、再利用が発生した場合、convertView が null で無い＝チェックボックスのリスナが古いまま新しいデータの表示に使われ、チェックボックスの変更は古いデータに対して処理が行われることになってしまいます。
 ---

コード修正

2022/10/28 13:28

投稿

jimbe

スコア13209

test CHANGED Viewed

@@ -95,7 +95,7 @@
       vh.viewCheck.setOnCheckedChangeListener(null); //次の setChecked で古いリスナが実行されないように、先に消す.
       vh.viewCheck.setChecked(cellDataItem.check);
       vh.viewCheck.setOnCheckedChangeListener((buttonView, isChecked) -> {
-        cellDataList.get(position).check = isChecked;
+        cellDataItem.check = isChecked;
         Log.i("Test", "cellDataList["+position+"]のcheckに["+isChecked+"]を入れました");
         notifyDataSetChanged(); //データが変わったのでリストに再表示を要求
       });

コード追加

2022/10/28 13:18

投稿

jimbe

スコア13209

test CHANGED Viewed

@@ -1,3 +1,187 @@
 レイアウトをインフレートした時だけリスナを定義するのでは、問題があります。
 getView で convertView が null かどうかによって infrate を制御するのは、ListView が convartView を再利用するからです。
 質問の画像を見る限り、まだリストビューがスクロールするほどの件数が無い為に再利用が発生していないようですが、再利用が発生した場合、convertView が null で無い＝チェックボックスのリスナが古いまま新しいデータの表示に使われ、チェックボックスの変更は古いデータに対して処理が行われることになってしまいます。
+---
+```java
+import android.graphics.Color;
+import android.os.*;
+import android.util.Log;
+import android.view.*;
+import android.widget.*;
+import androidx.annotation.NonNull;
+import androidx.appcompat.app.AppCompatActivity;
+import androidx.lifecycle.*;
+import java.text.*;
+import java.util.*;
+public class MainActivity extends AppCompatActivity {
+  @Override
+  protected void onCreate(Bundle savedInstanceState) {
+    super.onCreate(savedInstanceState);
+    setContentView(R.layout.activity_main);
+    getSupportActionBar().setTitle(R.string.app_name);
+    ListView listView = findViewById(R.id.list_view);
+    ListViewAdapter adapter = new ListViewAdapter();
+    listView.setAdapter(adapter);
+    NowTimer nowTimer = new NowTimer(100);
+    getLifecycle().addObserver(nowTimer);
+    Counter counter = new Counter(0, 9999);
+    EditText commentText = findViewById(R.id.comment_text);
+    Button addButton = findViewById(R.id.add_button);
+    addButton.setOnClickListener(v -> {
+      CellData cellData = new CellData(nowTimer.getDate(), counter.getCount(), commentText.getText().toString());
+      adapter.add(cellData);
+    });
+  }
+  // データを保持するクラス
+  private static class CellData {
+    boolean check = false;
+    Date time;
+    int count;
+    String comment;
+    CellData(Date time, int count, String comment) {
+      this.time = time;
+      this.count = count;
+      this.comment = comment;
+    }
+  }
+  // カスタムアダプタークラス
+  private class ListViewAdapter extends BaseAdapter {
+    private final SimpleDateFormat dateFormat = new SimpleDateFormat("HH:mm:ss");
+    private final int[] ROW_BG_COLORS = { Color.rgb(240,240,240), Color.WHITE };
+    private List<CellData> cellDataList = new ArrayList<>();
+    void add(CellData cellData) {
+      cellDataList.add(cellData);
+      notifyDataSetChanged();
+    }
+    @Override
+    public int getCount() {
+      return cellDataList.size();
+    }
+    @Override
+    public Object getItem(int position) {
+      return cellDataList.get(position);
+    }
+    @Override
+    public long getItemId(int position) {
+      return position;
+    }
+    @Override
+    public @NonNull
+    View getView(int position, View convertView, @NonNull ViewGroup parent) {
+      if(convertView == null) {
+        convertView = LayoutInflater.from(parent.getContext()).inflate(R.layout.list, parent, false);
+        convertView.setTag(new ViewHolder(convertView));
+      }
+      CellData cellDataItem = cellDataList.get(position);
+      ViewHolder vh = (ViewHolder)convertView.getTag();
+      vh.viewCheck.setOnCheckedChangeListener(null); //次の setChecked で古いリスナが実行されないように、先に消す.
+      vh.viewCheck.setChecked(cellDataItem.check);
+      vh.viewCheck.setOnCheckedChangeListener((buttonView, isChecked) -> {
+        cellDataList.get(position).check = isChecked;
+        Log.i("Test", "cellDataList["+position+"]のcheckに["+isChecked+"]を入れました");
+        notifyDataSetChanged(); //データが変わったのでリストに再表示を要求
+      });
+      vh.viewTime.setText(dateFormat.format(cellDataItem.time));
+      vh.viewCount.setText("" + cellDataItem.count);
+      vh.viewComment.setText(cellDataItem.comment);
+      convertView.setBackgroundColor(cellDataItem.check ? Color.GREEN : ROW_BG_COLORS[position%2]);
+      return convertView;
+    }
+    // Viewを保持するクラス
+    private class ViewHolder {
+      final CheckBox viewCheck;
+      final TextView viewTime;
+      final TextView viewCount;
+      final TextView viewComment;
+      ViewHolder(View itemView) {
+        viewCheck = itemView.findViewById(R.id.listCheckBox);
+        viewTime = itemView.findViewById(R.id.listTime);
+        viewCount = itemView.findViewById(R.id.listCount);
+        viewComment = itemView.findViewById(R.id.listComment);
+      }
+    }
+  }
+  private class NowTimer extends TimerTask implements DefaultLifecycleObserver {
+    private SimpleDateFormat dateFormat = new SimpleDateFormat("HH:mm:ss");
+    private TextView clockText;
+    private Handler handler;
+    private Timer timer;
+    private long period;
+    private Date now;
+    NowTimer(long period) {
+      this.period = period;
+      clockText = findViewById(R.id.clock_text);
+      handler = new Handler(getMainLooper());
+    }
+    Date getDate() {
+      return now;
+    }
+    @Override
+    public void run() {
+      now = new Date();
+      handler.post(() -> clockText.setText(dateFormat.format(now)));
+    }
+    @Override
+    public void onStart(@NonNull LifecycleOwner owner) {
+      DefaultLifecycleObserver.super.onStart(owner);
+      timer = new Timer();
+      timer.schedule(this, 0, period);
+    }
+    @Override
+    public void onStop(@NonNull LifecycleOwner owner) {
+      DefaultLifecycleObserver.super.onStop(owner);
+      timer.cancel();
+      timer = null;
+    }
+  }
+  private class Counter {
+    private DecimalFormat df = new DecimalFormat("#,###");
+    private TextView countText;
+    private int count;
+    Counter(int min, int max) {
+      countText = findViewById(R.id.count_text);
+      setCount(min);
+      Button plusButton = findViewById(R.id.plus_button);
+      plusButton.setOnClickListener(v -> setCount(Math.min(count+1, max)));
+      Button minusButton = findViewById(R.id.minus_button);
+      minusButton.setOnClickListener(v -> setCount(Math.max(count-1, min)));
+    }
+    void setCount(int count) {
+      this.count = count;
+      countText.setText(df.format(count));
+    }
+    int getCount() {
+      return count;
+    }
+  }
+}
+```