android - 使用动态屏幕时如何防止内存泄漏？

Question

我正在创建一个应用程序，该应用程序具有向用户显示数据的屏幕。每个 Screen 都有自己的数据和自己的布局，所以它有一个方法来返回一个 int 代表用于膨胀它的布局，然后这个 View 被传递给函数以查找特定 View 并用数据填充它。

生命周期是这样的:主讲人:

screen.getNextScreen ->
screen.getLayout -> 
view = inflateScreen ->
screen.populateScreen(view) ->
(wait for time elappsed or click) -> repeat

SettingsActivity 中也需要那些 Screens 来启用\禁用它们。

所以我创建了一个单例ScreenProvider，它初始化一次然后返回列表。

public class ScreenProvider {

    private List<Screen> screens;

    private static ScreenProvider instance = new ScreenProvider();

    public static ScreenProvider getInstance(){
        return instance;
    }

    private ScreenProvider() {
        screens = new ArrayList<>();

        screens.add(new Welcome());
        screens.add(new CompoundScreen());
        screens.add(new Times());
        screens.add(new Messages());
        screens.add(new Weekly());
    }

    public List<Screen> getScreenList() {
        return Lists.newArrayList(screens);
    }
}

它的接缝是当运行时间过长时应用程序崩溃或因内存泄漏而关闭，所以我添加了 leakcanary，这是其报告的一个示例:

MainActivity has leaked:
D: * static ScreenProvider.!(instance)!
D: * ↳ ScreenProvider.!(screens)!
D: * ↳ ArrayList.!(array)!
D: * ↳ array Object[].!([0])!
D: * ↳ CompoundScreen.!(disposable)!
D: * ↳ LambdaObserver.!(onNext)!
D: * ↳ -$$Lambda$Screen$67KdQ1jl3VSjSvoRred5JqLGY5Q.!(f$1)!
D: * ↳ AppCompatTextView.mContext
D: * ↳ MainActivity

这只是一个例子，但几乎每个屏幕都有这样的漏洞。LeakCanary 报告显示 TextView 有这个:D: | mAttachInfo = null 所以我认为这不是问题所在。此外，每个 Screen 都有一个 onHide() 来清除一次性用品，当当前 Screen 隐藏并在 MainActivity.onStop() 中调用。

如何解决这个漏洞？我不应该为屏幕使用单例吗？如果没有，我如何从其他 Activity 访问屏幕列表？

** 编辑 **添加一些每个屏幕都会覆盖的 Screen 主要方法。

public abstract int getLayout();

public boolean shouldShow()

public void populateData(View view)

public void onHide()

public abstract int getScreenIndex();

public boolean shouldCacheView()

public int getDuration()

Answer 1

好的。从你所说的和你所展示的来看，你似乎将一些生成的 View 的实例保留在单例中。不。每个 View 都需要通过代码或通货膨胀(这基本上是一个基于 XML 的、基于反射的工厂方法)来创建一个上下文，以访问应用程序和系统的资源，并保留对所述上下文的引用，例如只要他们活着。在您的场景中，这意味着保留对生成 View 的 Activity 的引用。通常，关于 View 和 Activity ，这是关于 GC 的情况:

GC: Hey! Does anybody need this... MainActivity class?
View: I do! I do! I have a reference!
GC: Okay... and besides MainActivity, Does anybody else need this View class?
-Nobody answers-
GC: It does not matter my friend, you are being collected as well. Come with me.
And they both go.

在你的情况下:

GC: Hey! Does anybody need this... MainActivity class?
View: I do! I do! I have a reference! and MainActivity references me as well.
GC: Okay... and besides MainActivity, Does anybody else need this View class?
ScreenProvider: I do.
GC: Okay, keep moving View, and take MainActivity with you. Let me know when you folks are done so I can collect you.

And thus the leak.

为了将 View 从一个 Activity 传递到另一个 Activity ，您需要删除对前一个 Activity 的引用(mContext 字段)。由于没有用于执行此操作的 API，因此您需要使用反射。并且出现了另一个问题:每个 UI 部分都是 View 的子类。布局、小部件等。因此，要么保留对 XML 文件的每一部分的引用，以便通过反射删除上下文，要么遍历 View 的子列表，删除上下文，然后继续直到有在任何级别都没有更多的 subview 。之后，您必须以相同的方式设置对新 Activity 的引用。这听起来像是一个巨大的 hack，因为它确实是，而且事情在某种程度上必然会崩溃。一个 Context 毕竟代表了你的 View 中存在的环境和状态。

针对您的情况，更好的解决方案是从单例中删除 View 引用，并仅使用它来保留给定 View 的状态/配置的表示。创建一个回调支持方法(或类似方法)，在后台膨胀 View 并在返回所述 View 之前执行必要的配置。如果您仍想保留 Activity 可能具有的所有屏幕的单个存储库，请将其作为成员添加到 Activity 类中，以便与 Activity 一起收集。

作为旁注，您的情况建议您应该使用单个 Activity ，然后只需交换由“屏幕”组成的“主屏幕”，或者根据情况在屏幕之间切换。这会更有意义，风险也会更小。

最后，引用我自己:Remember the first rule of the android fight club