Fragment onCreateView et onActivityCreated appelés deux fois

101

Je développe une application utilisant Android 4.0 ICS et des fragments.

Considérez cet exemple modifié de l'exemple d'application de démonstration de l'API ICS 4.0.3 (niveau d'API 15):

public class FragmentTabs extends Activity {

private static final String TAG = FragmentTabs.class.getSimpleName();

@Override
protected void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);

    final ActionBar bar = getActionBar();
    bar.setNavigationMode(ActionBar.NAVIGATION_MODE_TABS);
    bar.setDisplayOptions(0, ActionBar.DISPLAY_SHOW_TITLE);

    bar.addTab(bar.newTab()
            .setText("Simple")
            .setTabListener(new TabListener<SimpleFragment>(
                    this, "mysimple", SimpleFragment.class)));

    if (savedInstanceState != null) {
        bar.setSelectedNavigationItem(savedInstanceState.getInt("tab", 0));
        Log.d(TAG, "FragmentTabs.onCreate tab: " + savedInstanceState.getInt("tab"));
        Log.d(TAG, "FragmentTabs.onCreate number: " + savedInstanceState.getInt("number"));
    }

}

@Override
protected void onSaveInstanceState(Bundle outState) {
    super.onSaveInstanceState(outState);
    outState.putInt("tab", getActionBar().getSelectedNavigationIndex());
}

public static class TabListener<T extends Fragment> implements ActionBar.TabListener {
    private final Activity mActivity;
    private final String mTag;
    private final Class<T> mClass;
    private final Bundle mArgs;
    private Fragment mFragment;

    public TabListener(Activity activity, String tag, Class<T> clz) {
        this(activity, tag, clz, null);
    }

    public TabListener(Activity activity, String tag, Class<T> clz, Bundle args) {
        mActivity = activity;
        mTag = tag;
        mClass = clz;
        mArgs = args;

        // Check to see if we already have a fragment for this tab, probably
        // from a previously saved state.  If so, deactivate it, because our
        // initial state is that a tab isn't shown.
        mFragment = mActivity.getFragmentManager().findFragmentByTag(mTag);
        if (mFragment != null && !mFragment.isDetached()) {
            Log.d(TAG, "constructor: detaching fragment " + mTag);
            FragmentTransaction ft = mActivity.getFragmentManager().beginTransaction();
            ft.detach(mFragment);
            ft.commit();
        }
    }

    public void onTabSelected(Tab tab, FragmentTransaction ft) {
        if (mFragment == null) {
            mFragment = Fragment.instantiate(mActivity, mClass.getName(), mArgs);
            Log.d(TAG, "onTabSelected adding fragment " + mTag);
            ft.add(android.R.id.content, mFragment, mTag);
        } else {
            Log.d(TAG, "onTabSelected attaching fragment " + mTag);
            ft.attach(mFragment);
        }
    }

    public void onTabUnselected(Tab tab, FragmentTransaction ft) {
        if (mFragment != null) {
            Log.d(TAG, "onTabUnselected detaching fragment " + mTag);
            ft.detach(mFragment);
        }
    }

    public void onTabReselected(Tab tab, FragmentTransaction ft) {
        Toast.makeText(mActivity, "Reselected!", Toast.LENGTH_SHORT).show();
    }
}

public static class SimpleFragment extends Fragment {
    TextView textView;
    int mNum;

    /**
     * When creating, retrieve this instance's number from its arguments.
     */
    @Override
    public void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        Log.d(FragmentTabs.TAG, "onCreate " + (savedInstanceState != null ? ("state " + savedInstanceState.getInt("number")) : "no state"));
        if(savedInstanceState != null) {
            mNum = savedInstanceState.getInt("number");
        } else {
            mNum = 25;
        }
    }

    @Override
    public void onActivityCreated(Bundle savedInstanceState) {
        Log.d(TAG, "onActivityCreated");
        if(savedInstanceState != null) {
            Log.d(TAG, "saved variable number: " + savedInstanceState.getInt("number"));
        }
        super.onActivityCreated(savedInstanceState);
    }

    @Override
    public void onSaveInstanceState(Bundle outState) {
        Log.d(TAG, "onSaveInstanceState saving: " + mNum);
        outState.putInt("number", mNum);
        super.onSaveInstanceState(outState);
    }

    @Override
    public View onCreateView(LayoutInflater inflater, ViewGroup container, Bundle savedInstanceState) {
        Log.d(FragmentTabs.TAG, "onCreateView " + (savedInstanceState != null ? ("state: " + savedInstanceState.getInt("number")) : "no state"));
        textView = new TextView(getActivity());
        textView.setText("Hello world: " + mNum);
        textView.setBackgroundDrawable(getResources().getDrawable(android.R.drawable.gallery_thumb));
        return textView;
    }
}

}

