Kotlin RecyclerView中Checkbox随机选中问题排查与解决

问题描述 #

实现带复选框的问题列表，点击某条的“Yes”复选框时，其他条目（如第8、14条）的复选框会被随机选中。例如勾选第1条“Yes”，第8、14条对应复选框也会被标记。

问题根源 #

这是RecyclerView的ViewHolder复用机制导致的：

• 列表滚动时，屏幕外的ViewHolder会被回收复用给新进入屏幕的条目
• 原代码仅在onBindViewHolder中设置了chbksi的状态，但未处理用户点击后的状态保存，也未对其他复选框的状态做正确绑定，更没有在复用的时候重置所有复选框状态，导致旧的选中状态被带到新条目中
• 不同位置的条目显示不同的复选框组合，但未为这些特殊复选框绑定对应的数据状态，复用后就会出现异常选中

解决方案 #

1. 完善数据类，保存所有复选框状态 #

修改epreguntas类，为每个可能显示的复选框添加独立的状态字段，确保每个条目能保存自己的选择：

class epreguntas(
    var id_pregunta: String,
    var num_pregunta: String,
    var pregunta : String,
    var icono_estado: String,
    // 基础复选框状态
    var checkSi: Boolean = false,
    var checkNo: Boolean = false,
    // 特殊复选框状态
    var checkIVSS: Boolean = false,
    var checkDSS: Boolean = false,
    var checkFSERV: Boolean = false,
    var checkFMano: Boolean = false,
    var checkNA: Boolean = false
)

2. 修改Adapter，正确绑定状态并添加监听 #

在onBindViewHolder中，先重置所有复选框状态，再根据位置设置显示/隐藏，同时绑定对应的数据状态和点击监听，将用户选择同步到数据类：

class preusoadapter(private val listpreguntaspreuso: ArrayList<epreguntas>) : RecyclerView.Adapter<PreUsoViewHolder>() {
    override fun onCreateViewHolder(parent: ViewGroup, viewType: Int): PreUsoViewHolder {
        val layoutInflater = LayoutInflater.from(parent.context)
        return PreUsoViewHolder(layoutInflater.inflate(R.layout.activity_estructura_listapreuso, parent, false))
    }

    override fun onBindViewHolder(holder: PreUsoViewHolder, position: Int) {
        val item = listpreguntaspreuso[position]
        holder.render(item)

        // 先重置所有复选框状态，避免复用旧状态
        resetAllCheckboxes(holder.itemView)

        // 根据位置配置复选框显示/隐藏、状态绑定与监听
        when(position) {
            2 -> {
                holder.itemView.chbkno.visibility = View.GONE
                holder.itemView.chbksi.visibility = View.GONE
                holder.itemView.chbkIVSS.visibility = View.VISIBLE
                holder.itemView.chbkDSS.visibility = View.VISIBLE
                
                holder.itemView.chbkIVSS.isChecked = item.checkIVSS
                holder.itemView.chbkDSS.isChecked = item.checkDSS
                
                holder.itemView.chbkIVSS.setOnCheckedChangeListener { _, isChecked ->
                    item.checkIVSS = isChecked
                }
                holder.itemView.chbkDSS.setOnCheckedChangeListener { _, isChecked ->
                    item.checkDSS = isChecked
                }
            }
            6 -> {
                holder.itemView.chbkFSERV.visibility = View.VISIBLE
                holder.itemView.chbkfmano.visibility = View.VISIBLE
                holder.itemView.chbkno.visibility = View.GONE
                holder.itemView.chbksi.visibility = View.GONE
                
                holder.itemView.chbkFSERV.isChecked = item.checkFSERV
                holder.itemView.chbkfmano.isChecked = item.checkFMano
                
                holder.itemView.chbkFSERV.setOnCheckedChangeListener { _, isChecked ->
                    item.checkFSERV = isChecked
                }
                holder.itemView.chbkfmano.setOnCheckedChangeListener { _, isChecked ->
                    item.checkFMano = isChecked
                }
            }
            14, 18, 22, 25 -> {
                holder.itemView.chbkna.visibility = View.VISIBLE
                holder.itemView.chbkno.visibility = View.VISIBLE
                holder.itemView.chbksi.visibility = View.VISIBLE
                
                holder.itemView.chbksi.isChecked = item.checkSi
                holder.itemView.chbkno.isChecked = item.checkNo
                holder.itemView.chbkna.isChecked = item.checkNA
                
                setBasicCheckListeners(holder.itemView, item)
                holder.itemView.chbkna.setOnCheckedChangeListener { _, isChecked ->
                    item.checkNA = isChecked
                }
            }
            else -> {
                holder.itemView.chbkno.visibility = View.VISIBLE
                holder.itemView.chbksi.visibility = View.VISIBLE
                
                holder.itemView.chbksi.isChecked = item.checkSi
                holder.itemView.chbkno.isChecked = item.checkNo
                
                setBasicCheckListeners(holder.itemView, item)
            }
        }
    }

    // 重置所有复选框状态
    private fun resetAllCheckboxes(view: View) {
        view.chbksi.isChecked = false
        view.chbkno.isChecked = false
        view.chbkIVSS.isChecked = false
        view.chbkDSS.isChecked = false
        view.chbkFSERV.isChecked = false
        view.chbkfmano.isChecked = false
        view.chbkna.isChecked = false
    }

    // 封装基础复选框的监听设置（支持Yes/No互斥）
    private fun setBasicCheckListeners(view: View, item: epreguntas) {
        view.chbksi.setOnCheckedChangeListener { _, isChecked ->
            item.checkSi = isChecked
            // 若需要互斥，取消注释下面一行
            // if (isChecked) view.chbkno.isChecked = false
        }
        view.chbkno.setOnCheckedChangeListener { _, isChecked ->
            item.checkNo = isChecked
            // 若需要互斥，取消注释下面一行
            // if (isChecked) view.chbksi.isChecked = false
        }
    }

    override fun getItemCount(): Int = listpreguntaspreuso.size
}

3. 核心逻辑说明 #

• 所有复选框状态与数据类字段一一绑定，用户选择直接修改数据类，确保ViewHolder复用时能读取最新状态
• 每次绑定ViewHolder先重置所有复选框状态，彻底避免旧状态残留
• 若需要复选框互斥（如Yes和No不能同时选中），可取消代码中的互斥逻辑注释

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