java - Activity 里面的内部类是不是不能接受？

Question

内存泄漏的一个众所周知的来源是 AsyncTasks 持有对 Activity 的引用，例如，当将任务声明为内部类时:

    public class MyActivity extends Activity {

        ...

        private class MyTask extends AsyncTask<Void, Void, Void> {
            ...
        }
    }

因为它们有关联的后台线程(或线程池)，只要线程在运行，即使 Activity 被关闭，对 Activity 的引用也会保留。

现在(如果我错了请纠正我)，任务完成后， Activity 将再次可供收集。

我有一个 Activity ，其中有:

• Runnables 定义为内部类，并通过处理程序发布到主线程。
• TimerTasks 定义为内部类。

我是否可以通过取消计时器并在处理程序上调用 removeCallbacks 来防止内存泄漏？

例子:

    public class MyActivity extends Activity {
        Handler handler;
        Timer timer;


        @Override
        public void onCreate(Bundle savedInstanceState) {
            super.onCreate(savedInstanceState);

            handler = new Handler();
            timer = new Timer();

            handler.postDelayed(new MyRunnable(), 10000);
            timer.schedule(new MyTimerTask(), 1000);
        }

        @Override
        protected void onDestroy() {
            super.onDestroy();

            handler.removeCallbacksAndMessages(null);
            timer.cancel();
            handler = timer = null;
        }

        private class MyTimerTask extends TimerTask {
            ...
        }

        private class MyRunnable implements Runnable {
            ...
        }
    }

我在目前工作的项目中看到过这样的 Activity ，我正在考虑重构它，使内部类静态化，并向 Activity 中添加 WeakReferences。
问题是内部类类很方便，因为它们可以访问 Activity 的私有(private)变量和方法，将它们更改为静态也迫使我将 Activity 类中这些成员的访问修饰符从私有(private)更改为包默认，从而使它们可用于任何其他类在同一个包中。这也打破了对称性，因为现在 Activity 中有一些方法是包默认的，并且它们的对称对应物仍然是私有(private)的(例如，我需要使 Activity 中的 stopFoo 方法可用到现在的静态类，那么我应该如何处理未从这些类调用的 startFoo？)。 Activity 中需要访问的变量又如何呢？我现在应该提供 setter/getter 吗？
添加对 Activity 的弱引用还在重构的静态类中引入了大量的 null 检查，以便它们仅在 Activity 仍然存在时才起作用。除此之外，公司中不熟练的程序员可能难以理解弱引用方法(此外，他们可能只是在没有真正需要的情况下开始到处使用它)。

简而言之，重构后的 Activity 看起来很丑，而以前的 Activity 很整洁。所以我想知道基于内部类的设计是否可以接受，因为我知道我真的不介意 Activity 结束后可运行任务或计时器任务是否继续运行一小段时间。

Answer 1

Now (and correct me if I'm wrong), when the task is done, the activity will be again available for collection

正确，假设没有其他任何东西对它有静态引用。

Could I prevent memory leaks just cancelling timers and calling removeCallbacks on the handler?

这些步骤会有所帮助。此外，您无论如何都需要这样做——您的 Timer 不会神奇地停止运行，除非您停止它。

I don't really mind if a runnable or timer task is running for a short time when the activity is over

从内存管理的角度来看，您的论点是合理的。虽然“避免非静态内部类”是一个很好的高级设计标准，但它不是绝对的。

但是，恕我直言，您真正的问题不是内存管理，而是配置更改。当用户旋转屏幕、更改语言、将设备放入汽车底座等时，您的 Activity 将被销毁并重新创建。然而，据推测，任务的工作仍然需要，但它们将与错误的 Activity 实例对话。使用保留的 fragment 来管理这些任务有助于解决这个问题。