RecyclerView: Wykryto niespójność. Nieprawidłowa pozycja elementu

Nasz dział kontroli jakości wykrył błąd: podczas obracania urządzenia z Androidem (Droid Turbo) miała miejsce następująca awaria związana z RecyclerView :

java.lang.IndexOutOfBoundsException: Wykryto niespójność. Nieprawidłowa pozycja 2 pozycji (przesunięcie: 2). Stan: 3

Dla mnie wygląda to na wewnętrzny błąd w RecyclerView, ponieważ nie mogę sobie wyobrazić, aby to było spowodowane bezpośrednio przez nasz kod ...

Czy ktoś napotkał ten problem?

Jakie byłoby rozwiązanie?

Brutalnym obejściem może być złapanie wyjątku, kiedy to nastąpi, i odtworzenie instancji RecyclverView od zera, aby uniknąć pozostawienia uszkodzonego stanu.

Ale jeśli to możliwe, chciałbym lepiej zrozumieć problem (i być może naprawić go u źródła), zamiast maskować.

Błąd nie jest łatwy do odtworzenia, ale jest fatalny w skutkach.

Pełny ślad stosu:

W/dalvikvm( 7546): threadid=1: thread exiting with uncaught exception (group=0x41987d40)
    E/AndroidRuntime( 7546): FATAL EXCEPTION: main
    E/AndroidRuntime( 7546): Process: com.oblong.mezzedroid, PID: 7546
    E/AndroidRuntime( 7546): java.lang.IndexOutOfBoundsException: Inconsistency detected. Invalid item position 2(offset:2).state:3
    E/AndroidRuntime( 7546):    at android.support.v7.widget.RecyclerView$Recycler.getViewForPosition(RecyclerView.java:3382)
    E/AndroidRuntime( 7546):    at android.support.v7.widget.RecyclerView$Recycler.getViewForPosition(RecyclerView.java:3340)
    E/AndroidRuntime( 7546):    at android.support.v7.widget.LinearLayoutManager$LayoutState.next(LinearLayoutManager.java:1810)
    E/AndroidRuntime( 7546):    at android.support.v7.widget.LinearLayoutManager.layoutChunk(LinearLayoutManager.java:1306)
    E/AndroidRuntime( 7546):    at android.support.v7.widget.LinearLayoutManager.fill(LinearLayoutManager.java:1269)
    E/AndroidRuntime( 7546):    at android.support.v7.widget.LinearLayoutManager.onLayoutChildren(LinearLayoutManager.java:523)
    E/AndroidRuntime( 7546):    at org.liboid.recycler_view.RecyclerViewContainer$LiLinearLayoutManager.onLayoutChildren(RecyclerViewContainer.java:179)
    E/AndroidRuntime( 7546):    at android.support.v7.widget.RecyclerView.dispatchLayout(RecyclerView.java:1942)
    E/AndroidRuntime( 7546):    at android.support.v7.widget.RecyclerView.onLayout(RecyclerView.java:2237)
    E/AndroidRuntime( 7546):    at org.liboid.recycler_view.LiRecyclerView.onLayout(LiRecyclerView.java:30)
    E/AndroidRuntime( 7546):    at android.view.View.layout(View.java:14946)
    E/AndroidRuntime( 7546):    at android.view.ViewGroup.layout(ViewGroup.java:4651)
    E/AndroidRuntime( 7546):    at android.widget.FrameLayout.layoutChildren(FrameLayout.java:453)
    E/AndroidRuntime( 7546):    at android.widget.FrameLayout.onLayout(FrameLayout.java:388)
    E/AndroidRuntime( 7546):    at android.view.View.layout(View.java:14946)
    E/AndroidRuntime( 7546):    at android.view.ViewGroup.layout(ViewGroup.java:4651)
    E/AndroidRuntime( 7546):    at android.widget.FrameLayout.layoutChildren(FrameLayout.java:453)
    E/AndroidRuntime( 7546):    at android.widget.FrameLayout.onLayout(FrameLayout.java:388)
    E/AndroidRuntime( 7546):    at android.view.View.layout(View.java:14946)
    E/AndroidRuntime( 7546):    at android.view.ViewGroup.layout(ViewGroup.java:4651)
    E/AndroidRuntime( 7546):    at android.widget.LinearLayout.setChildFrame(LinearLayout.java:1671)
    E/AndroidRuntime( 7546):    at android.widget.LinearLayout.layoutVertical(LinearLayout.java:1525)
    E/AndroidRuntime( 7546):    at android.widget.LinearLayout.onLayout(LinearLayout.java:1434)
    E/AndroidRuntime( 7546):    at com.oblong.mezzedroid.workspace.content.bins.BinsContainerLayout.onLayout(BinsContainerLayout.java:22)
    E/AndroidRuntime( 7546):    at android.view.View.layout(View.java:14946)
    E/AndroidRuntime( 7546):    at android.view.ViewGroup.layout(ViewGroup.java:4651)
    E/AndroidRuntime( 7546):    at android.widget.LinearLayout.setChildFrame(LinearLayout.java:1671)
    E/AndroidRuntime( 7546):    at android.widget.LinearLayout.layoutVertical(LinearLayout.java:1525)
    E/AndroidRuntime( 7546):    at android.widget.LinearLayout.onLayout(LinearLayout.java:1434)
    E/AndroidRuntime( 7546):    at android.view.View.layout(View.java:14946)
    E/AndroidRuntime( 7546):    at android.view.ViewGroup.layout(ViewGroup.java:4651)
    E/AndroidRuntime( 7546):    at android.widget.FrameLayout.layoutChildren(FrameLayout.java:453)
    E/AndroidRuntime( 7546):    at android.widget.FrameLayout.onLayout(FrameLayout.java:388)
    E/AndroidRuntime( 7546):    at android.view.View.layout(View.java:14946)
    E/AndroidRuntime( 7546):    at android.view.ViewGroup.layout(ViewGroup.java:4651)
    E/AndroidRuntime( 7546):    at android.widget.FrameLayout.layoutChildren(FrameLayout.java:453)
    E/AndroidRuntime( 7546):    at android.widget.FrameLayout.onLayout(FrameLayout.java:388)
    E/AndroidRuntime( 7546):    at android.view.View.layout(View.java:14946)
    E/AndroidRuntime( 7546):    at android.view.ViewGroup.layout(ViewGroup.java:4651)
    E/AndroidRuntime( 7546):    at android.widget.LinearLayout.setChildFrame(LinearLayout.java:1671)
    E/AndroidRuntime( 7546):    at android.widget.LinearLayout.layoutVertical(LinearLayout.java:1525)
    E/AndroidRuntime( 7546):    at android.widget.LinearLayout.onLayout(LinearLayout.java:1434)
    E/AndroidRuntime( 7546):    at android.view.View.layout(View.java:14946)
    E/AndroidRuntime( 7546):    at android.view.ViewGroup.layout(ViewGroup.java:4651)
    E/AndroidRuntime( 7546):    at android.widget.FrameLayout.layoutChildren(FrameLayout.java:453)
    E/AndroidRuntime( 7546):    at android.widget.FrameLayout.onLayout(FrameLayout.java:388)
    E/AndroidRuntime( 7546):    at android.view.View.layout(View.java:14946)
    E/AndroidRuntime( 7546):    at android.view.ViewGroup.layout(ViewGroup.java:4651)
    E/AndroidRuntime( 7546):    at android.widget.LinearLayout.setChildFrame(LinearLayout.java:1671)
    E/AndroidRuntime( 7546):    at android.widget.LinearLayout.layoutVertical(LinearLayout.java:1525)
    E/AndroidRuntime( 7546):    at android.widget.LinearLayout.onLayout(LinearLayout.java:1434)
    E/AndroidRuntime( 7546):    at android.view.View.layout(View.java:14946)
    E/AndroidRuntime( 7546):    at android.view.ViewGroup.layout(ViewGroup.java:4651)
    E/AndroidRuntime( 7546):    at android.widget.FrameLayout.layoutChildren(FrameLayout.java:453)
    E/AndroidRuntime( 7546):    at android.widget.FrameLayout.onLayout(FrameLayout.java:388)
    E/AndroidRuntime( 7546):    at android.view.View.layout(View.java:14946)
    E/AndroidRuntime( 7546):    at android.view.ViewGroup.layout(ViewGroup.java:4651)
    E/AndroidRuntime( 7546):    at android.view.ViewRootImpl.performLayout(ViewRootImpl.java:2132)
    E/AndroidRuntime( 7546):    at android.view.ViewRootImpl.performTraversals(ViewRootImpl.java:1872)
    E/AndroidRuntime( 7546):    at andro

