正确的onDestroy（）/如何避免内存泄漏

Question

我读了很多关于如何避免Android内存泄漏的文章，但我仍然不太确定我是否做对了。

1. 我的应用程序由一个活动组成

问题1：这够了吗？

让我困惑的是，你可以在网上找到一个经典的“不去”的例子(http://www.curious-creature.org/2008/12/18/avoid-memory-leaks-on-android/):

@Override
protected void onCreate(Bundle state) {
  super.onCreate(state);

  TextView label = new TextView(this);
  label.setText("Leaks are bad");

  setContentView(label);
}

我认为，一旦创建完成，标签就超出了范围，并且是GCed。

Q2：这怎么会造成内存泄漏？

我的活动基本上是这样的：

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

    /* Statics */
    AssetUtils.initIndex(this);
    BitmapLoader.startInstance(this);

    /* frame */
    ViewGroup frame = (ViewGroup) getLayoutInflater().inflate(R.layout.frame, null);
    this.setContentView(frame);

    /* create controller */
    Controller controller = new Controller(frame, getLayoutInflater());

    /* START */
    controller.start();
}

@Override
public void onDestroy() {
    super.onStop();

    /* Statics */
    AssetUtils.destroyInstance();
    BitmapLoader.destroyInstance();
}

在控制器内部，我偶尔会使用查看#getContext（）检索上下文，将其传递给手动创建的查看等。它从不静态存储在某个地方，只存储在类的成员变量中，这些变量都返回到控制器。

问题3：我是否忽略了什么？

Answer 1

Q1。你断章取义了（无意开玩笑：）

如果您看到原始文章，泄漏实际上发生在引入位图字段的下一个代码片段中。Roman然后清楚地解释了它泄漏的原因。您展示的代码不会泄漏。

http://www.curious-creature.org/2008/12/18/avoid-memory-leaks-on-android/

问题2：只有在没有其他选择的情况下才使用活动上下文，并且绝不允许在范围大于其引用的活动范围的内容中引用活动上下文。据我所知，您显示的代码没有泄漏，因为没有任何内容引用的作用域大于您的活动。你怀疑是吗？

当使用位图、静态引用或非静态引用时，我习惯于从onPause（）中的视图中解除位图的绑定（请记住，onDestroy（）不能保证，但它有点无关紧要，就像你正在被破坏一样，你的进程被终止了，所以GC不成问题）。链接的文章还解释了如何做到这一点。我已将其作为处理位图的任何活动的模板模式。

[编辑]

对不起，我刚查过。本文没有说明如何解除绑定。以下是我使用的模式：

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

        unbindDrawables(findViewById(R.id.mainLayout));
        System.gc();
}


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

        unbindDrawables(findViewById(R.id.mainLayout));
        System.gc();
}

private void unbindDrawables(View view)
{
        if (view.getBackground() != null)
        {
                view.getBackground().setCallback(null);
        }
        if (view instanceof ViewGroup && !(view instanceof AdapterView))
        {
                for (int i = 0; i < ((ViewGroup) view).getChildCount(); i++)
                {
                        unbindDrawables(((ViewGroup) view).getChildAt(i));
                }
                ((ViewGroup) view).removeAllViews();
        }
}

主布局是活动布局的根视图。

我包括onDestroy（），因为我可能会手动（）完成我的活动。

笔记呼叫系统。gc（）是一个复杂的主题，您可能希望省略它。这里有一些很好的讨论，为什么这可能不是一件好事，主要是关于性能。然而，在我看来，当一个活动被销毁时，暗示现在是执行GC的好时机不会有什么坏处。不必要地调用它的低效性将在销毁活动的日常开支中消失。