RecyclerView点击单一行却触发多行外观变更问题排查

哥们，你碰到的这个问题是RecyclerView开发者几乎都会踩的坑——ViewHolder复用导致的UI状态残留。

为啥会这样？ #

RecyclerView为了高效利用内存，会重复利用那些已经滚出屏幕的ViewHolder，而不是每次都创建新的实例。你现在的代码只在数据项isSelected()为true时设置了紫色背景，但完全没处理isSelected()为false时的重置逻辑。举个实际场景：

• 你点击第1行，它的背景变成紫色，接着把它滑出屏幕；
• 当第5行滑进屏幕时，RecyclerView会直接复用第1行用过的ViewHolder；
• 第5行的isSelected()明明是false，但你的代码没给它设置默认背景色，ViewHolder上残留的紫色就直接显示在第5行上了，看起来就像未点击的行也变了色。

怎么修复？ #

很简单，在onBindViewHolder里补全未选中状态的UI重置逻辑，确保不管ViewHolder有没有被复用，都能根据当前数据的状态设置正确的UI：

修改你的Adapter中onBindViewHolder代码如下：

@Override 
public void onBindViewHolder(@NonNull RecyclerView.ViewHolder holder, final int position) { 
    if (holder instanceof ViewHolderHeader){ 
        ViewHolderHeader headerHolder = (ViewHolderHeader) holder;
        objClass_offerDateHeader headerItem = (objClass_offerDateHeader) arrayList.get(position);
        
        // 设置文本
        headerHolder.dateHeaderTextView.setText(Integer.toString(headerItem.getDate()));
        
        // 同时处理选中和未选中两种状态，彻底覆盖复用的残留UI
        if (headerItem.isSelected()){ 
            headerHolder.dateHeaderTextView.setBackgroundColor(Color.parseColor("#b642f4")); 
        } else {
            // 这里替换成你原本的默认背景色，比如白色或者透明，根据你的需求来
            headerHolder.dateHeaderTextView.setBackgroundColor(Color.WHITE);
        } 
        
        headerHolder.dateHeaderTextView.setOnClickListener(v -> 
            onHeaderClickListener.onHeaderClicked(position)
        ); 
    } 
}

额外的小优化 #

• 不要每次都调用notifyDataSetChanged()，改用notifyItemChanged(index)只刷新被点击的那一行，性能会更好；
• 可以把点击事件的绑定移到onCreateViewHolder里，避免每次绑定ViewHolder都创建新的OnClickListener，减少不必要的对象创建。

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