PostAdapter点击more弹出底部对话框后其他操作触发重复弹出如何解决

你当前的问题核心是误用了Firebase Realtime Database的addValueEventListener：

• addValueEventListener是持久化监听器，注册之后只要对应节点的数据发生任何变更（包括后续修改帖子内容、点赞、评论，甚至列表刷新触发的同步），都会自动回调onDataChange方法
• 每次点击more按钮都会注册一个新的监听器，不会自动销毁，点击n次more就会注册n个监听器，后续数据变更时n个监听器都会执行弹出对话框的逻辑，自然会出现多个对话框需要逐个关闭
• 就算切换Fragment再回来，只要监听器没被移除，对应节点数据更新依然会触发回调，导致对话框意外弹出

1. 替换持久化监听器为单次查询监听器 #

你只需要在点击more的时候查询一次帖子数据即可，不需要持续监听，把addValueEventListener替换成addListenerForSingleValueEvent，这个监听器执行完一次查询回调后就会自动销毁，不会后续重复触发：

viewHolder.binding.more.setOnClickListener(new View.OnClickListener() {
    @Override
    public void onClick(View v) {
        FirebaseDatabase.getInstance().getReference()
                .child("Posts")
                .child(post.getPostId())
                // 替换成单次查询监听器
                .addListenerForSingleValueEvent(new ValueEventListener() {
                    @Override
                    public void onDataChange(@NonNull DataSnapshot snapshot) {
                        if (snapshot.exists()) {
                            String postedBy = snapshot.child("postedBy").getValue().toString();
                            if (FirebaseAuth.getInstance().getCurrentUser().getUid().equals(postedBy)) {
                                showBottomSheetDialog(post, holder);
                            } else {
                                showBottomSheetDialogForOthersPosts(post, holder);
                            }
                        }
                    }

                    @Override
                    public void onCancelled(@NonNull DatabaseError error) {

                    }
                });
    }
});

2. 可选优化：避免重复弹出对话框 #

如果还有特殊场景需要保留addValueEventListener，可以增加全局变量标记当前是否已经有对话框显示，避免重复创建：

• 在Adapter中增加成员变量 private Dialog currentShowingDialog;
• 每次创建对话框之前先判断是否已有显示的对话框，有的话先dismiss：

private void showBottomSheetDialogForOthersPosts(Post post, RecyclerView.ViewHolder holder) {
    // 已有对话框显示时先关闭
    if (currentShowingDialog != null && currentShowingDialog.isShowing()) {
        currentShowingDialog.dismiss();
    }
    final Dialog dialog = new Dialog(context);
    // 其余原有逻辑不变
    dialog.show();
    // 把当前对话框赋值给全局变量
    currentShowingDialog = dialog;
}

• 在Adapter的onViewRecycled或者Fragment的onDestroyView生命周期中主动dismiss对话框，避免窗口泄露：

@Override
public void onViewRecycled(@NonNull PostViewHolder holder) {
    super.onViewRecycled(holder);
    if (currentShowingDialog != null) {
        currentShowingDialog.dismiss();
        currentShowingDialog = null;
    }
}

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