Android中使用WeakReference避免内存泄漏的疑问

你的理解基本是正确的，但咱们得把细节和代码里的坑捋清楚，这样才能彻底避免内存泄漏和潜在的崩溃问题。

1. WeakReference为什么能解决旋转时的内存泄漏？ #

原来的问题在于：如果AsyncTask持有Activity的强引用，当屏幕旋转导致旧Activity被销毁时，只要AsyncTask还在后台运行，这个强引用就会让旧Activity无法被GC回收，从而引发内存泄漏。

而使用WeakReference后，它对Activity的引用是弱引用——GC在回收时，只要没有其他强引用指向这个Activity，就会直接回收它，不会因为AsyncTask的存在而卡住。所以屏幕旋转时，旧Activity确实能被正常销毁，不会出现内存泄漏，这部分你的判断是对的。

2. 你的代码里需要修正的几个问题 #

先看代码里的明显错误：

• 构造方法写错了：Java的构造方法没有返回值，你写了public void Background(Activity act)，这不是构造方法，只是普通方法！应该改成public Background(Activity act)（去掉void），否则你的WeakReference根本不会被初始化。
• WeakReference应该指定泛型：改成private WeakReference<Activity> activity;，这样后续使用时不需要强制类型转换，更安全。

3. 是否需要把对象置为null？ #

对于WeakReference本身，其实不需要特意置null——因为弱引用不会阻止GC回收。但有两个关键优化点要做：

（1）在Activity销毁时取消AsyncTask #

当Activity被销毁（比如旋转），后台任务其实已经不需要继续运行了，这时候应该主动取消AsyncTask：

@Override
protected void onDestroy() {
    super.onDestroy();
    if (mBackgroundTask != null && !mBackgroundTask.isCancelled()) {
        mBackgroundTask.cancel(true);
    }
}

调用cancel(true)会触发isCancelled()返回true，你可以在doInBackground()里定期检查这个状态，及时退出后台操作，避免浪费资源。

（2）在onPostExecute()里做双重判断 #

即使WeakReference能拿到Activity，也不代表它还能安全更新UI——比如Activity可能已经处于销毁流程中（isFinishing()或isDestroyed()为true）。所以要加双重判断：

@Override 
protected void onPostExecute(String result) { 
    Activity activity = activityRef.get();
    if (activity != null && !activity.isFinishing() && !activity.isDestroyed()) { 
        // 这里安全地更新UI
    } 
}

修正后的完整代码示例 #

Main.java #

public class Main extends Activity { 
    private TextView mMessageView;
    private Background mBackgroundTask;

    @Override 
    protected void onCreate(Bundle savedInstanceState) { 
        super.onCreate(savedInstanceState); 
        setContentView(R.layout.your_main_layout); // 替换成你的布局ID
        mMessageView = findViewById(R.id.message_text); // 初始化TextView
        mBackgroundTask = new Background(this); 
        mBackgroundTask.execute(); 
    }

    @Override
    protected void onDestroy() {
        super.onDestroy();
        // 销毁时取消后台任务
        if (mBackgroundTask != null && !mBackgroundTask.isCancelled()) {
            mBackgroundTask.cancel(true);
        }
    }
}

Background.java #

public class Background extends AsyncTask<String, Void, String> { 
    private WeakReference<Activity> activityRef;

    // 修正后的构造方法
    public Background(Activity act) { 
        activityRef = new WeakReference<>(act); 
    } 

    @Override 
    protected String doInBackground(String... params) { 
        // 后台操作中定期检查是否被取消，及时退出
        if (isCancelled()) {
            return null;
        }
        // 模拟耗时操作
        try {
            Thread.sleep(3000);
        } catch (InterruptedException e) {
            // 被取消时会抛出中断异常，直接返回
            return null;
        }
        return "任务完成！";
    } 

    @Override 
    protected void onPostExecute(String result) { 
        Activity activity = activityRef.get();
        if (activity != null && !activity.isFinishing() && !activity.isDestroyed()) { 
            // 安全更新UI
            ((Main)activity).mMessageView.setText(result);
        } 
    }

    @Override
    protected void onCancelled(String result) {
        // 任务取消时可以清空引用，不过弱引用本身不影响GC，可选操作
        activityRef.clear();
    }
}

最后总结 #

• 你的核心理解是对的：WeakReference确实能避免AsyncTask导致的Activity内存泄漏；
• 额外的优化能让代码更健壮：取消后台任务、判断Activity状态、修正构造方法错误。

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