RecyclerView删除图片后列表出现重复图片问题求助

问题描述 #

从RecyclerView删除图片后，列表内其他图片出现重复显示的情况。

删除逻辑代码 #

galleryAdapter.setOnLongClickListiner(new GalleryAdapter.OnItemLongClickListiner() {
    @Override
    public void onItemLongClick(Gallery gallery, int position) {
       
        DocumentReference documentReference=firebaseFirestore.collection("Gallery").document(firebaseUser.getUid()).collection("myGallery").document(galleryModelArrayList.get(position).docId);
        documentReference.delete().addOnCompleteListener(new OnCompleteListener<Void>() {
            @Override
            public void onComplete(@NonNull Task<Void> task) {
                if(task.isSuccessful()){
                    galleryModelArrayList.remove(galleryModelArrayList.get(position));
                    galleryAdapter.notifyDataSetChanged();
                    Toast.makeText(getApplicationContext(),R.string.gallerySilindi,Toast.LENGTH_SHORT).show();

                }else{
                    Toast.makeText(getApplicationContext(),"ERROR:"+task.getException().getMessage(),Toast.LENGTH_SHORT).show();
                }
            }
        });
        

    }
});

图片加载逻辑代码 #

public  void  getData() {

    firebaseFirestore.collection("Gallery").document(firebaseUser.getUid()).collection("myGallery").orderBy("date", Query.Direction.DESCENDING).addSnapshotListener(new EventListener<QuerySnapshot>() {
        @Override
        public void onEvent(@Nullable QuerySnapshot value, @Nullable FirebaseFirestoreException error) {

            if (error != null) {
                Toast.makeText(getApplicationContext(), error.getLocalizedMessage(), Toast.LENGTH_LONG).show();
            }
            if (value != null) {
                for (DocumentSnapshot snapshot : value.getDocuments()) {
                    Map<String, Object> data = snapshot.getData();
                    String downloadUrl = (String) data.get("downloadUrl");
                    String docId = (String) data.get("docId");
                    String ımagaId = (String) data.get("imageId");
                   // checkBox=findViewById(R.id.checkimages);
                        GalleryModel galleryModel=new GalleryModel(docId,downloadUrl,checkBox,ımagaId);
                    galleryModelArrayList.add(galleryModel);

                }
                galleryAdapter.notifyDataSetChanged();

            }

        }
    });
}

疑问 #

用上述方式加载图片到列表并通过RecyclerView展示，这部分逻辑是不是存在问题？

问题截图 #

问题根源与修复方法 #

核心问题 #

1. 实时监听器重复累加数据：addSnapshotListener是实时监听Firestore数据的方法，每次数据变化（包括删除操作）都会触发onEvent回调，但你在回调里直接往galleryModelArrayList中添加数据，没有先清空旧列表，导致旧数据与新数据叠加，出现重复。
2. 手动更新列表与监听冲突：删除成功后你手动移除列表元素并通知更新，但紧接着实时监听到数据变化，又会把最新的全量数据（已移除删除的条目）再添加一遍，和之前剩下的元素重复。

修复步骤 #

1. 加载新数据前清空旧列表

在getData的onEvent回调中，遍历添加新数据前先清空galleryModelArrayList，避免旧数据残留：

if (value != null) {
    galleryModelArrayList.clear(); // 关键：先清空旧数据
    for (DocumentSnapshot snapshot : value.getDocuments()) {
        Map<String, Object> data = snapshot.getData();
        String downloadUrl = (String) data.get("downloadUrl");
        String docId = (String) data.get("docId");
        String ımagaId = (String) data.get("imageId");
        GalleryModel galleryModel=new GalleryModel(docId,downloadUrl,checkBox,ımagaId);
        galleryModelArrayList.add(galleryModel);
    }
    galleryAdapter.notifyDataSetChanged();
}

2. 移除手动更新列表的代码（可选）

由于使用了实时监听，删除操作成功后Firestore会自动同步数据，监听器会帮你更新列表，因此可以删掉手动修改列表的代码，避免重复操作：

if(task.isSuccessful()){
    Toast.makeText(getApplicationContext(),R.string.gallerySilindi,Toast.LENGTH_SHORT).show();
    // 删除下面两行，交给实时监听器处理列表更新
    // galleryModelArrayList.remove(galleryModelArrayList.get(position));
    // galleryAdapter.notifyDataSetChanged();
}else{
    Toast.makeText(getApplicationContext(),"ERROR:"+task.getException().getMessage(),Toast.LENGTH_SHORT).show();
}

3. 检查Adapter的ViewHolder复用逻辑

如果上述修改后仍有重复，需确认RecyclerView Adapter的onBindViewHolder方法是否每次都正确设置当前条目的数据（如图片URL、控件状态等），避免ViewHolder复用导致旧数据残留。

内容的提问来源于stack exchange，提问作者Y.İslam çalışkan