web-dev-qa-db-fra.com

Actions dans onActivityResult et "Erreur impossible d'effectuer cette action après onSaveInstanceState"

Implémentation d'une application où l'utilisateur peut se connecter J'ai la situation suivante: si l'utilisateur est connecté, effectuez l'action, sinon démarrez l'activité de connexion pour le résultat et si le résultat est Activity.RESULT_OK effectuez l'action.

Mon problème est que l’action à effectuer consiste à afficher un DialogFragment, mais en appelant 

DialogFragment newFragment = MyDialogFragment.newInstance(mStackLevel);
newFragment.show(ft, "dialog")

dans le rappel onActivityResult lève une exception: 

Caused by: Java.lang.IllegalStateException:  
Can not perform this action after onSaveInstanceState

Alors, comment puis-je résoudre ce problème? Je pense en levant un drapeau et montrer le dialogue dans le onResume mais je vois cette solution un peu sale

Edit: Ajout de code supplémentaire (dans cet exemple, je montre le DialogFragment

Lorsque l'action est demandée par l'utilisateur:

... 
if (!user.isLogged()){
 startActivityForResult(new Intent(cnt, Login.class), REQUEST_LOGIN_FOR_COMMENT);
}

Dans le même fragment

@Override
public void onActivityResult(int requestCode, int resultCode, Intent data) {
    super.onActivityResult(requestCode, resultCode, data);
    if (requestCode == REQUEST_LOGIN_FOR_COMMENT && resultCode == Activity.RESULT_OK) {
        FragmentTransaction ft = getFragmentManager().beginTransaction();
        DialogFragment newFragment = MyDialogFragment.newInstance();
        newFragment.show(ft, "dialog")
    }
}

Et si l'utilisateur se connecte à l'activité de connexion appelle;

setResult(Activity.RESULT_OK);
finish();
javaandroidandroid-fragments
42
Addev

La meilleure chose que j'ai faite est de ne pas utiliser .show () mais plutôt de le faire.

CheckinSuccessDialog dialog = new CheckinSuccessDialog();
//dialog.show(getSupportFragmentManager(), null);
FragmentTransaction ft = getSupportFragmentManager().beginTransaction();
ft.add(dialog, null);
ft.commitAllowingStateLoss();
96
schwiz

Voici la solution de contournement qui fonctionne bien pour moi.

private void alert(final String message) {
    Handler handler = new Handler(Looper.getMainLooper());
    handler.post(new Runnable() {
        public void run() {
            AlertDialogFragment alertDialogFragment = AlertDialogFragment.newInstance(message);
            alertDialogFragment.show(getFragmentManager(), ALERT_DIALOG_FRAGMENT);
        }
    });        
}
10
Cheok Yan Cheng

Je vérifie simplement si une activité est en train d'être détruite.

if (!getActivity().isFinishing()) {
    DialogFragment fragment = MyFragment.newInstance();
    fragment.show(getActivity().getSupportFragmentManager(), MyFragment.TAG);
}
4
CoolMind

Si vous utilisez un DialogFragment, la seule chose qui a fonctionné pour moi est de supprimer le fragment avec dissmissAllowingStateLoss () 

4
GaRRaPeTa

Remplacer la fonction show(Fragment manager, String tag) par l’autorisation de perdre l’état et modifier le mDismissed = false;mShownByMe = true; de la fonction originale par réflexion 

public class DialogParent extends DialogFragment {

    @Override
    public void show(FragmentManager manager, String tag) {
        try {
            Field mDismissed = DialogFragment.class.getDeclaredField("mDismissed");
            Field mShownByMe = DialogFragment.class.getDeclaredField("mShownByMe");
            mDismissed.setAccessible(true);
            mShownByMe.setAccessible(true);
            mDismissed.setBoolean(this, false);
            mShownByMe.setBoolean(this, true);
        } catch (NoSuchFieldException e) {
            e.printStackTrace();
        } catch (IllegalAccessException e) {
            e.printStackTrace();
        }
        FragmentTransaction ft = manager.beginTransaction();
        ft.add(this, tag);
        ft.commitAllowingStateLoss();
    }
}
1
georgehardcore

Cela fonctionne vraiment. 

CheckinSuccessDialog dialog = new CheckinSuccessDialog();
//dialog.show(getSupportFragmentManager(), null);
FragmentTransaction ft = getSupportFragmentManager().beginTransaction();
ft.add(dialog, null);
ft.commitAllowingStateLoss();

Mais toujours mauvais, parce que j'ai l'erreur

ava.lang.IllegalStateException: Activity has been destroyed fragmentTransaction.commitAllowingStateLoss();

Ma solution est donc d’ajouter check check if (!isFinishing()&&!isDestroyed())

CheckinSuccessDialog fragment = CheckinSuccessDialog.newInstance();

     if (fragment instanceof DialogFragment) {
                DialogFragment dialog = (DialogFragment) fragment;
                if (!dialog.isAdded()) {
                    fragmentTransaction.add(dialog, 
                          CheckinSuccessDialog.class.getName());
                    if (!isFinishing()&&!isDestroyed()) {
                        fragmentTransaction.commitAllowingStateLoss();
                    }
                }

sur le licenciement:

FragmentTransaction fragmentTransaction = getSupportFragmentManager().beginTransaction();
        Fragment fragment = getSupportFragmentManager().findFragmentByTag(CheckinSuccessDialog.class.getName());
        if (fragment != null && fragment instanceof DialogFragment) {
            DialogFragment dialog = (DialogFragment) fragment;
            dialog.dismiss();
            if (!isFinishing()&&!isDestroyed()) {
                fragmentTransaction.commitAllowingStateLoss();
            }
        }
0
Serg Burlaka

Cette exception est levée chaque fois qu'une FragmentTrasaction est validée après que FragmentManager a sauvegardé son état. La méthode simple et propre consiste à vérifier si FragmentManager a déjà été sauvegardé avant d'afficher une DialogFragment.

if(!getSupportFragmentManager.isStateSaved()) {
    MyDialogFragment dialogFragment = new MyDialogFragment()
    dialogFragment.show(getSupportFragmentManager, TAG);
}
0
jimmy0251