Android中如何在公共类使用Dialog并避免内存泄漏

问题分析 #

你在公共类CommonUtility中声明了静态DialogChooser字段，导致Android Studio警告内存泄漏——静态引用会长期持有Dialog实例，而Dialog内部持有Button等View组件，这些组件又关联着创建Dialog的Activity Context。即使Activity被销毁，静态引用依然存在，导致Activity无法被GC回收，最终引发内存泄漏。

解决方案 #

核心思路是移除静态Dialog字段，将Dialog改为方法内的局部变量，同时优化回调逻辑，消除不必要的强引用依赖：

1. 删除公共类中的静态Dialog字段 #

把CommonUtility里的静态purchaseDialog字段彻底移除，将Dialog改为openDialog方法内的局部变量。这样Dialog的生命周期会和方法调用的上下文绑定，不会长期占用内存，GC可以在Dialog销毁或Activity结束时正常回收相关资源。

2. 取消回调中的手动dismiss操作 #

原DialogChooser的onClick方法中已经自动调用了dismiss()，所以回调里无需再手动调用purchaseDialog.dismiss()，彻底摆脱对外部Dialog引用的依赖。

3. 用弱引用包裹Context（可选但推荐） #

匿名内部类会默认持有外部类的强引用，为了避免Activity被意外持有无法回收，用WeakReference包裹Context，确保Activity可以被正常GC回收。

修改后的代码示例 #

优化后的CommonUtility类 #

public class CommonUtility {

    // 移除静态Dialog字段
    public static void openDialog(Context context) {
        // 用WeakReference包裹Context，避免强引用导致内存泄漏
        WeakReference<Context> contextWeakRef = new WeakReference<>(context);
        
        // 局部变量持有Dialog实例
        DialogChooser purchaseDialog = new DialogChooser(
                context,
                context.getString(R.string.message),
                context.getString(R.string.text_yes),
                context.getString(R.string.text_no),
                new DialogClickListener() {
                    @Override
                    public void onYesClick(View view) {
                        Context currentContext = contextWeakRef.get();
                        if (currentContext != null) {
                            Intent i = new Intent(currentContext, ActivityForExample.class);
                            currentContext.startActivity(i);
                        }
                    }

                    @Override
                    public void onNoClick(View view) {
                        // 无需手动dismiss，Dialog内部已处理
                    }

                    @Override
                    public void onCrossClick(View view) {
                        // 无需手动dismiss，Dialog内部已处理
                    }
                });
        purchaseDialog.show();
    }
}

保持DialogChooser类不变（确认核心逻辑） #

原DialogChooser的onClick方法已经正确实现了自动关闭逻辑，无需修改：

@Override
public void onClick(View v) {
    if (v.getId() == R.id.btn_yes) {
        dialogClickListener.onYesClick(v);
    } else if (v.getId() == R.id.btn_no) {
        dialogClickListener.onNoClick(v);
    } else if (v.getId() == R.id.btn_cross) {
        dialogClickListener.onCrossClick(v);
    }
    // 自动关闭Dialog，无需外部调用
    dismiss();
}

额外注意事项 #

• 确保传入openDialog的Context是Activity Context，不要用ApplicationContext——Dialog需要依附于Activity窗口，使用ApplicationContext会抛出WindowManager$BadTokenException异常。
• 如果需要在Dialog关闭后执行后续逻辑，可在回调中处理，但必须先通过WeakReference检查Context是否存活，避免空指针异常。

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