Miałem (prawdopodobnie) związany z tym problem - wejście w nową instancję działania za pomocą RecyclerView, ale z mniejszym adapterem spowodowało to dla mnie awarię.

RecyclerView.dispatchLayout()może spróbować wyciągnąć przedmioty ze złomu przed wezwaniem mRecycler.clearOldPositions(). Konsekwencją tego jest to, że wyciągał przedmioty ze wspólnej puli, które miały pozycje wyższe niż rozmiar adaptera.

Na szczęście robi to tylko wtedy, gdy PredictiveAnimationssą włączone, więc moim rozwiązaniem było podklasę GridLayoutManager( LinearLayoutManagerma ten sam problem i „naprawę”) i zastąpienie, supportsPredictiveItemAnimations()aby zwrócić false:

/**
 * No Predictive Animations GridLayoutManager
 */
private static class NpaGridLayoutManager extends GridLayoutManager {
    /**
     * Disable predictive animations. There is a bug in RecyclerView which causes views that
     * are being reloaded to pull invalid ViewHolders from the internal recycler stack if the
     * adapter size has decreased since the ViewHolder was recycled.
     */
    @Override
    public boolean supportsPredictiveItemAnimations() {
        return false;
    }

    public NpaGridLayoutManager(Context context, AttributeSet attrs, int defStyleAttr, int defStyleRes) {
        super(context, attrs, defStyleAttr, defStyleRes);
    }

