Come evitare che le visualizzazioni personalizzate perdano stato attraverso le modifiche all'orientamento dello schermo

Ho implementato con successo il onRetainNonConfigurationInstance()mio principale Activityper salvare e ripristinare alcuni componenti critici attraverso i cambiamenti di orientamento dello schermo.

Ma sembra che le mie viste personalizzate vengano ricreate da zero quando cambia l'orientamento. Questo ha senso, anche se nel mio caso è scomodo perché la vista personalizzata in questione è una trama X / Y e i punti tracciati sono memorizzati nella vista personalizzata.

Esiste un modo astuto per implementare qualcosa di simile a onRetainNonConfigurationInstance()per una vista personalizzata o devo semplicemente implementare metodi nella vista personalizzata che mi consentano di ottenere e impostare il suo "stato"?

A tale scopo, l'attuazione View#onSaveInstanceStatee View#onRestoreInstanceStateed estendendo la View.BaseSavedStateclasse.

public class CustomView extends View {

  private int stateToSave;

  ...

  @Override
  public Parcelable onSaveInstanceState() {
    //begin boilerplate code that allows parent classes to save state
    Parcelable superState = super.onSaveInstanceState();

    SavedState ss = new SavedState(superState);
    //end

    ss.stateToSave = this.stateToSave;

    return ss;
  }

  @Override
  public void onRestoreInstanceState(Parcelable state) {
    //begin boilerplate code so parent classes can restore state
    if(!(state instanceof SavedState)) {
      super.onRestoreInstanceState(state);
      return;
    }

    SavedState ss = (SavedState)state;
    super.onRestoreInstanceState(ss.getSuperState());
    //end

    this.stateToSave = ss.stateToSave;
  }

  static class SavedState extends BaseSavedState {
    int stateToSave;

    SavedState(Parcelable superState) {
      super(superState);
    }

    private SavedState(Parcel in) {
      super(in);
      this.stateToSave = in.readInt();
    }

    @Override
    public void writeToParcel(Parcel out, int flags) {
      super.writeToParcel(out, flags);
      out.writeInt(this.stateToSave);
    }

    //required field that makes Parcelables from a Parcel
    public static final Parcelable.Creator<SavedState> CREATOR =
        new Parcelable.Creator<SavedState>() {
          public SavedState createFromParcel(Parcel in) {
            return new SavedState(in);
          }
          public SavedState[] newArray(int size) {
            return new SavedState[size];
          }
    };
  }
}

Il lavoro viene suddiviso tra la classe View e SavedState di View. Dovresti fare tutto il lavoro di lettura e scrittura da e verso Parcella SavedStateclasse. Quindi la tua classe View può fare il lavoro di estrarre i membri dello stato e fare il lavoro necessario per riportare la classe ad uno stato valido.

Note: View#onSavedInstanceStatee View#onRestoreInstanceStatevengono chiamati automaticamente se View#getIdrestituisce un valore> = 0. Ciò accade quando si assegna un ID in XML o si chiama setIdmanualmente. Altrimenti devi chiamare View#onSaveInstanceStatee scrivere il pacco restituito al pacco in cui ti trovi Activity#onSaveInstanceStateper salvare lo stato e successivamente leggerlo e passarlo View#onRestoreInstanceStateda Activity#onRestoreInstanceState.

Un altro semplice esempio di questo è il CompoundButton

Penso che questa sia una versione molto più semplice. Bundleè un tipo incorporato che implementaParcelable

public class CustomView extends View
{
  private int stuff; // stuff

  @Override
  public Parcelable onSaveInstanceState()
  {
    Bundle bundle = new Bundle();
    bundle.putParcelable("superState", super.onSaveInstanceState());
    bundle.putInt("stuff", this.stuff); // ... save stuff 
    return bundle;
  }

  @Override
  public void onRestoreInstanceState(Parcelable state)
  {
    if (state instanceof Bundle) // implicit null check
    {
      Bundle bundle = (Bundle) state;
      this.stuff = bundle.getInt("stuff"); // ... load stuff
      state = bundle.getParcelable("superState");
    }
    super.onRestoreInstanceState(state);
  }
}

Ecco un'altra variante che utilizza un mix dei due metodi precedenti. Combinando la velocità e la correttezza di Parcelablecon la semplicità di un Bundle:

@Override
public Parcelable onSaveInstanceState() {
    Bundle bundle = new Bundle();
    // The vars you want to save - in this instance a string and a boolean
    String someString = "something";
    boolean someBoolean = true;
    State state = new State(super.onSaveInstanceState(), someString, someBoolean);
    bundle.putParcelable(State.STATE, state);
    return bundle;
}

