RecyclerView多Holder项删除异常：Holder指向同一内存对象问题

这个问题我之前也踩过坑！本质是RecyclerView的ViewHolder复用机制在作祟，当列表项数量较多时，滑出屏幕的Holder会被回收复用给新进入的项，你的代码里有两个关键问题导致了这个现象：

问题根源 #

1. 位置获取时机错误：你在调用notifyItemRemoved()之后再用holder.getAdapterPosition()，此时这个Holder对应的位置已经被移除，getAdapterPosition()会返回RecyclerView.NO_POSITION，后续的notifyItemRangeChanged()用了无效的位置，会引发异常或状态混乱。
2. 复用状态未重置：点击后你把deleteIcon设为false，但RecyclerView复用这个Holder时，没有重置这个状态，导致被复用的新项的删除按钮也是禁用的，看起来像是多个Holder指向了同一对象（实际是同一个Holder实例被复用了）。

修复方案 #

1. 提前保存有效位置，避免无效操作 #

在修改数据集之前就获取并保存当前的有效位置，不要在notifyItemRemoved()之后再获取。

2. 在onBindViewHolder中重置Holder状态 #

每次绑定Holder时，强制重置删除按钮的启用状态，避免复用带来的状态残留。

完整修正代码 #

首先在Adapter的onBindViewHolder方法中添加状态重置：

@Override
public void onBindViewHolder(@NonNull MyViewHolder holder, int position) {
    // 这里写你的数据绑定逻辑，比如设置文本、图片等
    
    // 关键：重置删除按钮的启用状态，确保每个新绑定的项按钮都是可用的
    holder.deleteIcon.setEnabled(true);
}

然后修改删除按钮的点击事件：

holder.deleteIcon.setOnClickListener(new View.OnClickListener() {
    @Override
    public void onClick(View v) {
        // 先获取当前有效位置，在修改数据集前保存
        int currentPosition = holder.getAdapterPosition();
        
        // 检查位置是否有效，避免Holder已被回收时的无效操作
        if (currentPosition == RecyclerView.NO_POSITION) {
            return;
        }
        
        // 临时禁用按钮防止重复点击
        holder.deleteIcon.setEnabled(false);
        
        // 删除数据集中的对应项
        items.remove(currentPosition);
        
        // 通知RecyclerView移除对应位置的项
        notifyItemRemoved(currentPosition);
        
        // 如果你的列表项有依赖位置的内容（比如序号），再调用此方法更新后续项的位置
        // 注意：这里的数量是剩余项的数量，即 items.size() - currentPosition
        notifyItemRangeChanged(currentPosition, items.size() - currentPosition);
    }
});

额外优化建议 #

如果你的列表项有唯一且稳定的ID，可以在Adapter中设置：

@Override
public long getItemId(int position) {
    // 返回你的项的唯一ID，比如数据模型的id字段
    return items.get(position).getId();
}

// 在Adapter的构造方法中添加
setHasStableIds(true);

这会让RecyclerView的复用机制更稳定，减少位置错乱的概率。

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