    public NpaGridLayoutManager(Context context, int spanCount) {
        super(context, spanCount);
    }

    public NpaGridLayoutManager(Context context, int spanCount, int orientation, boolean reverseLayout) {
        super(context, spanCount, orientation, reverseLayout);
    }
}

W moim przypadku (usuń / wstaw dane w mojej strukturze danych) musiałem wyczyścić pulę recyklingu, a następnie powiadomić o zmianie zestawu danych!

mRecyclerView.getRecycledViewPool().clear(); mAdapter.notifyDataSetChanged();

W takim przypadku użyj notifyDataSetChanged()zamiast notifyItem...tego.

Rozwiązałem to, opóźniając mRecycler.setAdapter(itemsAdapter)kasę po dodaniu wszystkich elementów do adaptera mRecycler.addAll(items)i działało. Nie mam pojęcia, dlaczego to zrobiłem, zacząłem od kodu biblioteki, który przejrzałem i zobaczyłem te wiersze w „niewłaściwej kolejności”, ale jestem pewien, że to jest to, proszę, jeśli ktoś może to potwierdzić, wyjaśnić, dlaczego to jest więc? Nie jestem pewien, czy jest to nawet poprawna odpowiedź

Miałem podobny problem, ale nie dokładnie taki sam. W moim przypadku w 1 punkcie wyczyściłem tablicę, która została przekazana do widoku recyklera

mObjects.clear();

i nie wywoływanie powiadomieniaDataSetChanged, ponieważ nie chciałem, aby widok recyklera natychmiast wyczyścił widoki. Ponownie wypełniałem tablicę mObjects w AsyncTask.

Miałem ten sam problem z recyclinglerView Więc właśnie powiadomiłem adapter o zmianie zestawu danych zaraz po wyczyszczeniu listy.

mList.clear();
mAdapter.notifyDataSetChanged();

mList.addAll(newData);
mAdapter.notifyDataSetChanged();

Mam ten sam problem. Wystąpiło, gdy przewijałem szybko i dzwoniłem do API i aktualizowałem dane. Po wypróbowaniu wszystkich rzeczy, aby zapobiec awarii, znalazłem rozwiązanie.

mRecyclerView.stopScroll();

To będzie działać.

Zmieniam dane RecyclerVieww tle Thread. Mam to samo Exceptionco OP. Dodałem to po zmianie danych:

myRecyclerView.post(new Runnable() {
    @Override
    public void run() {
        myRecyclerAdapter.notifyDataSetChanged();
    }
});

Mam nadzieję, że to pomoże

Ten błąd występuje, gdy lista w adapterze zostaje wyczyszczona podczas przewijania przez użytkownika, co powoduje zmianę pozycji uchwytu elementu, utratę referencji między listą a pozycją w interfejsie użytkownika, błąd pojawia się w następnym żądaniu „replaceDataSetChanged” .

Naprawić:

Sprawdź metodę listy aktualizacji. Jeśli zrobisz coś takiego

mainList.clear();
...
mainList.add() or mainList.addAll()
...
notifyDataSetChanged();

===> Error occur

Jak naprawić. Utwórz nowy obiekt listy do przetwarzania bufora i następnie przypisz ponownie do listy głównej

List res = new ArrayList();
…..
res.add();  //add item or modify list
….
mainList = res;
notifyDataSetChanged();

Dzięki Nhan Cao za tę wspaniałą pomoc :)

Mój problem zniknął po zmodyfikowaniu mojej Adapterimplementacji w celu użycia kopii tablicy elementów zamiast odwołania. setItems()Metoda jest wywoływana za każdym razem mamy nowych elementów do pokazania w RecyclerView.

Zamiast:

private class MyAdapter extends RecyclerView.Adapter<ItemHolder> {
     private List<MyItem> mItems;  

    (....)

    void setItems(List<MyItem> items) {
        mItems = items;
    }
}

Zrobiłem:

void setItems(List<MyItem> items) {
    mItems = new ArrayList<>(items);
}

Mam do czynienia z tą samą sytuacją. Zostało to rozwiązane poprzez dodanie kodów przed wyczyszczeniem kolekcji.

mRecyclerView.getRecycledViewPool().clear();

W moim przypadku aktualizowałem elementy i dzwoniłem notifyDataSetChangedw wątku innym niż interfejs użytkownika. Przez większość czasu działało, ale gdy wiele zmian nastąpiło szybko, zawalił się. Kiedy to zrobiłem, w zasadzie

activity.runOnUiThread(new Runnable() {
    @Override
    public void run() {
        changeData();
        notifyDataSetChanged();
    }
});

potem przestało się zawieszać.

Musisz tylko wyczyścić listę, OnPostExecute()a nie robić toPull to Refresh

// Setup refresh listener which triggers new data loading
        swipeContainer.setOnRefreshListener(new SwipeRefreshLayout.OnRefreshListener() {
            @Override
            public void onRefresh() {

                AsyncTask<String,Void,String> task = new get_listings();
                task.execute(); // clear listing inside onPostExecute

            }
        });

Odkryłem, że dzieje się tak, gdy przewijasz podczas ściągania, aby odświeżyć , ponieważ wyczyściłem listę przed async task, w wyniku czego java.lang.IndexOutOfBoundsException: Inconsistency detected.

        swipeContainer.setRefreshing(false);
        //TODO : This is very crucial , You need to clear before populating new items 
        listings.clear();

W ten sposób nie skończysz z niekonsekwencją

Można to również powiązać z wielokrotnym ustawianiem adaptera jednocześnie. Miałem metodę wywołania zwrotnego, która była wyzwalana 5-6 razy w tym samym czasie i ustawiałem adapter w tym wywołaniu zwrotnym, aby RecycledViewPool nie mógł obsługiwać wszystkich tych danych jednocześnie. To wielka szansa, ale i tak lepiej to sprawdzić.

Posługiwać się

notifyDataSetChanged()

zamiast

notifyItemRangeInserted(0, YourArrayList.size())

w tym przypadku.

Aby rozwiązać ten problem, po prostu wywołaj powiadomienieDataSetChanged () z pustą listą przed aktualizacją widoku kosza.

Na przykład

//Method for refresh recycle view

    if (!hcpArray.isEmpty())

hcpArray.clear (); // Lista widoków przeglądu aktualizacji

adapter.notifyDataSetChanged();

W moim przypadku właśnie usunąłem linię z setHasStableIds(true);

