Android RecyclerView: notifyDataSetChanged () IllegalStateException

Mit einem Handler für das hinzufügen von Elementen und den Aufruf notify...() aus diesem Handler behoben das Problem für mich.

InformationsquelleAutor der Antwort cybergen

Weiß ich nicht, aber ich hatte auch das gleiche problem. Ich löste dies durch die Verwendung onClickListner auf checkbox

viewHolder.mCheckBox.setOnClickListener(new OnClickListener() {
        @Override
        public void onClick(View v) {
            //TODO Auto-generated method stub
            if (model.isCheckboxBoolean()) {
                model.setCheckboxBoolean(false);
                viewHolder.mCheckBox.setChecked(false);
            } else {
                model.setCheckboxBoolean(true);
                viewHolder.mCheckBox.setChecked(true);
            }
            notifyDataSetChanged();
        }
    });

Versuchen, diese, dies kann helfen!

InformationsquelleAutor der Antwort jigar

Fand eine einfache Lösung -

public class MyAdapter extends RecyclerView.Adapter<RecyclerView.ViewHolder>{

    private RecyclerView mRecyclerView; 

    @Override
    public void onAttachedToRecyclerView(RecyclerView recyclerView) {
        super.onAttachedToRecyclerView(recyclerView);
        mRecyclerView = recyclerView;
    }

    private CompoundButton.OnCheckedChangeListener checkedChangeListener 
    = (compoundButton, b) -> {
        final int position = (int) compoundButton.getTag();
        //This class is used to make changes to child view
        final Event event = mDataset.get(position);
        //Update state of checkbox or some other computation which you require
        event.state = b;
        //we create a runnable and then notify item changed at position, this fix crash
        mRecyclerView.post(new Runnable() {
            @Override public void run() {
                notifyItemChanged(position));
            }
        });
    }
}

Hier erstellen wir ein runnable zu notifyItemChanged für eine position bei der recyclerview ist bereit, es zu handhaben.

InformationsquelleAutor der Antwort Rohan Kandwal

Zuerst dachte ich, Moonsoo Antwort (die akzeptierte Antwort) würde nicht funktionieren für mich, denn ich kann nicht initialisiert werden meine setOnCheckedChangeListener() im ViewHolder-Konstruktor, weil ich zu binden, müssen Sie es jedes mal, so wird es eine aktualisierte position variabel. Aber es dauerte eine lange Zeit, um zu realisieren, was er sagte.

Hier ist ein Beispiel für die "Kreis-Methode aufrufen," er redet:

public void onBindViewHolder(final ViewHolder holder, final int position) {
    SwitchCompat mySwitch = (SwitchCompat) view.findViewById(R.id.switch);
    mySwitch.setOnCheckedChangeListener(new CompoundButton.OnCheckedChangeListener() {
                @Override
                public void onCheckedChanged(CompoundButton buttonView, boolean isChecked) {
                       if (isChecked) {
                           data.delete(position);
                           notifyItemRemoved(position);
                           //This will call onBindViewHolder, but we can't do that when we are already in onBindViewHolder!
                           notifyItemRangeChanged(position, data.size());
                       }
                   }
            });
    //Set the switch to how it previously was.
    mySwitch.setChecked(savedSwitchState); //If the saved state was "true", then this will trigger the infinite loop.
}

Das einzige problem mit diesem ist, dass, wenn wir Sie brauchen zum initialisieren der Schalter ein-oder ausgeschaltet (vom letzten gespeicherten Zustand, zum Beispiel), es ruft die Zuhörer, die nennen könnte nofityItemRangeChanged fordert onBindViewHolder wieder. Sie kann nicht rufen onBindViewHolder wenn Sie bereits in onBindViewHolder], weil Sie nicht notifyItemRangeChanged wenn Sie bereits in der Mitte der Benachrichtigung, dass der Artikel-Reihe hat sich geändert. Aber ich brauchte nur die Aktualisierung der Benutzeroberfläche, um zu zeigen oder zu deaktivieren, die nicht wollen, um tatsächlich etwas auslösen.

Hier ist die Lösung, die ich gelernt, aus JoniDS Antwortdass die Endlosschleife. Solange wir legen den Hörer auf "null", bevor wir Überprüft, dann wird es die Aktualisierung der Benutzeroberfläche, ohne dass die Zuhörer, die Vermeidung der Endlosschleife. Dann können wir die Zuhörer nach.

JoniDS code:

holder.checkbox.setOnCheckedChangeListener(null);
holder.checkbox.setChecked(condition);
holder.checkbox.setOnCheckedChangeListener(checkedListener);

Vollständige Lösung zu meinem Beispiel:

