如何清除Firebase实时数据库填充的ListView中的重复项

看起来你遇到的核心问题是每次调用fetchChooseList()时，都会重复绑定Firebase的ChildEventListener，旧的监听器没有被移除，导致数据库里的每条记录被多次触发onChildAdded回调，进而重复添加到ListView中。比如数据库有3条记录，第3次调用方法时，每条记录就会被添加3次，最终列表里出现9条重复项。

下面是具体的修复方案：

关键问题拆解 #

1. 重复绑定监听器：每次调用方法都会给shiftSwapDb新增一个ChildEventListener，但之前的监听器并没有被移除，同一个数据节点会被多次监听，每次触发都会往列表里追加数据。
2. 列表未清空复用：加载新数据前没有清空原有列表，新数据直接叠加在旧数据上，进一步加剧重复问题。

修复步骤 #

1. 将监听器保存为成员变量 #

这样我们可以在每次调用方法前，先移除之前绑定的监听器，避免重复监听数据库节点。

2. 加载数据前清空列表 #

在获取新数据前，清空swapBodyList和适配器中的数据，确保列表处于干净状态。

3. 优化适配器初始化逻辑 #

如果不需要每次都创建新适配器，可以复用已有的实例，只更新数据即可；如果必须重新创建，也要确保旧监听器已被移除。

修改后的完整代码 #

// 把ChildEventListener、列表、适配器声明为Activity的成员变量
private ChildEventListener swapChildEventListener;
private List<SwapDetails> swapBodyList;
private ShiftProfileAdapter shiftProfileAdapter;
private ListView listView;

private void fetchChooseList() { 
    DatabaseReference shiftSwapDb = FirebaseDatabase.getInstance().getReference().child("swaps").child("shift_swaps"); 
    
    // 1. 先移除之前的监听器（如果存在），避免重复监听
    if (swapChildEventListener != null) {
        shiftSwapDb.removeEventListener(swapChildEventListener);
    }

    // 2. 初始化或清空列表
    if (swapBodyList == null) {
        swapBodyList = new ArrayList<>();
    } else {
        swapBodyList.clear();
        // 如果你的Adapter有clear方法直接调用，没有的话可以用notifyDataSetChanged()
        if (shiftProfileAdapter != null) {
            shiftProfileAdapter.clear();
        }
    }

    // 3. 初始化ListView和适配器（仅第一次调用时创建）
    if (listView == null) {
        listView = chooseShiftProfileDialog.findViewById(R.id.listShiftProfileChooseDialog); 
        shiftProfileAdapter = new ShiftProfileAdapter(ProfileActivityShift.this, R.layout.shift_profile_list_item, swapBodyList); 
        listView.setAdapter(shiftProfileAdapter); 
    } else {
        shiftProfileAdapter.notifyDataSetChanged();
    }

    // 4. 创建并绑定新的监听器
    swapChildEventListener = new ChildEventListener() { 
        @Override 
        public void onChildAdded(DataSnapshot dataSnapshot, String s) { 
            if (dataSnapshot.exists()) { 
                SwapDetails swapDetails = dataSnapshot.getValue(SwapDetails.class); 
                if (swapDetails.getSwapperID().equals(fromID)) { 
                    // 可选：防止数据库本身存在重复数据，先判断是否已在列表中
                    if (!swapBodyList.contains(swapDetails)) {
                        swapBodyList.add(swapDetails);
                        shiftProfileAdapter.add(swapDetails);
                    }
                } 
            } 
        } 
        @Override 
        public void onChildChanged(DataSnapshot dataSnapshot, String s) { } 
        @Override 
        public void onChildRemoved(DataSnapshot dataSnapshot) { } 
        @Override 
        public void onChildMoved(DataSnapshot dataSnapshot, String s) { } 
        @Override 
        public void onCancelled(DatabaseError databaseError) { } 
    };
    shiftSwapDb.addChildEventListener(swapChildEventListener); 

    // 原有的ItemClickListener逻辑保持不变
    listView.setOnItemClickListener(new AdapterView.OnItemClickListener() { 
        @Override 
        public void onItemClick(AdapterView<?> adapterView, View view, int i, long l) { 
            // ... 你的原有点击事件代码 ...
        } 
    }); 
}

额外优化建议 #

• 给SwapDetails类重写equals()和hashCode()方法，这样swapBodyList.contains(swapDetails)才能正确判断是否存在重复记录（如果数据库本身可能有重复数据的话）。
• 如果fetchChooseList()会在对话框打开时多次调用，建议在对话框关闭时也移除监听器，避免不必要的资源消耗。

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