泄漏金丝雀，Recyclerview 泄漏 mAdapter

Question

我决定是时候学习如何使用 Leak Canary 来检测我的应用程序中的泄漏了，并且像往常一样，我尝试在我的项目中实施它以真正了解如何使用该工具。实现它很容易，困难的部分是阅读该工具向我抛出的内容。 我有一个滚动视图，当我上下滚动时，它似乎在内存管理器中累积内存（即使它没有加载任何新数据），所以我认为这是一个很好的候选对象来跟踪泄漏，结果如下：

看起来 v7.widget.RecyclerView 正在泄漏适配器，而不是我的应用程序。但这不可能是对的……对吧？

这是适配器和使用它的类的代码： https://gist.github.com/feresr/a53c7b68145d6414c40ec70b3b842f1e

我开始为这个问题悬赏，因为它在两年后在一个完全不同的应用程序上重新出现

Answer 1

如果适配器的寿命比RecyclerView 长，您必须清除onDestroyView 中的适配器引用：

@Override
public void onDestroyView() {
    recyclerView.setAdapter(null);
    super.onDestroyView();
}

否则适配器将持有对应该已经耗尽内存的RecyclerView 的引用。

如果屏幕涉及到过渡动画，你实际上必须更进一步，只有在视图变得分离时才清除适配器：

@Override
public void onDestroyView() {
    recyclerView.addOnAttachStateChangeListener(new View.OnAttachStateChangeListener() {
        @Override
        public void onViewAttachedToWindow(View v) {
            // no-op
        }

        @Override
        public void onViewDetachedFromWindow(View v) {
            recyclerView.setAdapter(null);
        }
    });
    super.onDestroyView();
}

Answer 2

我可以通过覆盖 RecyclerView 来解决这个问题。这是因为 RecyclerView 永远不会从 AdapterDataObservable 注销自己。

@Override protected void onDetachedFromWindow() {
    super.onDetachedFromWindow();
    if (getAdapter() != null) {
        setAdapter(null);
    }
}

Answer 3

首先，我引用的是this file。

看起来 v7.widget.RecyclerView 正在泄漏适配器，而不是我的应用程序。但这不可能是对的……对吧？

实际上是您的适配器泄漏了RecyclerView（跟踪图和 LeakCanary 活动的标题清楚地表明了这一点）。但是，我不确定它是“父” RecyclerView 还是 HourlyViewHolder 中的嵌套视图，或者两者兼而有之。 我认为罪魁祸首是你的 ViewHolders。通过使它们成为非静态内部类，您可以显式地为它们提供对封闭适配器类的引用，这几乎直接将适配器与回收的视图耦合在一起，因为您的持有者中每个 itemView 的父级都是 RecyclerView 本身。

我解决这个问题的第一个建议是通过将 ViewHolders 和 Adapter 设为 static 内部类来解耦它们。这样他们就没有对适配器的引用，因此他们将无法访问您的 context 字段，这也是一件好事，因为应该谨慎地传递和存储上下文引用（也避免大内存泄漏）。当您只需要上下文来获取字符串时，请在其他地方进行，例如在适配器构造函数中，但不要将上下文存储为成员。最后，DayForecastAdapter 似乎也很危险：你将它的同一个实例传递给每个HourlyViewHolder，这似乎是一个错误。

我认为修复设计和解耦这些类应该可以消除这种内存泄漏

Answer 4

我无法打开您的图像并查看实际泄漏，但如果您在 Fragment 中为您的 RecyclerView 定义一个局部变量并设置片段的 retainInstanceState true 它可能会导致旋转泄漏。

当使用Fragment 和retainInstance 时，您应该清除onDestroyView 中的所有用户界面引用

@Override
public void onDestroyView() {
     yourRecyclerView = null;
     super.onDestroyView();
}

您可以在此链接中找到详细信息： Retained Fragments with UI and memory leaks

Answer 5

所以，这可能是一个有点矫枉过正的解决方案，但我们的团队已经厌倦了不得不担心每个 RecyclerView 适配器泄漏。

这是一个抽象的 RecyclerView.Adapter 子类，它一劳永逸地解决泄漏问题。

abstract class AbstractRecyclerViewAdapter<VH : RecyclerView.ViewHolder>(fragment: Fragment) :
    RecyclerView.Adapter<VH>() {
    private val fragmentRef = WeakReference(fragment)

    override fun onAttachedToRecyclerView(recyclerView: RecyclerView) {
        super.onAttachedToRecyclerView(recyclerView)
        setupLifecycleObserver(recyclerView)
    }

    private fun setupLifecycleObserver(recyclerView: RecyclerView) {
        val fragment = fragmentRef.get() ?: return
        val weakThis = WeakReference(this)
        val weakRecyclerView = WeakReference(recyclerView)

        // Observe the fragment's lifecycle events in order
        // to set the Recyclerview's Adapter when the Fragment is resumed
        // and unset it when the Fragment is destroyed.
        fragment.viewLifecycleOwner.lifecycle.addObserver(object : LifecycleEventObserver {
            override fun onStateChanged(source: LifecycleOwner, event: Lifecycle.Event) {
                val actualRecyclerView = weakRecyclerView.get() ?: return
                when (event.targetState) {
                    Lifecycle.State.DESTROYED -> actualRecyclerView.adapter = null
                    Lifecycle.State.RESUMED -> {
                        val self = weakThis.get() ?: return
                        if (actualRecyclerView.adapter != self) {
                            actualRecyclerView.adapter = self
                        }
                    }
                    else -> {
                    }
                }
            }
        })
    }
}

基本上，Adapter 会观察包含它的 Fragment 的生命周期事件，并负责将自身设置和取消设置为 RecyclerView 的适配器。

您只需在自己的适配器中继承AbstractRecyclerViewAdapter，并在构建它们时提供容器片段。

用法：

你的适配器

class MyAdapter(fragment: Fragment): AbstractRecyclerViewAdapter<MyViewHolder>(fragment) {
    // Your usual adapter code...
}

你的片段

class MyFragment : Fragment {
    // Instantiate with the fragment.
    private val myAdapter = MyAdapter(this)

    override fun onViewCreated(view: View, savedInstanceState: Bundle?) {
        super.onViewCreated(view, savedInstanceState)
        // Set the RecyclerView adapter as you would normally.
        myRecyclerView.adapter = myAdapter
    }
}