一个关于 DialogFragment 状态引发的崩溃日志

谁借莪１个温暖的怀抱￠ 2022-01-19 23:39 307阅读 0赞

## 问题复现 ##

> 最近出现了一个很奇怪的问题，问题异常日志如下：

java.lang.IllegalStateException: Can not perform this action after onSaveInstanceState
    at android.support.v4.app.FragmentManagerImpl.checkStateLoss(FragmentManager.java:2053)
    at android.support.v4.app.FragmentManagerImpl.enqueueAction(FragmentManager.java:2079)
    at android.support.v4.app.BackStackRecord.commitInternal(BackStackRecord.java:678)
    at android.support.v4.app.BackStackRecord.commit(BackStackRecord.java:632)
    at android.support.v4.app.DialogFragment.dismissInternal(DialogFragment.java:223)
    at android.support.v4.app.DialogFragment.dismiss(DialogFragment.java:190)
    复制代码

刚开始以为是 DialogFragment 调用 show 的时候，没有判断 Activity 是否已经 Finish 了。结果发现在项目代码里是做了判断的，才知道肯定没这么简单。

**FragmentManagerImpl.checkStateLoss**

private void checkStateLoss() {
         //异常抛出的地方
         if (mStateSaved) {
             throw new IllegalStateException("Can not perform this action after onSaveInstanceState");
         }
         if (mNoTransactionsBecause != null) {
             throw new IllegalStateException("Can not perform this action inside of " + mNoTransactionsBecause);
         }
    }
    复制代码

**FragmentManagerImpl. enqueueAction**

/**
     * Adds an action to the queue of pending actions.
     *
     * @param action the action to add
     * @param allowStateLoss whether to allow loss of state information
     * @throws IllegalStateException if the activity has been destroyed
     */
    public void enqueueAction(OpGenerator action, boolean allowStateLoss) {
        //注意这里的allowStateLoss 它决定了checkStateLoss方法是否进行
        if (!allowStateLoss) {
            checkStateLoss();
        }
        synchronized (this) {
            if (mDestroyed || mHost == null) {
                throw new IllegalStateException("Activity has been destroyed");
            }
            if (mPendingActions == null) {
                mPendingActions = new ArrayList<>();
            }
            mPendingActions.add(action);
            scheduleCommit();
        }
    }
    复制代码

**BackStackRecord.commitInternal**

int commitInternal(boolean allowStateLoss) {
        if (mCommitted) throw new IllegalStateException("commit already called");
        if (FragmentManagerImpl.DEBUG) {
            Log.v(TAG, "Commit: " + this);
            LogWriter logw = new LogWriter(TAG);
            PrintWriter pw = new PrintWriter(logw);
            dump(" ", null, pw, null);
            pw.close();
        }
        mCommitted = true;
        if (mAddToBackStack) {
            mIndex = mManager.allocBackStackIndex(this);
        } else {
            mIndex = -1;
        }
        //重点
        mManager.enqueueAction(this, allowStateLoss);
        return mIndex;
    }
    复制代码

**BackStackRecord.commit**

public int commit() {
        //这里传入的就是allowStateLoss参数
        return commitInternal(false);
    }
    复制代码

**DialogFragment.show**

public void show(FragmentManager manager, String tag) {
        mDismissed = false;
        mShownByMe = true;
        FragmentTransaction ft = manager.beginTransaction();
        ft.add(this, tag);
        //没想到的时候DialogFragment的show方法会调用commit() 导火索
        ft.commit();
    }
    复制代码

> 目前来看是因为 DialogFragment 的 show 方法导致强行检查 Fragment 的状态，而恰好在检查的时候状态已经被保存导致 FragmentManager 的标记位 mStateSaved = true 。

## 情景再现 ##

当用户在进入首页模块时直接按 Home 键或者遇到内存不足等特殊情况下触发了onSaveInstanceState 导致 FragmentManager 进行“智能”保存当前 Activity 中 Fragment 的状态，因为 DialogFragment (广告弹窗)需要先从服务器拉取数据，所以之后在 show 的时候去 checkStateLoss 时触发了异常。

## 解决方案 ##

*  1、将广告弹窗实现改为纯 `Dialog` ，不再使用 `DialogFragment` ；
 *  2如果自己的项目里面不需要对 `Fragment` 的相关状态进行保存和维护，可以在相关 `Activity` 中复写 `onSaveInstance` 不进行 `super` 回调即可（**不建议使用此方案**）；
 *  3、可重写 `DialogFragment` 中的 `show` 方法，并使用 `commitAllowStateLoss` 提交，如下代码（**推荐使用此方案**）。

// 方案3代码示例
        override fun show(manager: FragmentManager?, tag: String?) {
            // super.show(manager, tag)
            val ft = manager?.beginTransaction()
            ft?.add(this, tag)
            ft?.commitAllowingStateLoss()
        }
    复制代码