RecyclerView删除条目后其他条目重复问题求助

兄弟，我仔细看了你的代码，问题根源其实很明确，咱们一步步来修复：

核心问题1：数据库监听时没有清空旧列表 #

你的onDataChange方法每次触发（比如删除条目后数据库更新），都会把符合条件的条目追加到mItemList里，而不是先清空原有数据。这就导致旧的条目还在，新的又加进来，视图自然就重复了。

修复代码：修改onDataChange #

@Override
public void onDataChange(DataSnapshot dataSnapshot) {
    // 关键：每次获取新数据前先清空本地列表
    mItemList.clear();
    
    for (DataSnapshot dataSnapshot1 : dataSnapshot.getChildren()) {
        Item value = dataSnapshot1.getValue(Item.class);
        String valueEmail = "";
        if (value.getUserEmail() != null) {
            valueEmail = value.getUserEmail();
        }
        for (int i = 0; i < realSize; i++) {
            if (allUsers[i].equals(valueEmail)) {
                Item item = new Item();
                String name = value.getName();
                Integer amount = value.getAmount();
                String container = value.getContainer();
                String expiryDate = value.getExpiryDate();
                String reminderDate = value.getReminderDate();
                String owner = value.getOwner();
                Double price = value.getPrice();
                item.setName(name);
                item.setAmount(amount);
                item.setContainer(container);
                item.setExpiryDate(expiryDate);
                item.setReminderDate(reminderDate);
                item.setOwner(owner);
                item.setPrice(price);
                // 新增：保存数据库条目的key，后面删除需要用到
                item.setKey(dataSnapshot1.getKey());
                
                mItemList.add(item);
            }
        }
    }
    // 遍历完所有数据后统一通知适配器更新，别在循环里调用！
    mItemAdapter.notifyDataSetChanged();
}

核心问题2：删除时未正确同步本地列表（可选优化） #

虽然数据库监听会自动刷新列表，但直接操作本地列表再通知适配器，体验会更流畅（避免全量刷新）。另外你需要在Item类里加个key字段，用来定位数据库中的条目：

第一步：给Item类添加key字段 #

public class Item {
    // 你的原有字段...
    private String key; // 存储数据库中条目的唯一key

    // Firebase需要空构造函数，别删
    public Item() {}

    // 新增getter和setter
    public String getKey() {
        return key;
    }

    public void setKey(String key) {
        this.key = key;
    }

    // 你的其他getter/setter...
}

第二步：处理长按删除逻辑 #

在你的ItemTouchListener的长按回调里添加删除逻辑（如果还没实现长按回调，先补上）：

@Override
public void onLongClick(View view, int position) {
    Item deletedItem = mItemList.get(position);
    // 先删除数据库中的条目
    myRef.child(deletedItem.getKey()).removeValue(new DatabaseReference.CompletionListener() {
        @Override
        public void onComplete(@Nullable DatabaseError error, @NonNull DatabaseReference ref) {
            if (error == null) {
                // 数据库删除成功后，同步删除本地列表的条目
                mItemList.remove(position);
                // 只刷新被删除的位置，比notifyDataSetChanged更高效
                mItemAdapter.notifyItemRemoved(position);
                // 可选：通知后续条目索引变化，避免索引混乱
                mItemAdapter.notifyItemRangeChanged(position, mItemList.size());
                Toast.makeText(ItemDisplayActivity.this, "条目已删除", Toast.LENGTH_SHORT).show();
            } else {
                Toast.makeText(ItemDisplayActivity.this, "删除失败：" + error.getMessage(), Toast.LENGTH_SHORT).show();
            }
        }
    });
}

额外注意点 #

• 别在onDataChange的循环里调用notifyDataSetChanged()，每次循环都刷新会导致RecyclerView频繁重绘，性能差，而且可能出现闪烁。
• 确保你的ItemAdapter是正确绑定mItemList的，适配器里的数据源要和Activity里的mItemList是同一个引用。

这样修改后，删除条目时本地列表和数据库都会同步更新，而且不会出现重复的问题了~

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