从其他Fragment返回后ViewPager调用getCurrentItem触发空指针异常

问题原因 #

错误出在update Runnable中调用carousal.currentItem时carousal为空，本质是：

• 当Fragment从后台返回时，旧的定时任务（TimerTask+Handler回调）未被彻底清理，仍在执行
• Fragment视图销毁后，carousal（ViewPager）实例已被回收，但遗留的Runnable还在尝试调用它的方法，触发空指针
• 每次onResume重新创建Timer和Runnable，但旧任务未终止，导致多任务并行，引用过期的ViewPager实例

修复方案1：彻底清理定时任务+空指针防护 #

修改代码如下：

private var timer: Timer? = null
private val handler = Handler(Looper.getMainLooper())
private var updateRunnable: Runnable? = null

private fun initCarousal() {
    // 初始化前先检查ViewPager是否存在
    carousal ?: return

    carousal.adapter = TransactCarousalAdapter {
        when (it) {
            0 -> Case1()
            1 -> Case2()
            2 -> case3()
        }
    }
    dots_indicator.setViewPager(carousal)

    updateRunnable = Runnable {
        // 执行轮播操作前强制判空，避免空指针
        carousal?.let { vp ->
            var currentPage = vp.currentItem
            currentPage = if (currentPage == 1) 0 else currentPage + 1
            vp.setCurrentItem(currentPage, true)
        }
    }

    timer = Timer().apply {
        schedule(object : TimerTask() {
            override fun run() {
                updateRunnable?.let { handler.post(it) }
            }
        }, 7000, 7000)
    }
}

override fun onResume() {
    super.onResume()
    initCarousal()
}

override fun onPause() {
    super.onPause()
    // 清理Timer和未执行的Handler回调
    timer?.cancel()
    timer = null
    updateRunnable?.let { handler.removeCallbacks(it) }
    updateRunnable = null
}

override fun onDestroyView() {
    super.onDestroyView()
    // 视图销毁时再次清理，确保无遗留任务
    timer?.cancel()
    timer = null
    updateRunnable?.let { handler.removeCallbacks(it) }
    updateRunnable = null
}

修复方案2：使用Coroutine替代Timer+Handler（更推荐） #

利用Kotlin协程的生命周期感知能力，自动管理任务，避免手动清理的繁琐：

private var carouselJob: Job? = null

private fun initCarousal() {
    carousal ?: return

    carousal.adapter = TransactCarousalAdapter {
        when (it) {
            0 -> Case1()
            1 -> Case2()
            2 -> case3()
        }
    }
    dots_indicator.setViewPager(carousal)

    // 使用viewLifecycleOwner的协程作用域，跟随Fragment视图生命周期自动取消
    carouselJob = viewLifecycleOwner.lifecycleScope.launch {
        while (true) {
            delay(7000)
            carousal?.let { vp ->
                var currentPage = vp.currentItem
                currentPage = if (currentPage == 1) 0 else currentPage + 1
                vp.setCurrentItem(currentPage, true)
            }
        }
    }
}

override fun onResume() {
    super.onResume()
    initCarousal()
}

override fun onPause() {
    super.onPause()
    carouselJob?.cancel()
    carouselJob = null
}

关键注意点 #

• 永远不要在未判空的情况下直接调用ViewPager的方法，尤其是在异步任务中
• Fragment视图销毁后（onDestroyView），所有视图引用都会变为null，必须清理所有可能触发视图操作的异步任务
• 协程的viewLifecycleOwner.lifecycleScope会自动在Fragment视图销毁时取消所有任务，是Android开发中处理异步生命周期的最优选择

内容的提问来源于stack exchange，提问作者Anees Ani