Voici la sortie extraite de l'exécution de cet exemple, puis de la rotation du téléphone:

06-11 11:31:42.559: D/FragmentTabs(10726): onTabSelected adding fragment mysimple
06-11 11:31:42.559: D/FragmentTabs(10726): onCreate no state
06-11 11:31:42.559: D/FragmentTabs(10726): onCreateView no state
06-11 11:31:42.567: D/FragmentTabs(10726): onActivityCreated
06-11 11:31:45.286: D/FragmentTabs(10726): onSaveInstanceState saving: 25
06-11 11:31:45.325: D/FragmentTabs(10726): onCreate state 25
06-11 11:31:45.340: D/FragmentTabs(10726): constructor: detaching fragment mysimple
06-11 11:31:45.340: D/FragmentTabs(10726): onTabSelected attaching fragment mysimple
06-11 11:31:45.348: D/FragmentTabs(10726): FragmentTabs.onCreate tab: 0
06-11 11:31:45.348: D/FragmentTabs(10726): FragmentTabs.onCreate number: 0
06-11 11:31:45.348: D/FragmentTabs(10726): onCreateView state: 25
06-11 11:31:45.348: D/FragmentTabs(10726): onActivityCreated
06-11 11:31:45.348: D/FragmentTabs(10726): saved variable number: 25
06-11 11:31:45.348: D/FragmentTabs(10726): onCreateView no state
06-11 11:31:45.348: D/FragmentTabs(10726): onActivityCreated

Ma question est la suivante: pourquoi onCreateView et onActivityCreated sont-ils appelés deux fois? La première fois avec un Bundle avec l'état enregistré et la deuxième fois avec un saveInstanceState nul?

Cela pose des problèmes de conservation de l'état du fragment lors de la rotation.

Dave
la source
2
Je pense que cette question peut être liée à stackoverflow.com/a/8678705/404395
marioosh

Réponses:

45

J'étais en train de me gratter la tête à ce sujet pendant un moment aussi, et comme l'explication de Dave est un peu difficile à comprendre, je posterai mon code (qui fonctionne apparemment):

private class TabListener<T extends Fragment> implements ActionBar.TabListener {
    private Fragment mFragment;
    private Activity mActivity;
    private final String mTag;
    private final Class<T> mClass;

    public TabListener(Activity activity, String tag, Class<T> clz) {
        mActivity = activity;
        mTag = tag;
        mClass = clz;
        mFragment=mActivity.getFragmentManager().findFragmentByTag(mTag);
    }

    public void onTabSelected(Tab tab, FragmentTransaction ft) {
        if (mFragment == null) {
            mFragment = Fragment.instantiate(mActivity, mClass.getName());
            ft.replace(android.R.id.content, mFragment, mTag);
        } else {
            if (mFragment.isDetached()) {
                ft.attach(mFragment);
            }
        }
    }

    public void onTabUnselected(Tab tab, FragmentTransaction ft) {
        if (mFragment != null) {
            ft.detach(mFragment);
        }
    }

    public void onTabReselected(Tab tab, FragmentTransaction ft) {
    }
}

Comme vous pouvez le voir, cela ressemble à peu près à l'exemple Android, mis à part le fait de ne pas se détacher dans le constructeur et d'utiliser replace au lieu d'ajouter .

Après beaucoup de headscratching et d'essais et d'erreurs, j'ai trouvé que trouver le fragment dans le constructeur semble faire disparaître le problème du double onCreateView comme par magie (je suppose qu'il finit par être nul pour onTabSelected lorsqu'il est appelé via le chemin ActionBar.setSelectedNavigationItem () lorsque état de sauvegarde / restauration).

Staffan
la source
Fonctionne parfaitement bien! Vous avez sauvé ma nuit de sommeil! Merci :)
jaibatrik
vous pouvez également utiliser fragment.getClass (). getName () si vous souhaitez supprimer la variable de classe et supprimer un paramètre de l'appel
Ben Sewards
Fonctionne parfaitement avec l'exemple Android "référence précédente TabListener" - tnx. Le plus récent "exemple de référence TabListener" d'Android [comme sur 4 ix 2013] est vraiment, vraiment faux.
Grzegorz Dev
où est l'appel de la méthode ft.commit () ??
MSaudi
1
@MuhammadBabar, voir stackoverflow.com/questions/23248789/… . Si vous utilisez à la addplace replaceet faites pivoter l'écran, vous aurez de nombreux fragments » onCreateView().
CoolMind
26

