一个常见Android崩溃问题的分析

最新推荐文章于 2025-10-23 13:30:34 发布

原创

最新推荐文章于 2025-10-23 13:30:34 发布 · 1.8k 阅读

1 ·

CC 4.0 BY-SA版权

文章标签：

#onSaveInstanceState #FragmentTransaction #commitAllowingStateLoss

本文探讨了一个常见的Android错误：IllegalStateException：在onSaveInstanceState后执行操作。分析了在AsyncTask回调中调用FragmentTransaction.commit的不安全性，解释了为何应使用commitAllowingStateLoss，并指出在onSaveInstanceState后调用commit可能导致状态丢失。通过深入源码，揭示了Activity的isDestroyed方法判断的不准确性，并提出了解决方案。

这个错误大家应该都有遇到过：IllegalStateException: Can not perform this action after onSaveInstanceState。最近又遇到这个问题，就想要梳理一下其中的一些细节。

我的具体使用场景是在一个AsyncTask的回调onPostExecute中调用了FragmentTransaction的commit方法，并且在调用commit方法之前调用了Activity的isDestroyed方法进行保护。代码如下：

这里的本意是要判断Activity是否已经在销毁了，是的话就不再调用commit方法。但是这里的代码有两个问题，首先在AsyncTask的回调中调用commit是不安全的，应该要调用commitAllowingStateLoss，其次，这里虽然通过判断Activity是否destroy做了保护，但这个保护也是不准确的。

首先来看第一个问题：为什么在AsyncTask的回调中调用commit是不安全的？

首先我们要找到这个方法实现的地方，FragmentTransaction的commit方法实际上是由BackStackRecord实现的，我们一步步查看内部的调用：commit -> commitInternal -> mManager.enqueueAction -> checkStateLoss。