RecyclerView可展开项的ToggleButton监听误触发前已展开项的问题

我太懂你遇到的这个坑了——给RecyclerView可展开项的ToggleButton设置setOnCheckedChangeListener后，之前已经展开过的项会莫名其妙触发这个监听。这本质上是RecyclerView的ViewHolder复用机制带来的问题，咱们一步步捋清楚解决它。

先看看你贴的代码片段（我补全了截断的部分）：

override fun onBindViewHolder(holder: MyViewHolder, position: Int) {
    val isExpanded = position == expandedPosition
    holder.itemView.configurationLayout.visibility = if (isExpanded) View.VISIBLE else View.GONE
    holder.itemView.isActivated = isExpanded
    if (isExpanded) previousExpandedPosition = position

    holder.itemView.expandButton.setOnClickListener {
        expandedPosition = if (isExpanded) -1 else position
        notifyItemChanged(previousExpandedPosition)
        notifyItemChanged(position)
    }
    // 你后来加的ToggleButton监听大概是类似这样？
    holder.itemView.toggleButton.setOnCheckedChangeListener { _, isChecked ->
        // 处理展开/收起逻辑
    }
}

问题根源 #

RecyclerView会复用已经滑出屏幕的ViewHolder，如果你没在绑定新数据时做好这两件事，就会出问题：

• 没有清除旧的监听：ViewHolder被复用到新位置时，之前设置的监听还挂在ToggleButton上，一旦按钮状态因新数据更新变化，旧监听就会被执行
• 没有同步ToggleButton的当前状态：按钮状态和实际展开状态不一致，会触发不必要的监听回调

解决方法 #

我给你调整一下代码，把这两个核心问题解决掉：

1. 绑定新监听前先移除旧监听：用setOnCheckedChangeListener(null)先清空，避免残留
2. 同步ToggleButton的状态：根据当前项的展开状态设置isChecked，而且要在设置监听之前做，避免设置状态时触发新监听
3. 优化展开状态管理：确保previousExpandedPosition的更新和notifyItemChanged的调用更严谨

修改后的完整代码示例：

// 在Adapter里定义这两个状态变量
private var expandedPosition = -1
private var previousExpandedPosition = -1

override fun onBindViewHolder(holder: MyViewHolder, position: Int) {
    val isExpanded = position == expandedPosition

    // 1. 先同步ToggleButton的状态（必须在设置监听之前！）
    holder.itemView.toggleButton.isChecked = isExpanded
    // 同步展开布局的显示状态
    holder.itemView.configurationLayout.visibility = if (isExpanded) View.VISIBLE else View.GONE
    holder.itemView.isActivated = isExpanded

    // 2. 先清空旧监听，再绑定新监听
    holder.itemView.toggleButton.setOnCheckedChangeListener(null)
    holder.itemView.toggleButton.setOnCheckedChangeListener { _, isChecked ->
        // 更新展开状态
        previousExpandedPosition = expandedPosition
        expandedPosition = if (isChecked) position else -1
        // 只刷新需要变化的项，避免全量刷新
        notifyItemChanged(previousExpandedPosition)
        notifyItemChanged(position)
    }
}

// 额外保险：ViewHolder回收时清空监听，避免内存泄漏和残留回调
override fun onViewRecycled(holder: MyViewHolder) {
    super.onViewRecycled(holder)
    holder.itemView.toggleButton.setOnCheckedChangeListener(null)
}

额外提醒 #

• 一定要在设置监听之前同步ToggleButton的isChecked状态，不然设置状态时会立刻触发刚绑定的监听，导致不必要的逻辑执行
• 用notifyItemChanged()单独更新需要变化的项，别用notifyDataSetChanged()，既高效又能避免整个列表的ViewHolder被重新绑定带来的问题

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