解决RecyclerView适配器中选中复选框ID无法全部存入List<Integer>的问题

看起来你遇到的问题是RecyclerView里的复选框只能把最后一个选中的ID存入列表，没法保存多个选中项的ID。我帮你分析下代码里的问题，然后给出修复方案。

原代码里的核心问题 #

1. 复选框状态复用与重复添加ID：RecyclerView的ViewHolder是复用的，onBindViewHolder里的if(holder.checkBox.isChecked())逻辑会在滚动时重复添加已选中的ID到列表，而且初始绑定的时候没有正确关联当前项的选中状态和列表。
2. 未处理取消选中的情况：当用户取消勾选复选框时，没有从ProcedureIDs列表中移除对应的ID，导致列表数据不准确。
3. ID获取方式不够可靠：从TextView中解析ID容易出现问题（比如TextView内容被意外修改），应该直接从数据源或ViewHolder的变量中获取。
4. ViewHolder的procedureID未正确赋值：在绑定数据时没有把当前项的ID赋值给ViewHolder的procedureID，导致复选框监听里的ID可能不准确。

修改后的完整代码 #

public class ProcedureAdapter extends RecyclerView.Adapter<ProcedureAdapter.MyViewHolder> { 
    private List<LabAppointmentProcedure> labAppointmentProcedureList; 
    private List<Integer> procedureIDs = new ArrayList<>(); // 变量名遵循驼峰命名规范

    class MyViewHolder extends RecyclerView.ViewHolder { 
        TextView procedureName, procedureId; 
        private int procedureID; 
        CheckBox checkBox; 

        MyViewHolder(final View itemView){ 
            super(itemView); 
            procedureName = itemView.findViewById(R.id.txtProcedureName); 
            procedureId = itemView.findViewById(R.id.txtProcedureId); 
            checkBox = itemView.findViewById(R.id.chkProcedure); 

            // 复选框状态变化监听
            checkBox.setOnCheckedChangeListener((buttonView, isChecked) -> {
                if (isChecked) {
                    // 先判断是否已存在，避免重复添加
                    if (!procedureIDs.contains(procedureID)) {
                        procedureIDs.add(procedureID);
                    }
                } else {
                    // 取消选中时移除对应的ID
                    procedureIDs.remove(Integer.valueOf(procedureID));
                }
                // 更新外部的列表引用
                EnterSample.ProcedureSampleIDs = procedureIDs;
            });
        } 
    } 

    public ProcedureAdapter(List<LabAppointmentProcedure> labAppointmentProceduresList){ 
        this.labAppointmentProcedureList = labAppointmentProceduresList; 
        // 如果需要保留之前的选中状态，可以在这里初始化procedureIDs（比如从外部传入已选ID）
    } 

    @NonNull 
    @Override 
    public MyViewHolder onCreateViewHolder(@NonNull ViewGroup viewGroup, int i){ 
        View itemView = LayoutInflater.from(viewGroup.getContext()).inflate(R.layout.procedure_row_item, viewGroup, false); 
        return new MyViewHolder(itemView); 
    } 

    @Override 
    public void onBindViewHolder(@NonNull MyViewHolder holder, int position){ 
        LabAppointmentProcedure labAppointmentProcedure = labAppointmentProcedureList.get(position); 
        holder.procedureName.setText(labAppointmentProcedure.getProcedure()); 
        holder.procedureId.setText(String.valueOf(labAppointmentProcedure.getProcedureID())); 
        // 正确赋值当前项的ID给ViewHolder
        holder.procedureID = labAppointmentProcedure.getProcedureID(); 

        // 绑定复选框状态：根据当前ID是否在选中列表中来设置
        boolean isChecked = procedureIDs.contains(holder.procedureID);
        holder.checkBox.setChecked(isChecked);
    } 

    @Override 
    public int getItemCount(){ 
        return labAppointmentProcedureList.size(); 
    } 

    // 提供外部获取选中ID列表的方法（可选，比直接暴露变量更安全）
    public List<Integer> getSelectedProcedureIDs() {
        return new ArrayList<>(procedureIDs); // 返回副本，避免外部修改内部列表
    }
}

关键修改点说明 #

1. 规范变量命名：把ProcedureIDs改成procedureIDs，遵循Java驼峰命名规范，代码更易读。
2. 修复复选框状态绑定：在onBindViewHolder中根据procedureIDs列表来设置复选框的选中状态，解决RecyclerView复用导致的状态混乱问题。
3. 处理取消选中逻辑：在复选框监听中，取消选中时从列表移除对应的ID，保证列表数据和选中状态一致。
4. 避免重复添加ID：添加ID前先判断是否已存在，防止滚动时重复添加。
5. 可靠获取ID：直接从ViewHolder的procedureID变量获取ID，不再依赖TextView，避免潜在问题。
6. 安全的外部访问：新增getSelectedProcedureIDs方法返回列表副本，避免外部直接修改适配器内部的列表，提高代码健壮性。

内容的提问来源于stack exchange，提问作者Adeel Ahmed