Ok, voici ce que j'ai découvert.

Ce que je n'ai pas compris, c'est que tous les fragments associés à une activité lorsqu'un changement de configuration se produit (le téléphone tourne) sont recréés et ajoutés à l'activité. (ce qui a du sens)

Ce qui se passait dans le constructeur TabListener était que l'onglet était détaché s'il était trouvé et attaché à l'activité. Voir ci-dessous:

mFragment = mActivity.getFragmentManager().findFragmentByTag(mTag);
    if (mFragment != null && !mFragment.isDetached()) {
        Log.d(TAG, "constructor: detaching fragment " + mTag);
        FragmentTransaction ft = mActivity.getFragmentManager().beginTransaction();
        ft.detach(mFragment);
        ft.commit();
    }

Plus tard dans l'activité onCreate, l'onglet précédemment sélectionné a été sélectionné dans l'état de l'instance enregistrée. Voir ci-dessous:

if (savedInstanceState != null) {
    bar.setSelectedNavigationItem(savedInstanceState.getInt("tab", 0));
    Log.d(TAG, "FragmentTabs.onCreate tab: " + savedInstanceState.getInt("tab"));
    Log.d(TAG, "FragmentTabs.onCreate number: " + savedInstanceState.getInt("number"));
}

Lorsque l'onglet était sélectionné, il était rattaché dans le rappel onTabSelected.

public void onTabSelected(Tab tab, FragmentTransaction ft) {
    if (mFragment == null) {
        mFragment = Fragment.instantiate(mActivity, mClass.getName(), mArgs);
        Log.d(TAG, "onTabSelected adding fragment " + mTag);
        ft.add(android.R.id.content, mFragment, mTag);
    } else {
        Log.d(TAG, "onTabSelected attaching fragment " + mTag);
        ft.attach(mFragment);
    }
}

Le fragment attaché est le deuxième appel aux méthodes onCreateView et onActivityCreated. (Le premier étant lorsque le système recrée l'activité et tous les fragments attachés) La première fois que le bundle onSavedInstanceState aurait enregistré des données, mais pas la deuxième fois.

La solution est de ne pas détacher le fragment dans le constructeur TabListener, laissez-le simplement attaché. (Vous devez toujours le trouver dans le FragmentManager par sa balise) De plus, dans la méthode onTabSelected, je vérifie si le fragment est détaché avant de l'attacher. Quelque chose comme ça:

public void onTabSelected(Tab tab, FragmentTransaction ft) {
            if (mFragment == null) {
                mFragment = Fragment.instantiate(mActivity, mClass.getName(), mArgs);
                Log.d(TAG, "onTabSelected adding fragment " + mTag);
                ft.add(android.R.id.content, mFragment, mTag);
            } else {

                if(mFragment.isDetached()) {
                    Log.d(TAG, "onTabSelected attaching fragment " + mTag);
                    ft.attach(mFragment);
                } else {
                    Log.d(TAG, "onTabSelected fragment already attached " + mTag);
                }
            }
        }
Dave
la source
4
Les solutions mentionnées "ne pas détacher le fragment dans le constructeur TabListener" font que les fragments d'onglets se chevauchent. Je peux voir le contenu des autres fragments. Ça ne marche pas pour moi.
Aksel Fatih
@ flock.dux Je ne suis pas sûr de ce que vous entendez par chevauchement. Android s'occupe de leur mise en page, nous spécifions simplement l'attachement ou le détachement. Il doit y en avoir plus. Peut-être que si vous posez une nouvelle question avec un exemple de code, nous pouvons comprendre ce qui se passe pour vous.
Dave
1
J'ai eu le même problème (plusieurs appels de constructeur de fragments depuis Android). Votre découverte résout mon problème: ce que je n'ai pas compris, c'est que tous les fragments attachés à une activité lors d'un changement de configuration (le téléphone tourne) sont recréés et ajoutés à l'activité. (ce qui a du sens)
eugene
26

J'ai eu le même problème avec une activité simple ne portant qu'un seul fragment (qui serait parfois remplacé). J'ai alors réalisé que j'utilisais onSaveInstanceState uniquement dans le fragment (et onCreateView pour vérifier savedInstanceState), pas dans l'activité.

Sur l'appareil, l'activité contenant les fragments est redémarrée et onCreated est appelé. Là, j'ai attaché le fragment requis (ce qui est correct au premier démarrage).

Sur l'appareil, Android a tout d'abord recréé le fragment qui était visible, puis appelé onCreate de l'activité contenant où mon fragment était attaché, remplaçant ainsi l'original visible.

Pour éviter cela, j'ai simplement changé mon activité pour vérifier la présence de savedInstanceState:

protected void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);