@Override
public void onRestoreInstanceState(Parcelable state) {
    if (state instanceof Bundle) {
        Bundle bundle = (Bundle) state;
        State customViewState = (State) bundle.getParcelable(State.STATE);
        // The vars you saved - do whatever you want with them
        String someString = customViewState.getText();
        boolean someBoolean = customViewState.isSomethingShowing());
        super.onRestoreInstanceState(customViewState.getSuperState());
        return;
    }
    // Stops a bug with the wrong state being passed to the super
    super.onRestoreInstanceState(BaseSavedState.EMPTY_STATE); 
}

protected static class State extends BaseSavedState {
    protected static final String STATE = "YourCustomView.STATE";

    private final String someText;
    private final boolean somethingShowing;

    public State(Parcelable superState, String someText, boolean somethingShowing) {
        super(superState);
        this.someText = someText;
        this.somethingShowing = somethingShowing;
    }

    public String getText(){
        return this.someText;
    }

    public boolean isSomethingShowing(){
        return this.somethingShowing;
    }
}

Le risposte qui sono già ottime, ma non funzionano necessariamente per i ViewGroup personalizzati. Per ottenere che tutte le viste personalizzate mantengano il loro stato, è necessario eseguire l'override onSaveInstanceState()e onRestoreInstanceState(Parcelable state)in ciascuna classe. È inoltre necessario assicurarsi che tutti abbiano ID univoci, siano essi gonfiati da XML o aggiunti a livello di programmazione.

Quello che mi è venuto in mente è stato notevolmente simile alla risposta di Kobor42, ma l'errore è rimasto perché stavo aggiungendo le visualizzazioni a un ViewGroup personalizzato a livello di codice e non assegnando ID univoci.

Il collegamento condiviso da mato funzionerà, ma significa che nessuna delle singole viste gestisce il proprio stato: l'intero stato viene salvato nei metodi ViewGroup.

Il problema è che quando più di questi ViewGroup vengono aggiunti a un layout, gli ID dei loro elementi da xml non sono più univoci (se definito in xml). In fase di esecuzione, è possibile chiamare il metodo statico View.generateViewId()per ottenere un ID univoco per una vista. Questo è disponibile solo da API 17.

Ecco il mio codice dal ViewGroup (è astratto e mOriginalValue è una variabile di tipo):

public abstract class DetailRow<E> extends LinearLayout {

    private static final String SUPER_INSTANCE_STATE = "saved_instance_state_parcelable";
    private static final String STATE_VIEW_IDS = "state_view_ids";
    private static final String STATE_ORIGINAL_VALUE = "state_original_value";

    private E mOriginalValue;
    private int[] mViewIds;

// ...

    @Override
    protected Parcelable onSaveInstanceState() {

        // Create a bundle to put super parcelable in
        Bundle bundle = new Bundle();
        bundle.putParcelable(SUPER_INSTANCE_STATE, super.onSaveInstanceState());
        // Use abstract method to put mOriginalValue in the bundle;
        putValueInTheBundle(mOriginalValue, bundle, STATE_ORIGINAL_VALUE);
        // Store mViewIds in the bundle - initialize if necessary.
        if (mViewIds == null) {
            // We need as many ids as child views
            mViewIds = new int[getChildCount()];
            for (int i = 0; i < mViewIds.length; i++) {
                // generate a unique id for each view
                mViewIds[i] = View.generateViewId();
                // assign the id to the view at the same index
                getChildAt(i).setId(mViewIds[i]);
            }
        }
        bundle.putIntArray(STATE_VIEW_IDS, mViewIds);
        // return the bundle
        return bundle;
    }

    @Override
    protected void onRestoreInstanceState(Parcelable state) {

        // We know state is a Bundle:
        Bundle bundle = (Bundle) state;
        // Get mViewIds out of the bundle
        mViewIds = bundle.getIntArray(STATE_VIEW_IDS);
        // For each id, assign to the view of same index
        if (mViewIds != null) {
            for (int i = 0; i < mViewIds.length; i++) {
                getChildAt(i).setId(mViewIds[i]);
            }
        }
        // Get mOriginalValue out of the bundle
        mOriginalValue = getValueBackOutOfTheBundle(bundle, STATE_ORIGINAL_VALUE);
        // get super parcelable back out of the bundle and pass it to
        // super.onRestoreInstanceState(Parcelable)
        state = bundle.getParcelable(SUPER_INSTANCE_STATE);
        super.onRestoreInstanceState(state);
    } 
}

Ho avuto il problema che onRestoreInstanceState ha ripristinato tutte le mie viste personalizzate con lo stato dell'ultima vista. L'ho risolto aggiungendo questi due metodi alla mia vista personalizzata:

@Override
protected void dispatchSaveInstanceState(SparseArray<Parcelable> container) {
    dispatchFreezeSelfOnly(container);
}

@Override
protected void dispatchRestoreInstanceState(SparseArray<Parcelable> container) {
    dispatchThawSelfOnly(container);
}

Invece di usare onSaveInstanceStatee onRestoreInstanceState, puoi anche usare a ViewModel. Estendi il tuo modello di dati ViewModele quindi puoi utilizzare ViewModelProvidersper ottenere la stessa istanza del tuo modello ogni volta che l'attività viene ricreata:

class MyData extends ViewModel {
    // have all your properties with getters and setters here
}

public class MyActivity extends FragmentActivity {
    @Override
    public void onCreate(Bundle savedInstanceState) {

        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_main);

        // the first time, ViewModelProvider will create a new MyData
        // object. When the Activity is recreated (e.g. because the screen
        // is rotated), ViewModelProvider will give you the initial MyData
        // object back, without creating a new one, so all your property
        // values are retained from the previous view.
        myData = ViewModelProviders.of(this).get(MyData.class);

        ...
    }
}

Per utilizzare ViewModelProviders, aggiungere quanto segue in dependenciesin app/build.gradle:

implementation "android.arch.lifecycle:extensions:1.1.1"
implementation "android.arch.lifecycle:viewmodel:1.1.1"

Nota che l' MyActivityestensione si estende FragmentActivityinvece di estenderla Activity.

Puoi leggere di più su ViewModels qui:

• Guida per sviluppatori Android, Gestire le modifiche alla configurazione
• Guida per gli sviluppatori Android, Salvataggio degli stati dell'interfaccia utente, Utilizzare ViewModel per gestire le modifiche alla configurazione
• Tutorial ViewModels: un semplice esempio

Ho scoperto che questa risposta stava causando alcuni arresti anomali nelle versioni Android 9 e 10. Penso che sia un buon approccio ma quando stavo guardando un po 'di codice Android ho scoperto che mancava un costruttore. La risposta è piuttosto vecchia, quindi al momento probabilmente non ce n'era bisogno. Quando ho aggiunto il costruttore mancante e l'ho chiamato dal creatore, il crash è stato risolto.

Quindi ecco il codice modificato:

public class CustomView extends View {

    private int stateToSave;

    ...

    @Override
    public Parcelable onSaveInstanceState() {
        Parcelable superState = super.onSaveInstanceState();
        SavedState ss = new SavedState(superState);

        // your custom state
        ss.stateToSave = this.stateToSave;

        return ss;
    }

    @Override
    protected void dispatchSaveInstanceState(SparseArray<Parcelable> container)
    {
        dispatchFreezeSelfOnly(container);
    }

    @Override
    public void onRestoreInstanceState(Parcelable state) {
        SavedState ss = (SavedState) state;
        super.onRestoreInstanceState(ss.getSuperState());

        // your custom state
        this.stateToSave = ss.stateToSave;
    }

    @Override
    protected void dispatchRestoreInstanceState(SparseArray<Parcelable> container)
    {
        dispatchThawSelfOnly(container);
    }

    static class SavedState extends BaseSavedState {
        int stateToSave;

        SavedState(Parcelable superState) {
            super(superState);
        }

        private SavedState(Parcel in) {
            super(in);
            this.stateToSave = in.readInt();
        }

        // This was the missing constructor
        @RequiresApi(Build.VERSION_CODES.N)
        SavedState(Parcel in, ClassLoader loader)
        {
            super(in, loader);
            this.stateToSave = in.readInt();
        }

        @Override
        public void writeToParcel(Parcel out, int flags) {
            super.writeToParcel(out, flags);
            out.writeInt(this.stateToSave);
        }    

        public static final Creator<SavedState> CREATOR =
            new ClassLoaderCreator<SavedState>() {

            // This was also missing
            @Override
            public SavedState createFromParcel(Parcel in, ClassLoader loader)
            {
                return Build.VERSION.SDK_INT >= Build.VERSION_CODES.N ? new SavedState(in, loader) : new SavedState(in);
            }

            @Override
            public SavedState createFromParcel(Parcel in) {
                return new SavedState(in, null);
            }

            @Override
            public SavedState[] newArray(int size) {
                return new SavedState[size];
            }
        };
    }
}

Per aumentare altre risposte: se si dispone di più viste composte personalizzate con lo stesso ID e vengono tutte ripristinate con lo stato dell'ultima vista in una modifica della configurazione, è sufficiente indicare alla vista di inviare solo eventi di salvataggio / ripristino a se stesso scavalcando un paio di metodi.

class MyCompoundView : ViewGroup {

    ...

    override fun dispatchSaveInstanceState(container: SparseArray<Parcelable>) {
        dispatchFreezeSelfOnly(container)
    }

    override fun dispatchRestoreInstanceState(container: SparseArray<Parcelable>) {
        dispatchThawSelfOnly(container)
    }
}

Per una spiegazione di ciò che sta accadendo e perché funziona, consulta questo post sul blog . Fondamentalmente gli ID di visualizzazione per bambini della vista composta sono condivisi da ciascuna vista composta e il ripristino dello stato viene confuso. Inviando solo lo stato per la vista composta stessa, impediamo ai loro figli di ricevere messaggi misti da altre viste composte.