RecyclerView滚动时CheckBox选中状态自动变更问题求助（附代码）

嘿，这个问题我太熟了！RecyclerView的ViewHolder复用机制就是导致CheckBox状态乱跳的元凶——当你滚动列表时，之前滑出屏幕的ViewHolder会被重新拿来显示新的条目，而如果CheckBox的状态没有和对应的数据项绑定，就会出现状态错乱的情况。给你一步步解决：

问题根源 #

RecyclerView为了优化性能会复用ViewHolder，当列表滚动时，离开可视区域的ViewHolder会被重新绑定到新的列表项上。如果你的CheckBox状态没有和对应的数据关联，复用的ViewHolder会保留之前的选中状态，就出现了“滚动时状态自动改变”的问题。

解决方案步骤 #

1. 新增状态存储结构

首先，我们需要一个地方记录每个列表项的选中状态。可以在QuestionAdapter里添加一个HashMap，用来存储每个问题的选中状态（用位置或问题ID当key都可以，这里先用位置示例）：

private class QuestionAdapter extends RecyclerView.Adapter<QuestionHolder> {
    private JSONArray questions;
    // 新增：存储每个问题的选中状态
    private HashMap<Integer, Boolean> checkStateMap = new HashMap<>();

    public QuestionAdapter(JSONArray questions) {
        this.questions = questions;
        // 初始化状态：默认所有问题都未选中
        for (int i = 0; i < questions.length(); i++) {
            checkStateMap.put(i, false);
        }
    }
    // ... 其他原有代码
}

2. 修改ViewHolder的bind方法，绑定状态

在QuestionHolder的bind方法里，不仅要设置问题文本，还要根据存储的状态设置CheckBox的选中状态，同时处理选中状态变化的逻辑：

public void bind(JSONObject question, final int position, final HashMap<Integer, Boolean> checkStateMap) {
    this.question = question;
    try {
        question_txt.setText(question.getString("_question"));
        // 根据存储的状态，设置CheckBox的选中状态
        checkBox.setChecked(checkStateMap.get(position));
    } catch (JSONException je) {
        je.printStackTrace();
    }

    // 监听CheckBox的选中状态变化，同步更新存储的状态
    checkBox.setOnCheckedChangeListener(new CompoundButton.OnCheckedChangeListener() {
        @Override
        public void onCheckedChanged(CompoundButton buttonView, boolean isChecked) {
            // 更新状态存储
            checkStateMap.put(position, isChecked);
            // 这里处理你的业务逻辑：比如添加/移除问题ID到answered列表
            try {
                String questionId = question.getString("_id");
                if (isChecked) {
                    // 将questionId加入answered列表
                } else {
                    // 从answered列表移除questionId
                }
            } catch (JSONException e) {
                e.printStackTrace();
            }
        }
    });
}

3. 更新Adapter的onBindViewHolder方法

在onBindViewHolder里，把当前位置和状态存储传递给bind方法，确保每个ViewHolder绑定对应的数据和状态：

@Override
public void onBindViewHolder(QuestionHolder holder, int position) {
    try {
        final JSONObject question = questions.getJSONObject(position);
        // 传递位置和状态map给bind方法
        holder.bind(question, position, checkStateMap);
    } catch (JSONException je) {
        je.printStackTrace();
    }
}

4. 清理ViewHolder构造方法里的冗余监听

原来的构造方法里给CheckBox设置了setOnClickListener，现在我们改用setOnCheckedChangeListener，所以可以移除这个冗余监听，避免冲突：

public QuestionHolder(final LayoutInflater inflater, ViewGroup parent) {
    super(inflater.inflate(R.layout.question_item, parent, false));
    checkBox = (CheckBox) itemView.findViewById(R.id.question_checkbox);
    question_txt = (TextView) itemView.findViewById(R.id.question);
    itemView.setOnClickListener(this);
    // 移除原来的checkBox.setOnClickListener，改用bind方法里的状态监听
}

额外建议 #

• 如果你的问题有唯一ID（比如_id），用ID作为checkStateMap的key会比位置更稳妥——因为如果列表数据有增删，位置会变化，但ID不会。
• 若后续有数据更新操作，记得同步更新checkStateMap的状态，避免出现数据和状态不一致的情况。

内容的提问来源于stack exchange，提问作者Abhishek Reddy