开始我的Android博客之旅

今天在g买卖的客户端查线上问题日志的时候发现一个bug

java.lang.IllegalStateException: Can not perform this action after onSaveInstanceState
at android.support.v4.app.FragmentManagerImpl.checkStateLoss(FragmentManager.java:1365)
at android.support.v4.app.FragmentManagerImpl.enqueueAction(FragmentManager.java:1383)
at android.support.v4.app.BackStackRecord.commitInternal(BackStackRecord.java:636)
at android.support.v4.app.BackStackRecord.commit(BackStackRecord.java:615)
at com.snda.mhh.business.home.HomeActivity.showFragment(HomeActivity.java:269)
at com.snda.mhh.business.home.HomeActivity.showTab(HomeActivity.java:304)
at com.snda.mhh.business.home.HomeActivity.access$400(HomeActivity.java:53)
at com.snda.mhh.business.home.HomeActivity$3.onClick(HomeActivity.java:259)
at android.view.View.performClick(View.java:4475)
at android.view.View$PerformClick.run(View.java:18786)
at android.os.Handler.handleCallback(Handler.java:730)
at android.os.Handler.dispatchMessage(Handler.java:92)
at android.os.Looper.loop(Looper.java:137)
at android.app.ActivityThread.main(ActivityThread.java:5493)
at java.lang.reflect.Method.invokeNative(Native Method)
at java.lang.reflect.Method.invoke(Method.java:525)
at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:1209)
at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1025)
at dalvik.system.NativeStart.main(Native Method)

是在首页下面的切换tab的时候报replace  fragment的时候出错，不能在onSaveInstanceState(这个方法是系统未经你的同意回收你的activity的时候调用) 之后调用，也就是说activity已经被系统回收了，但是我们不应该在被回收后还要继续执行这个操作，解决办法 在此之前判断了 activity.isFinishing() , 同时切换tab的时候 fragment的commit 替换成了 commitallowingStateLoss

--------------------------------------------

下面是查阅的资料

用add(), remove(), replace()方法，把所有需要的变化加进去，然后调用commit()方法，将这些变化应用。
在commit()方法之前，你可以调用addToBackStack()，把这个transaction加入back stack中去，这个back stack是由activity管理的，当用户按返回键时，就会回到上一个fragment的状态。
你只能在activity存储它的状态（当用户要离开activity时）之前调用commit()，如果在存储状态之后调用commit()，将会抛出一个异常。
这是因为当activity再次被恢复时commit之后的状态将丢失。如果丢失也没关系，那么使用commitAllowingStateLoss()方法。

问什么在存储状态之后调用commit会报异常？我们查看Android源码发现FragmentManager和FragmentTransaction是一个虚类
那他们在activity中的实例化代码是如何处理的呢？
首先是getSupportFragmentManager的方法


[java] view plain copy
 print?
/** 


     * Return the FragmentManager for interacting with fragments associated 


     * with this activity. 


     */  


    public FragmentManager getSupportFragmentManager() {  


        return mFragments;  


    }  




查找到mFragments。
final FragmentManagerImpl mFragments = new FragmentManagerImpl();
我们发现FragmentManagerImpl是继承于FragmentManager的一个实体类


[java] view plain copy
 print?
/** 


 * Container for fragments associated with an activity. 


 */  


final class FragmentManagerImpl extends FragmentManager {  


      


    ........  


  


  


    @Override  


    public FragmentTransaction beginTransaction() {  


        return new BackStackRecord(this);  


    }  


  


  


    ........  


  


  


    }  




为了简便我们删除了一些不要的代码只留下关键的方法。
通过这段代码，我们可以查看到beginTransaction方法实际返回的是一个继承于FragmentTransaction的BackStackRecord类
我们来查看BackStackRecord的代码，查看他的用法


[java] view plain copy
 print?
/** 


 * @hide Entry of an operation on the fragment back stack. 


 */  


final class BackStackRecord extends FragmentTransaction implements  


        FragmentManager.BackStackEntry, Runnable {  


  


  


    ..........  


    public int commit() {  


        return commitInternal(false);  


    }  


  


  


    public int commitAllowingStateLoss() {  


        return commitInternal(true);  


    }  


  


  


    int commitInternal(boolean allowStateLoss) {  


        if (mCommitted) throw new IllegalStateException("commit already called");  


        if (FragmentManagerImpl.DEBUG) Log.v(TAG, "Commit: " + this);  


        mCommitted = true;  


        if (mAddToBackStack) {  


            mIndex = mManager.allocBackStackIndex(this);  


        } else {  


            mIndex = -1;  


        }  


        mManager.enqueueAction(this, allowStateLoss);  


        return mIndex;  


    }  


    ..........  


  


  


}  




绕了大半天，终于找到commit方法和commitAllowingStateLoss方法，他们都同时调用了commitInternal方法，只是传的参数略有不同，一个是true，一个是false。我们发现在执行这个方法之前会首先对mCommitted进行判断，根据代码语义我们可以知道mCommitted就是是否已经commit的意思
最后，commitInternal调用了mManager.enqueueAction的方法。让我们回到FragmentManager，看这个方法是如何操作的。我们找到这个方法。


[java] view plain copy
 print?
/** 


 * @hide Entry of an operation on the fragment back stack. 


 */  


final class BackStackRecord extends FragmentTransaction implements  


        FragmentManager.BackStackEntry, Runnable {  


  


  


    ..........  


    public int commit() {  


        return commitInternal(false);  


    }  


  


  


    public int commitAllowingStateLoss() {  


        return commitInternal(true);  


    }  


  


  


    int commitInternal(boolean allowStateLoss) {  


        if (mCommitted) throw new IllegalStateException("commit already called");  


        if (FragmentManagerImpl.DEBUG) Log.v(TAG, "Commit: " + this);  


        mCommitted = true;  


        if (mAddToBackStack) {  


            mIndex = mManager.allocBackStackIndex(this);  


        } else {  


            mIndex = -1;  


        }  


        mManager.enqueueAction(this, allowStateLoss);  


        return mIndex;  


    }  


    ..........  


  


  


}  




经分析后，我们可以发现，此方法在对 commit和commitAllowingStateLoss的传参进行判断后，将任务扔进activity的线程队列中。那这个两个方法区别就在传参判断后的处理方法checkStateLoss，那接下来，让我们查看一下checkStateLoss方法，看对参数进行判断后，做了什么样的处理。


[java] view plain copy
 print?
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);  


        }  


    }  




ok，到这里，真相总算大明，当使用commit方法时，系统将进行状态判断，如果状态（mStateSaved）已经保存，将发生"Can not perform this action after onSaveInstanceState"错误。
如果mNoTransactionsBecause已经存在，将发生"Can not perform this action inside of " + mNoTransactionsBecause错误。