W moim przypadku próbowałem zmienić zawartość adaptera w wątku w tle, ale nazwałem powiadomienie * w wątku main / ui.

Nie jest możliwe! Powodem, dla którego powiadomienie jest zmuszane do głównego wątku, jest to, że widok recyklera chce, abyś edytował adapter kopii zapasowej w głównym wątku, nawet na tym samym stosie wywołań.

Aby rozwiązać problem, upewnij się, że każda operacja na twoim adapterze, a także każde powiadomienie ... jest wykonywane w wątku interfejsu użytkownika / głównym !

Niedawno natrafiłem na ten paskudny ślad stosu z nowymi komponentami architektury Androida. Zasadniczo mam listę elementów w moim ViewModel, które są obserwowane przez mój Fragment, używając LiveData. Gdy ViewModel publikuje nową wartość danych, Fragment aktualizuje adapter, przekazując te nowe elementy danych i powiadamiając adapter o zmianach.

Niestety, przekazując nowe elementy danych do adaptera, nie uwzględniłem faktu, że zarówno ViewModel, jak i Adapter wskazywałyby na to samo odwołanie do obiektu! Oznacza to, że jeśli zaktualizuję dane i zadzwoniępostValue() z poziomu ViewModel, pojawi się bardzo małe okno, w którym dane mogą zostać zaktualizowane, a karta jeszcze nie powiadomiona!

Moją poprawką było utworzenie nowej kopii elementów po przekazaniu do adaptera:

mList = new ArrayList<>(passedList);

Dzięki tej bardzo łatwej poprawce możesz mieć pewność, że dane adaptera nie zmienią się, dopóki nie zostanie powiadomiony o tym adapter.

To jest jedyne rozwiązanie, które działało dla mnie, nawet próbując wielu z powyższych rozwiązań.

1.) Intilizacja

CustomAdapter scrollStockAdapter = new CustomAdapter(mActivity, new ArrayList<StockListModel>());
list.setAdapter(scrollStockAdapter);
scrollStockAdapter.updateList(stockListModels);

2.) Napisz tę metodę w adapterze

public void updateList(List<StockListModel> list) {
stockListModels.clear();
stockListModels.addAll(list);
notifyDataSetChanged();
}

stockListModels -> ta lista jest używana w adapterze.

Dla mnie zadziałało po dodaniu następującego wiersza kodu:

mRecyclerView.setItemAnimator(null);

ten problem może wystąpić podczas próby wyczyszczenia listy, jeśli zamierzasz wyczyścić listę danych, zwłaszcza gdy używasz metody pull do odświeżenia, spróbuj użyć flagi boolean, zainicjuj ją jako false, a wewnątrz metody OnRefresh spraw, aby była prawdziwa, wyczyść swoją listę danych jeśli flaga jest prawdziwa tuż przed dodaniem do niej nowych danych, a następnie ustaw ją jako fałsz.

