Fragment already added問題解決

問題

當快速點擊切換不同的Fragment的時候部分手機的app竟然掛了，

報出了下面的錯誤 Fragment already added

java.lang.IllegalStateException: Fragment already added: xxxxFragment

上網找了很多，大致得到的原因是當快速雙擊調用FragmentTransaction.add()方法添加fragmentA，而fragmentA不是每次單獨生成的，就會引起這個異常。

上面的內容是網上的解釋，不過我認為是因為同一個Fragment被add兩次導致的錯誤，但是奇怪的是我在加載這個Fragment的時候已經做了是否add的判斷，為什么還會有這個問題呢。

private void addFragment(FragmentManager fm, Fragment fragment ) {
? ? ? ? Log.i( "addFragment", "### " + fragment.getId() + " ?" + fragment.isAdded() + " ?" + fragment.isHidden() );

? ? ? ? if (!fragment.isAdded() ) {
? ? ? ? ? ? FragmentTransaction ft = fm.beginTransaction();
? ? ? ? ? ? fm.executePendingTransactions();
? ? ? ? ? ? ft.add( R.id.main_content, fragment );
? ? ? ? ? ? ft.commitAllowingStateLoss();
? ? ? ? }

通過打印結果發現，第一次切換到FragmentA的時候，FragmentA明明已經add了，但是isAdded()依然顯示false

addFragment: ###1 2131755357 ?false true
addFragment: ###2 2131755357 ?false true
addFragment: ###3 2131755357 ?false true
addFragment: ###4 2131755357 ?false true

第二次切換到相同FragmentA的時候isAdded()就顯示為true了

addFragment: ###1 2131755357 ?true true
addFragment: ###2 2131755357 ?true true
addFragment: ###3 2131755357 ?true true
addFragment: ###4 2131755357 ?true true

在不斷快速切換不同的Fragment的時候，isAdded()偶爾會顯示false，就因為isAdded()顯示了false，那么 ft.add( R.id.main_content, fragment )就會再次執行一次，就會報錯，說明通過isAdded()這個方法判斷Fragment是否被add可能并不準確。

方法

解決方法就是每次add的時候加上一個tag，然后不僅要通過isAdded()判斷Fragment是否add，還要通過FragmentManager.findFragmentByTag(tag)獲取Fragment，然后判斷此Fragment是否為null。

?case R.id.home_tab_a:
? ? ? ? hideAllFragment( fm );
? ? ? ? addFragment( fm, fragmentA, "A" );
? ? ? ? showFragment( fm, fragmentA );
? break;

?case R.id.home_tab_b:
? ? ? ? hideAllFragment( fm );
? ? ? ? addFragment( fm, fragmentB, "B" );
? ? ? ? showFragment( fm, fragmentB );
? break;

?case R.id.home_tab_c:
? ? ? ? hideAllFragment( fm );
? ? ? ? addFragment( fm, fragmentC, "C" );
? ? ? ? showFragment( fm, fragmentC );
? break;

影藏所有的Fragment

?private void hideAllFragment(FragmentManager fm) {
? ? ? ? FragmentTransaction ft = fm.beginTransaction();
? ? ? ? if (!shijianFragment.isHidden())
? ? ? ? ? ? ft.hide( fragmentA );

? ? ? ? if (!riliFragment.isHidden())
? ? ? ? ? ? ft.hide( fragmentB );

? ? ? ? if (!gongjuFragment.isHidden()) {
? ? ? ? ? ? ft.hide( fragmentC );
? ? ? ? }
? ? ? ? ft.commitAllowingStateLoss();
? ? }

通過isAdded()判斷Fragment是否add，同時通過tag獲取Fragment，判斷Fragment是否為空，雙重判斷

? ?private void addFragment(FragmentManager fm, Fragment fragment, String tag) {
? ? ? ? if (!fragment.isAdded()&&null == fm.findFragmentByTag( tag )) {
? ? ? ? ? ? FragmentTransaction ft = fm.beginTransaction();
? ? ? ? ? ? fm.executePendingTransactions();
? ? ? ? ? ? ft.add( R.id.main_content, fragment, tag );
? ? ? ? ? ? ft.commitAllowingStateLoss();
? ? ? ? }

? ? }

顯示Fragment

?private void showFragment(FragmentManager fm, Fragment fragment) {
? ? ? ? FragmentTransaction ft = fm.beginTransaction();
? ? ? ? ft.show( fragment );
? ? ? ? ft.commitAllowingStateLoss();
? ? }

java.lang.IllegalStateException: Fragment already added 異常處理

10-15 15:52:00.094 3808 3808 E AndroidRuntime: FATAL EXCEPTION: main
10-15 15:52:00.094 3808 3808 E AndroidRuntime: Process: com.xxxx.xxx, PID: 3808
10-15 15:52:00.094 3808 3808 E AndroidRuntime: java.lang.IllegalStateException: Fragment already added
10-15 15:52:00.094 3808 3808 E AndroidRuntime: at androidx.fragment.app.Fragment.setInitialSavedState(Fragment.java:679)
10-15 15:52:00.094 3808 3808 E AndroidRuntime: at androidx.viewpager2.adapter.FragmentStateAdapter.ensureFragment(FragmentStateAdapter.java:269)
10-15 15:52:00.094 3808 3808 E AndroidRuntime: at androidx.viewpager2.adapter.FragmentStateAdapter.onBindViewHolder(FragmentStateAdapter.java:175)
10-15 15:52:00.094 3808 3808 E AndroidRuntime: at androidx.viewpager2.adapter.FragmentStateAdapter.onBindViewHolder(FragmentStateAdapter.java:67)
10-15 15:52:00.094 3808 3808 E AndroidRuntime: at androidx.recyclerview.widget.RecyclerViewA d a p t e r . o n B i n d V i e w H o l d e r ( R e c y c l e r V i e w . j a v a : 7065 ) 10 ? 1515 : 52 : 00.09438083808 E A n d r o i d R u n t i m e : a t a n d r o i d x . r e c y c l e r v i e w . w i d g e t . R e c y c l e r V i e w Adapter.onBindViewHolder(RecyclerView.java:7065) 10-15 15:52:00.094 3808 3808 E AndroidRuntime: at androidx.recyclerview.widget.RecyclerViewAdapter.onBindViewHolder(RecyclerView.java:7065)10?1515:52:00.09438083808EAndroidRuntime:atandroidx.recyclerview.widget.RecyclerViewAdapter.bindViewHolder(RecyclerView.java:7107)
10-15 15:52:00.094 3808 3808 E AndroidRuntime: at androidx.recyclerview.widget.RecyclerViewR e c y c l e r . t r y B i n d V i e w H o l d e r B y D e a d l i n e ( R e c y c l e r V i e w . j a v a : 6012 ) 10 ? 1515 : 52 : 00.09438083808 E A n d r o i d R u n t i m e : a t a n d r o i d x . r e c y c l e r v i e w . w i d g e t . R e c y c l e r V i e w Recycler.tryBindViewHolderByDeadline(RecyclerView.java:6012) 10-15 15:52:00.094 3808 3808 E AndroidRuntime: at androidx.recyclerview.widget.RecyclerViewRecycler.tryBindViewHolderByDeadline(RecyclerView.java:6012)10?1515:52:00.09438083808EAndroidRuntime:atandroidx.recyclerview.widget.RecyclerViewRecycler.tryGetViewHolderForPositionByDeadline(RecyclerView.java:6279)
10-15 15:52:00.094 3808 3808 E AndroidRuntime: at androidx.recyclerview.widget.GapWorker.prefetchPositionWithDeadline(GapWorker.java:288)
10-15 15:52:00.094 3808 3808 E AndroidRuntime: at androidx.recyclerview.widget.GapWorker.flushTaskWithDeadline(GapWorker.java:345)
10-15 15:52:00.094 3808 3808 E AndroidRuntime: at androidx.recyclerview.widget.GapWorker.flushTasksWithDeadline(GapWorker.java:361)
10-15 15:52:00.094 3808 3808 E AndroidRuntime: at androidx.recyclerview.widget.GapWorker.prefetch(GapWorker.java:368)
10-15 15:52:00.094 3808 3808 E AndroidRuntime: at androidx.recyclerview.widget.GapWorker.run(GapWorker.java:399)
10-15 15:52:00.094 3808 3808 E AndroidRuntime: at android.os.Handler.handleCallback(Handler.java:755)
10-15 15:52:00.094 3808 3808 E AndroidRuntime: at android.os.Handler.dispatchMessage(Handler.java:95)
10-15 15:52:00.094 3808 3808 E AndroidRuntime: at android.os.Looper.loop(Looper.java:154)
10-15 15:52:00.094 3808 3808 E AndroidRuntime: at android.app.ActivityThread.main(ActivityThread.java:6141)
10-15 15:52:00.094 3808 3808 E AndroidRuntime: at java.lang.reflect.Method.invoke(Native Method)
10-15 15:52:00.094 3808 3808 E AndroidRuntime: at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:913)
10-15 15:52:00.094 3808 3808 E AndroidRuntime: at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:803)

報錯AndroidRuntime: at androidx.fragment.app.Fragment.setInitialSavedState(Fragment.java:679)

這個錯誤是我從 fragment A 切換到 fragment B ,fragment B再切換到 fragment C 中，再切換到 fragment B中， fragment B 里面 使用了ViewPager2 + FragmentStateAdapter + RadioButton切換主題，問題就出在我使用的ViewPager2 + FragmentStateAdapter這里。

每次我在多個fragmentA,B,C之間多次切換后，切換到fragment B，點擊 radioButton 通知ViewPager2 切換 fragment的主題時,使用方法viewPager2.setCurrentItem(index);，就報上面的錯誤。

我在想，是否是滑動效果的問題，于是 用viewPager2.setCurrentItem(index,false);然后，由于viewpager2 里面的fragment 里面有recycleviewer ,把viewpager2 的寬高 固定 ，問題解決。

小結一下

1、viewPager2.setCurrentItem(index)改成viewPager2.setCurrentItem(index,false);

2、viewPager2 在布局文件里面把寬高 固定。