Android开发求助：Adapter类中Dialog.dismiss()方法失效但回调函数正常运行

看起来你遇到的问题是在RecyclerView Adapter里，点击关闭按钮时dialog.dismiss()没效果，但回调callback.onItemClicked(position)能正常执行对吧？这大概率是因为Dialog实例被覆盖或者ViewHolder复用导致多个Dialog叠加的问题，咱们一步步来解决：

可能的原因及解决方案 #

1. 避免Dialog实例被覆盖（最可能的原因） #

你在alertSuccessShowDialog方法里每次都新建一个Dialog并赋值给成员变量dialog，但RecyclerView的onBindViewHolder会在ViewHolder复用的时候多次调用，这就会导致成员变量dialog被后续创建的Dialog覆盖。当你点击之前弹出的Dialog的关闭按钮时，实际上调用的是最后一次创建的Dialog的dismiss()，而不是当前显示的那个Dialog，所以看起来没效果。

解决方法：不要用成员变量存储Dialog，而是在点击事件中直接获取当前Dialog实例，或者把Dialog作为局部变量并在点击事件中引用它：

修改alertSuccessShowDialog方法：

private void alertSuccessShowDialog(Context context, int position) {
    // 用局部变量存储Dialog，避免被后续创建的实例覆盖
    Dialog dialog = new Dialog(context);
    dialog.setContentView(R.layout.enquiry_popup);
    dialog.getWindow().setLayout(ViewGroup.LayoutParams.MATCH_PARENT, ViewGroup.LayoutParams.WRAP_CONTENT);
    dialog.getWindow().setBackgroundDrawableResource(android.R.color.transparent);
    dialog.getWindow().setGravity(Gravity.CENTER);
    dialog.setCancelable(false);
    dialog.findViewById(R.id.iv_close_btn).setOnClickListener(v -> {
        // 直接操作当前显示的Dialog实例
        dialog.dismiss();
        callback.onItemClicked(position);
    });
    dialog.show();
}

这样每次创建的Dialog都是独立的，点击关闭时操作的就是当前屏幕上显示的那个Dialog，不会被后续创建的实例干扰。

2. 防止多次弹出Dialog（ViewHolder复用导致） #

因为onBindViewHolder会在RecyclerView滚动、数据更新时反复调用，你的代码会在满足条件时多次弹出Dialog，导致多个Dialog叠加在屏幕上。即使你dismiss了一个，下面还有其他Dialog，看起来像是没生效。

解决方法：添加一个标记集合，确保每个item只弹出一次Dialog：

给Adapter添加一个用于记录已弹出Dialog的position集合：

private final Set<Integer> shownDialogPositions = new HashSet<>();

// 在onBindViewHolder中添加检查逻辑
@Override
public void onBindViewHolder(@NonNull ViewHolder holder, int position) {
    if (Helper.readSharedPref(mContext, Constants.PREF_KEY_Sub).equalsIgnoreCase("notActive")) {
        if (!mData.get(position).equalsIgnoreCase("PENDING")) {
            // 检查该position是否已经弹出过Dialog
            if (!shownDialogPositions.contains(position)) {
                alertSuccessShowDialog(mContext, position);
                shownDialogPositions.add(position);
            }
        }
    }
}

如果你的列表数据会动态更新，记得在数据刷新时调用shownDialogPositions.clear()来重置标记。

3. 确认Context的正确性 #

确保你传递给Adapter的Context是Activity的Context，而不是ApplicationContext。Dialog需要依附于Activity生命周期，使用ApplicationContext可能会导致一些异常行为（虽然你的回调能正常执行，这个可能性相对小，但还是要确认）。

可以在Adapter构造方法里加个简单校验：

public Adapter(Context mContext, ArrayList<String> mData, AdapterCallback callback) {
    // 确保传入的是Activity上下文
    if (!(mContext instanceof Activity)) {
        throw new IllegalArgumentException("Context must be an Activity context!");
    }
    this.mContext = mContext;
    this.mData = mData;
    this.callback = callback;
}

总结 #

优先尝试第一个方案，把Dialog改成局部变量，这应该能解决大部分情况。如果还有问题，再检查是否存在多次弹出Dialog的情况，或者Context是否正确。

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