twój kod może być taki

 private boolean pullToRefreshFlag = false ;
 private ArrayList<your object> dataList ;
 private Adapter adapter ;

 public class myClass extend Fragment implements SwipeRefreshLayout.OnRefreshListener{

 private void requestUpdateList() {

     if (pullToRefresh) {
        dataList.clear
        pullToRefreshFlag = false;
     }

     dataList.addAll(your data);
     adapter.notifyDataSetChanged;


 @Override
 OnRefresh() {
 PullToRefreshFlag = true
 reqUpdateList() ; 
 }

}

Wcześniej miałem ten sam problem. W końcu znalazłem obejście tego problemu

Chcę powiadomić adapter, że element został usunięty, a następnie powiadomić o zmianie zestawu danych adaptera

 public void setData(List<Data> dataList) {
      if (this.dataList.size() > 0) {
          notifyItemRangeRemoved(0, dataList.size());
          this.dataList.clear();
      }
      this.dataList.addAll(dataList)
      notifyItemRangeChanged(0, dataList.size());

 }

Natknąłem się na podobny problem i właśnie go rozgryzłem. Na stałe zapisałem kilka przykładów dla przypadku testowego, ale nie upewniłem się, że każdy zwrócił unikalny identyfikator, co spowodowało dla mnie poniższą awarię. Naprawienie identyfikatorów rozwiązało problem, mam nadzieję, że pomoże to komuś innemu!

raz też dostałem błąd:

Przyczyna: Próbowałem zaktualizować widok Recycler View z zadania Async, jednocześnie próbując odzyskać stare usunięte viewHolders;

Kod: Generuję dane jednym naciśnięciem przycisku, logika jest następująca

1. Wyczyść ostatnie elementy w widoku recyklera
2. Wywołaj zadanie asynchroniczne, aby wygenerować dane
3. OnPostExecute Zaktualizuj widok Recycler i NotifyDataSetChanged

Problem: Ilekroć przewijam szybko przed wygenerowaniem moich danych, dostaję

Wykryto niespójność. Nieprawidłowa pozycja adaptera uchwytu widokuViewHolder java.lang.IndexOutOfBoundsException: Wykryto niespójność. Nieprawidłowa pozycja 20 (przesunięcie: 2). Stan: 3

Rozwiązanie: zamiast wyczyścić RecyclerView przed wygenerowaniem moich danych, zamiast tego zostawiam go, a następnie zastępuję nowymi danymi, Call NotifyDatasetChanged, jak pokazano poniżej;

       @Override
        protected void onPostExecute(List<Objects> o) {
            super.onPostExecute(o);
            recyclerViewAdapter.setList(o);
            mProgressBar.setVisibility(View.GONE);
            mRecyclerView.setVisibility(View.VISIBLE);
        }

Po prostu usuń wszystkie widoki swojego menedżera układu przed powiadomieniem. lubić:

myLayoutmanager.removeAllViews();

Korzystanie z ListAdapter (androidx.recyclerview.widget.ListAdapter)połączenia adapter.submitList(null)przed połączeniem adapter.submitList(list):

adapter.submitList(null)
adapter.submitList(someDataList)

Znalazłem to ustawienie mRecycler.setLayoutFrozen (true); w metodzie onRefresh swipeContainer.

rozwiązał problem dla mnie.

swipeContainer.setOnRefreshListener(new   SwipeRefreshLayout.OnRefreshListener() {
        @Override
        public void onRefresh() {
            orderlistRecycler.setLayoutFrozen(true);
            loadData(false);

        }
    });

To dość paskudny błąd.

Aby obsłużyć kliknięcie elementu, użyłem implementacji RecyclerView.OnItemTouchListenerpodobnej do rozwiązania znalezionego w tym pytaniu .

Po wielokrotnym odświeżeniu RecyclerViewźródła danych i kliknięciu elementu IndexOutOfBoundsExceptionspowoduje to awarię mojej aplikacji. Po kliknięciu elementu RecyclerViewwewnętrzny szuka właściwego widoku podstawowego i przywraca mu pozycję. Sprawdzając kod źródłowy, zauważyłem, że było trochę TasksiThreads zaplanowane. Krótko mówiąc, jest to po prostu jakiś nielegalny stan, w którym dwa źródła danych są ze sobą połączone i nie są zsynchronizowane, a wszystko szaleje.

Na tej podstawie Zdjąłem wdrożenie systemu RecyclerView.OnItemTouchListeneri po prostu złapał kliknij na ViewHolderna Adaptersobie:

public void onBindViewHolder (final BaseContentView holder, final int position) {

    holder.itemView.setOnClickListener(new OnClickListener() {

      @Override
      public void onClick (View view) {

        // do whatever you like here
      }
    });

}

To może nie być najlepsze rozwiązanie, ale na razie bezproblemowe. Mam nadzieję, że pozwoli ci to zaoszczędzić trochę czasu :).

Lint dał mi radę dotyczącą niespójności: napisałem (onBindViewHolder ()):

pholder.mRlayout.setOnClickListener(new View.OnClickListener() {
                    @Override
                    public void onClick(View v) {
                        doStuff(position);
                    }
                });

który musiał zostać zastąpiony przez:

pholder.mRlayout.setOnClickListener(new View.OnClickListener() {
                    @Override
                    public void onClick(View v) {
                        doStuff(pholder.getAdapterPosition());
                    }
                });

Uruchom oba kody w swoim kodzie, a następnie uruchom Lint, aby uzyskać pełne wyjaśnienie !!