public void onBindViewHolder(final ViewHolder holder, final int position) {
    SwitchCompat mySwitch = (SwitchCompat) view.findViewById(R.id.switch);

    //Set it to null to erase an existing listener from a recycled view.
    mySwitch.setOnCheckedChangeListener(null);

    //Set the switch to how it previously was without triggering the listener.
    mySwitch.setChecked(savedSwitchState); //If the saved state was "true", then this will trigger the infinite loop.

    //Set the listener now.
    mySwitch.setOnCheckedChangeListener(new CompoundButton.OnCheckedChangeListener() {
        @Override
        public void onCheckedChanged(CompoundButton buttonView, boolean isChecked) {
            if (isChecked) {
                data.delete(position);
                notifyItemRemoved(position);
                //This will call onBindViewHolder, but we can't do that when we are already in onBindViewHolder!
                notifyItemRangeChanged(position, data.size());
            }
        }
    });
}

InformationsquelleAutor der Antwort Rock Lee

Während Element gebunden wird, durch den layout-manager, ist es sehr wahrscheinlich, dass Sie der gecheckte Status des Kontrollkästchens, denen ist die Auslösung der Rückruf.

Dies ist natürlich nur eine Vermutung, weil Sie nicht veröffentlichen, das volle stack-trace.

Können Sie nicht ändern-adapter Inhalt, während RV Neuberechnung des Layouts. Sie können dies vermeiden, indem nicht aufrufen notifyDataSetChanged, wenn item ausgewählt ist-Zustand ist gleich dem Wert gesendet der Rückruf (das wird der Fall sein, wenn der Aufruf checkbox.setChecked ist die Auslösung der callback).

InformationsquelleAutor der Antwort yigit

Bevor notifyDataSetChanged() einfach überprüfen, ob mit dieser Methode: recyclerView.IsComputingLayout()

InformationsquelleAutor der Antwort Amir Hossein Ghasemi

Lief ich in genau dieses Problem! Nach Moonsoo die Antwort nicht wirklich zu schwimmen, mein Boot, ich verwirrte Sie ein bisschen herum und fand eine Lösung, die für mich gearbeitet.

Erste, hier sind einige meiner code:

    @Override
    public void onBindViewHolder(ViewHolder holder, final int position) {

    final Event event = mDataset.get(position);

    //
    // .......
    //

    holder.mSwitch.setOnCheckedChangeListener(new CompoundButton.OnCheckedChangeListener() {
        @Override
        public void onCheckedChanged(CompoundButton buttonView, boolean isChecked) {
            event.setActive(isChecked);
            try {
                notifyItemChanged(position);
            } catch (Exception e) {
                Log.e("onCheckChanged", e.getMessage());
            }
        }
    });

Sie werden bemerken, ich bin insbesondere die Benachrichtigung der adapter für die position, die ich ändern bin, anstatt den gesamten Datenbestand wie du tust. , Die being said, obwohl ich nicht garantieren kann, wird dies für Sie arbeiten, ich habe das problem gelöst, indem er meine notifyItemChanged() Aufruf in einen try/catch-block. Diese einfach gefangen die Ausnahme, aber immer noch erlaubt, meinen adapter zu registrieren, den Zustand zu ändern, und aktualisieren Sie die Anzeige!

Hoffe, das jemand hilft!

EDIT: ich gebe zu, dies ist wahrscheinlich nicht die richtige/Reife Art, behandeln das Thema, aber da es scheint nicht zu sein, der verursacht keine Probleme, indem man die Ausnahme nicht behandelte, ich dachte, ich würde teilen, falls es gut genug für jemand anderes.

InformationsquelleAutor der Antwort Andrew

Warum nicht überprüfung der RecyclerView.isComputingLayout() Zustand wie folgt?

public class MyAdapter extends RecyclerView.Adapter<RecyclerView.ViewHolder>{

    private RecyclerView mRecyclerView; 

    @Override
    public void onAttachedToRecyclerView(RecyclerView recyclerView) {
        super.onAttachedToRecyclerView(recyclerView);
        mRecyclerView = recyclerView;
    }

    @Override
    public void onBindViewHolder(ViewHolder viewHolder, int position) {

        viewHolder.getCheckbox().setOnCheckedChangeListener(new CompoundButton.OnCheckedChangeListener() {
            @Override
            public void onCheckedChanged(CompoundButton buttonView, boolean isChecked) {
                if (mRecyclerView != null && !mRecyclerView.isComputingLayout()) {
                    notifyDataSetChanged();
                }
            }
        });
    }
}

InformationsquelleAutor der Antwort NcJie

Für mich, ich hörte zu ändern, in der Bewertung der rating-bar, aber auf lange drücken der mehrere Klicks auf einmal app stürzt ab wegen dem Thema, dann eine klare Lösung, wenn ich wollte notifydatasetchange(); in bindviewholder behandelt es mit handler gegeben durch:

//inside bindViewHolder                 
new Handler().post(new Runnable() {
                    @Override
                    public void run() {
                        notifyDataSetChanged();
                    }
                });

Hoffe, so wird es das Problem zu beheben.

InformationsquelleAutor der Antwort Ali Nawaz

verwenden DiffUtil statt um bessere Leistung zu erhalten

InformationsquelleAutor der Antwort Zafer Celaloglu