if (savedInstanceState != null) {
/**making sure you are not attaching the fragments again as they have 
 been 
 *already added
 **/
 return; 
 }
 else{
  // following code to attach fragment initially
 }

 }

Je n'ai même pas écrasé onSaveInstanceState de l'activité.

Gunnar Bernstein
la source
Je vous remercie. Cela m'a aidé avec AppCompatActivity + PreferenceFragmentCompat et le blocage lors de l'affichage des boîtes de dialogue dans le fragment de préférence après le changement d'orientation, car le gestionnaire de fragments était nul lors de la création du deuxième fragment.
RoK
12

Les deux réponses votées ici montrent des solutions pour une activité avec le mode de navigation NAVIGATION_MODE_TABS, mais j'ai eu le même problème avec un NAVIGATION_MODE_LIST. Cela a fait perdre inexplicablement à mes fragments leur état lorsque l'orientation de l'écran a changé, ce qui était vraiment ennuyeux. Heureusement, grâce à leur code utile, j'ai réussi à le comprendre.

Fondamentalement, lorsque vous utilisez une navigation de liste, `` onNavigationItemSelected () is automatically called when your activity is created/re-created, whether you like it or not. To prevent your Fragment'sonCreateView () from being called twice, this initial automatic call toonNavigationItemSelected () should check whether the Fragment is already in existence inside your Activity. If it is, return immediately, because there is nothing to do; if it isn't, then simply construct the Fragment and add it to the Activity like you normally would. Performing this check prevents your Fragment from needlessly being created again, which is what causesonCreateView () '' doit être appelé deux fois!

Voir ma onNavigationItemSelected()mise en œuvre ci-dessous.

public class MyActivity extends FragmentActivity implements ActionBar.OnNavigationListener
{
    private static final String STATE_SELECTED_NAVIGATION_ITEM = "selected_navigation_item";

    private boolean mIsUserInitiatedNavItemSelection;

    // ... constructor code, etc.

    @Override
    public void onRestoreInstanceState(Bundle savedInstanceState)
    {
        super.onRestoreInstanceState(savedInstanceState);

        if (savedInstanceState.containsKey(STATE_SELECTED_NAVIGATION_ITEM))
        {
            getActionBar().setSelectedNavigationItem(savedInstanceState.getInt(STATE_SELECTED_NAVIGATION_ITEM));
        }
    }

    @Override
    public void onSaveInstanceState(Bundle outState)
    {
        outState.putInt(STATE_SELECTED_NAVIGATION_ITEM, getActionBar().getSelectedNavigationIndex());

        super.onSaveInstanceState(outState);
    }

    @Override
    public boolean onNavigationItemSelected(int position, long id)
    {    
        Fragment fragment;
        switch (position)
        {
            // ... choose and construct fragment here
        }

        // is this the automatic (non-user initiated) call to onNavigationItemSelected()
        // that occurs when the activity is created/re-created?
        if (!mIsUserInitiatedNavItemSelection)
        {
            // all subsequent calls to onNavigationItemSelected() won't be automatic
            mIsUserInitiatedNavItemSelection = true;

            // has the same fragment already replaced the container and assumed its id?
            Fragment existingFragment = getSupportFragmentManager().findFragmentById(R.id.container);
            if (existingFragment != null && existingFragment.getClass().equals(fragment.getClass()))
            {
                return true; //nothing to do, because the fragment is already there 
            }
        }

        getSupportFragmentManager().beginTransaction().replace(R.id.container, fragment).commit();
        return true;
    }
}

J'ai emprunté l'inspiration pour cette solution à ici .

XåpplI'-I0llwlg'I -
la source
Cette solution fonctionne pour mon problème similaire avec un tiroir de navigation. Je trouve le fragment existant par ID et vérifie s'il a la même classe que le nouveau fragment avant de le recréer.
William le
8

Il me semble que c'est parce que vous instanciez votre TabListener à chaque fois ... donc le système recrée votre fragment à partir de savedInstanceState et vous le faites à nouveau dans votre onCreate.

Vous devez encapsuler cela dans un if(savedInstanceState == null)afin qu'il ne se déclenche que s'il n'y a pas de savedInstanceState.

Barak
la source
Je ne pense pas que ce soit correct. Lorsque j'enveloppe mon code addTab dans le bloc if, le fragment est attaché à l'activité mais il n'y a pas d'onglets. Il semble que vous deviez ajouter les onglets à chaque fois dans la méthode onCreate. Je continuerai à étudier cela et posterai plus si je comprends mieux.
Dave