RecyclerView删除数据后未更新问题求助

核心问题分析 #

你遇到的问题主要有两个：

• position参数失效：onBindViewHolder的position是初始绑定的位置，但RecyclerView会复用ViewHolder，当列表项被复用后，这个position就不再对应当前ViewHolder的实际位置了。
• Room数据库实例重复创建：每次点击删除按钮都重新构建Room数据库实例，会导致性能损耗和潜在的内存泄漏。

修复后的代码 #

public void onBindViewHolder(@NonNull final favAdapter.ViewHolder holder, int position) {
    holder.favText.setText(favlist.get(position).getPoetry());
    holder.delbtn.setOnClickListener(new View.OnClickListener() {
        @Override
        public void onClick(View view) {
            // 获取当前ViewHolder对应的正确位置
            int currentPosition = holder.getAdapterPosition();
            if (currentPosition == RecyclerView.NO_POSITION) {
                return; // 位置无效时直接返回，避免数组越界
            }

            // 注意：Room数据库实例应该全局初始化，不要每次点击都创建
            // 建议在Application或Activity中初始化DataBaseHandler，这里暂时保留你的逻辑示例
            DataBaseHandler db = Room.databaseBuilder(holder.delbtn.getContext(),
                    DataBaseHandler.class, "room_db").allowMainThreadQueries().build();
            Dao_Class userDao = db.userDao();
            userDao.delete(favlist.get(currentPosition));

            // 更新数据源并通知RecyclerView
            favlist.remove(currentPosition);
            notifyItemRemoved(currentPosition);
            // 可选：让后续列表项的位置更新动画更自然
            notifyItemRangeChanged(currentPosition, favlist.size());
        }
    });
}

额外优化建议 #

• 全局初始化Room实例：在Application类或者Activity的onCreate方法中初始化一次DataBaseHandler，通过单例模式获取，避免重复创建数据库连接。
• 移除主线程数据库操作：allowMainThreadQueries()仅适合测试场景，正式项目中建议用协程、RxJava等异步方式执行数据库操作，防止阻塞主线程引发ANR。

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