最佳实践：静态方法中对活动的引用较弱

Question

我需要在几个静态方法中引用一个活动。我很想知道避免内存泄漏的最佳实践。让我们使用例子：

例1：

static void hideKeyboard(Activity activity) {
   WeakReference<Activity> activityReference = new WeakReference<>(activity);
   // ... Call activityReference.get() when needed, check if null...
}

例2：

static void hideKeyboard(WeakReference<Activity> activityReference) {
   // ... Call activityReference.get() when needed, check if null...
}

所以三个问题：

1. 例1或例2有什么区别吗

Answer 1

绝对没有理由在传递给方法的参数中使用WeakReference，除非存储了该参数。如果参数仅在方法中使用，则只需传入活动引用即可。

Answer 2

不，这没什么区别。Java中的垃圾收集基于GC根的思想。如果变量是GC根或由GC根引用（包括传递），则不能对其进行垃圾收集。函数的参数是GC根，直到函数返回无法收集的参数为止。由于Activity是函数的一个参数，所以只要该函数在调用堆栈中，Activity就不可收回。使用WeakReference不会加快速度。

线程和异步任务（实际上只是围绕线程的包装）略有不同。每个正在运行的线程也是一个GC根。但是线程可以有很长的生命周期，并且存在于对象的生命周期之外。在这里，使用WeakReference可能会有所帮助，因为没有其他原因需要保留它（如示例中的参数）。

你的例子2稍微好一点，它不是完全不必要的。但是我质疑为什么需要它。一般来说，当做一个线程时，模式应该是：

run() {
   do_async_work()
   update_ui()
}

update_ui() {
  Activity activity = weakReference.get()
  if(activity == null) {
     return
  }

  //update the UI
}

这样做可以避免很多问题，比如需要多次检查弱引用。