17370845950

leakcanary 检测到 `search` fragment 存在严重内存泄漏，根源在于 `ondestroyview()` 中未及时清理视图引用（如 `binding`、`recyclerview.adapter`）和后台任务，导致 `cardsliderviewpager` 等组件及其持有链长期驻留内存。

该 LeakCanary 报告清晰地揭示了一个典型的 Fragment 视图生命周期管理不当引发的内存泄漏：泄漏追踪链最终指向 mwonyaa.Fragments.Search，其 onDestroyView() 回调已被触发（LeakCanary 明确标注 “received Fragment#onDestroyView() callback”），但该 Fragment 的视图（FrameLayout）、父容器（SwipeRefreshLayout → RecyclerView → ConstraintLayout → CardSliderViewPager）及内部持有的 SlidingTask 定时器任务仍未被释放。关键线索包括：

• View.mAttachInfo is null (view detached)：视图已从 Window 分离，但对象仍被强引用；
• mContext instance of ...RootActivity with mDestroyed = false：Activity 尚未销毁，但 Fragment 视图已解绑，此时若 Fragment 仍持有视图引用，就会阻止整个视图树 GC；
• CardSliderViewPager$SlidingTask.this$0 强引用宿主 Fragment，而该 Task 又被 Timer 的 TaskQueue 持有 —— 这是典型的「内部类 + 定时器」泄漏模式。

✅ 正确修复方案

核心原则：在 onDestroyView() 中彻底切断 Fragment 对所有 UI 组件和异步任务的强引用，尤其注意以下三类资源：

1. 清理 ViewBinding / Layout 引用

务必将 binding 设为 null，否则 binding.root 及其整个视图树（含 RecyclerView、ViewPager、ExoPlayerView 等）将持续被持有。

private var _binding: FragmentSearchBinding? = null
private val binding get() = _binding!!

override fun onCreateView(
    inflater: LayoutInflater,
    container: ViewGroup?,
    savedInstanceState: Bundle?
): View {
    _binding = FragmentSearchBinding.inflate(inflater, container, false)
    return binding.root
}

override fun onDestroyView() {
    // ✅ 关键：置空 binding，解除对视图树的强引用
    _binding = null
    super.onDestroyView()
}

⚠️ 注意：使用 _binding（私有可变属性）+ binding（只读委托）模式，避免在 onDestroyView() 后误用已释放的 binding。

2. 解绑 RecyclerView Adapter 并清空数据源

Adapter 若持有 Activity/Fragment 引用（如通过 context 或 listener），或自身未清理监听器，也会导致泄漏：

override fun onDestroyView() {
    // ✅ 清空 Adapter 并解除绑定
    binding.mainRecycler.adapter = null
    // ✅ 若使用 ListAdapter，建议同时 submitList(null)
    (binding.mainRecycler.adapter as? ListAdapter<*, *>?)?.submitList(null)

    _binding = null
    super.onDestroyView()
}

3. 取消定时器、协程、RxJava 订阅等后台任务

CardSliderViewPager$SlidingTask 是泄漏源头之一，说明该 ViewPager 使用了 Timer 轮播逻辑。必须在 onDestroyView() 中显式取消：

private var slidingTimer: Timer? = null
private var slidingTask: TimerTask? = null

// 在启动轮播时：
slidingTimer = Timer()
slidingTask = object : TimerTask() {
    override fun run() { /* ... */ }
}
slidingTimer?.schedule(slidingTask, 0, 3000)

// ✅ onDestroyView 中必须取消：
override fun onDestroyView() {
    slidingTask?.cancel()
    slidingTimer?.cancel()
    slidingTimer = null
    slidingTask = null

    binding.mainRecycler.adapter = null
    _binding = null
    super.onDestroyView()
}

? 更优实践：优先使用 Handler + removeCallbacks() 或 Kotlin 协程 Job（配合 lifecycleScope.launchWhenStarted）替代 Timer，它们天然与生命周期绑定，不易遗漏取消。

4. ExoPlayer 特别注意事项

虽然报告中未直接显示 Player 泄漏，但 CardSliderViewPager 嵌套播放器时极易因未释放 Player 实例导致泄漏：

• ✅ onDestroyView() 中调用 player.release()
• ✅ 确保 PlayerView.setPlayer(null) 已调用
• ✅ 避免在 Player.Listener 回调中隐式持有 Fragment（如使用 this@Fragment）

override fun onDestroyView() {
    // ... 其他清理 ...
    binding.playerView.player?.release()
    binding.playerView.player = null
    super.onDestroyView()
}

? 验证与预防

• 修复后重新运行 App，触发相同操作路径，观察 LeakCanary 是否不再报告 Search Fragment 泄漏；
• 在 Fragment 中启用严格模式：requireActivity().application.registerActivityLifecycleCallbacks(...) 监听 onActivitySaveInstanceState 前检查 isAdded && isResumed；
• 使用 Android Studio Profiler 的 Memory Tab 手动触发 GC 并 dump heap，搜索 Search 或 CardSliderViewPager 确认实例数归零。

遵循以上规范，不仅能解决当前泄漏，更能建立健壮的 Fragment 生命周期意识——onDestroyView() 不是终点，而是释放所有 UI 相关资源的强制截止点。