RecyclerView CheckBox setOnCheckedChangeListener多选异常排查

问题现象 #

依次勾选列表选项a、b、c后，退出页面再返回，仅最后勾选的选项c保持选中，其余项选中状态全部丢失，勾选/取消操作仅最后点击的项状态能正常留存。

根因定位 #

你的代码存在3个核心问题，共同导致状态覆盖：

• 监听内引用了外层可变的checklistChildViewHolder和position变量，RecyclerView复用时这些变量会被最后一次绑定的项覆盖，所有监听实际生效的对象都是最后一个绑定的列表项。
• 每次修改状态时，拆分使用的status是全局缓存的旧值，不是之前操作更新后的最新状态。比如勾选a时把a位置设为1存入数据库，但全局status没有更新，后续勾选b时还是基于初始的全0状态拆分，仅把b位置设为1存入，直接覆盖了a的选中状态，最后一次操作的c就成了唯一被标记为1的项。
• 绑定CheckBox时没有先移除旧监听，复用过程中旧监听会被意外触发，导致状态错乱；另外拼接状态串时每次都在末尾加换行，多次操作后状态数组长度和列表项数不匹配，会出现位置错位。

修复代码 #

按照RecyclerView复用规则调整绑定逻辑，核心是「先清旧监听→按最新数据设控件状态→绑定捕获固定位置的新监听」，修改后onBindViewHolder中的CheckBox逻辑如下：

@Override
public void onBindViewHolder(ChecklistChildViewHolder holder, int position) {
    // 1. 先移除旧监听，避免复用过程中旧监听误触发
    holder.checkBox.setOnCheckedChangeListener(null);

    // 2. 基于最新的状态数据，设置CheckBox当前的选中状态和文本样式
    String currentItemContent = dataList.get(position).getContent();
    boolean currentChecked = "1".equals(latestStatusList.get(position));
    holder.checkBox.setChecked(currentChecked);
    holder.checkBox.setText(currentChecked ? lineThrough(currentItemContent) : currentItemContent);

    // 3. 用final固定当前项的位置，避免复用时position变量变动导致错位
    final int fixedPosition = position;
    holder.checkBox.setOnCheckedChangeListener(new CompoundButton.OnCheckedChangeListener() {
        @Override
        public void onCheckedChanged(CompoundButton buttonView, boolean isChecked) {
            // 基于内存中最新的全量状态修改，不要用全局缓存的旧status
            List<String> updateStatusList = new ArrayList<>(latestStatusList);
            updateStatusList.set(fixedPosition, isChecked ? "1" : "0");

            // 拼接新状态串，去掉末尾多余换行避免数组长度错位
            StringBuilder statusSb = new StringBuilder();
            for (int i = 0; i < updateStatusList.size(); i++) {
                statusSb.append(updateStatusList.get(i));
                if (i != updateStatusList.size() - 1) {
                    statusSb.append("\n");
                }
            }
            String finalStatus = statusSb.toString();

            // 更新当前项UI
            String itemText = dataList.get(fixedPosition).getContent();
            buttonView.setText(isChecked ? lineThrough(itemText) : itemText);

            // 更新数据库，同步更新内存中的最新状态，避免后续操作拿旧值
            dbHelper.updateChecklist(
                    dataList.get(fixedPosition).getId(),
                    finalStatus,
                    itemText,
                    DateTime.date(),
                    DateTime.time(),
                    System.currentTimeMillis()
            );
            latestStatusList = updateStatusList;
        }
    });
}

注意事项 #

列表控件绑定可交互子View时，不要在监听内持有外层ViewHolder的引用，直接通过onCheckedChanged回调返回的buttonView操作当前控件即可；所有状态修改必须基于最新的全量数据源，每次更新后同步刷新内存中的数据源缓存，避免旧值覆盖新状态。

内容的提问来源于stack exchange，提问作者nano tech