RecyclerView с DiffUtil (ListAdapter) блокирует поток пользовательского интерфейса - PullRequest
0 голосов
/ 20 марта 2019

Я использую RecyclerView с ListAdapter (который использует AsyncListDiffer для вычисления и анимации изменений при замене списка).

Проблема в том, что иногда, когда я вызываю submit(), чтобы обновить RecyclerView новыми данными, он блокирует весь поток пользовательского интерфейса на 5-10 секунд. Это происходит после того, как я возобновляю действие (возвращаясь с главного экрана), но я не уверен, является ли это причиной.

Я убедился, что

  • представленный список содержит хорошие, ожидаемые данные
  • используется recyclerView.setHasFixedSize(true);, поскольку занимает все пространство экрана

Это дамп потока, когда устройство находится в этом неотвечающем состоянии ( см. Raw ):

"main@10549" prio=5 runnable
  java.lang.Thread.State: RUNNABLE
      at java.util.ArrayList.get(ArrayList.java:315)
      at android.support.v7.widget.OpReorderer.getLastMoveOutOfOrder(OpReorderer.java:215)
      at android.support.v7.widget.OpReorderer.reorderOps(OpReorderer.java:33)
      at android.support.v7.widget.AdapterHelper.preProcess(AdapterHelper.java:93)
      at android.support.v7.widget.RecyclerView.processAdapterUpdatesAndSetAnimationFlags(RecyclerView.java:3583)
      at android.support.v7.widget.RecyclerView.dispatchLayoutStep1(RecyclerView.java:3829)
      at android.support.v7.widget.RecyclerView.dispatchLayout(RecyclerView.java:3639)
      at android.support.v7.widget.RecyclerView.onLayout(RecyclerView.java:4194)
      at android.view.View.layout(View.java:16967)
      at android.view.ViewGroup.layout(ViewGroup.java:5584)
      at android.support.v4.widget.SwipeRefreshLayout.onLayout(SwipeRefreshLayout.java:625)
      at android.view.View.layout(View.java:16967)
      at android.view.ViewGroup.layout(ViewGroup.java:5584)
      at android.widget.LinearLayout.setChildFrame(LinearLayout.java:2001)
      at android.widget.LinearLayout.layoutVertical(LinearLayout.java:1844)
      at android.widget.LinearLayout.onLayout(LinearLayout.java:1753)
      at android.view.View.layout(View.java:16967)
      at android.view.ViewGroup.layout(ViewGroup.java:5584)
      at android.widget.FrameLayout.layoutChildren(FrameLayout.java:396)
      at android.widget.FrameLayout.onLayout(FrameLayout.java:333)
      at android.view.View.layout(View.java:16967)
      at android.view.ViewGroup.layout(ViewGroup.java:5584)
      at android.widget.RelativeLayout.onLayout(RelativeLayout.java:1189)
      at android.view.View.layout(View.java:16967)
      at android.view.ViewGroup.layout(ViewGroup.java:5584)
      at android.support.design.widget.HeaderScrollingViewBehavior.layoutChild(HeaderScrollingViewBehavior.java:142)
      at android.support.design.widget.ViewOffsetBehavior.onLayoutChild(ViewOffsetBehavior.java:41)
      at android.support.design.widget.AppBarLayout$ScrollingViewBehavior.onLayoutChild(AppBarLayout.java:1556)
      at android.support.design.widget.CoordinatorLayout.onLayout(CoordinatorLayout.java:888)
      at android.view.View.layout(View.java:16967)
      at android.view.ViewGroup.layout(ViewGroup.java:5584)
      at android.support.v4.widget.DrawerLayout.onLayout(DrawerLayout.java:1231)
      at android.view.View.layout(View.java:16967)
      at android.view.ViewGroup.layout(ViewGroup.java:5584)
      at android.widget.FrameLayout.layoutChildren(FrameLayout.java:396)
      at android.widget.FrameLayout.onLayout(FrameLayout.java:333)
      at android.view.View.layout(View.java:16967)
      at android.view.ViewGroup.layout(ViewGroup.java:5584)
      at android.widget.FrameLayout.layoutChildren(FrameLayout.java:396)
      at android.widget.FrameLayout.onLayout(FrameLayout.java:333)
      at android.view.View.layout(View.java:16967)
      at android.view.ViewGroup.layout(ViewGroup.java:5584)
      at android.widget.FrameLayout.layoutChildren(FrameLayout.java:396)
      at android.widget.FrameLayout.onLayout(FrameLayout.java:333)
      at android.view.View.layout(View.java:16967)
      at android.view.ViewGroup.layout(ViewGroup.java:5584)
      at android.widget.LinearLayout.setChildFrame(LinearLayout.java:2001)
      at android.widget.LinearLayout.layoutVertical(LinearLayout.java:1844)
      at android.widget.LinearLayout.onLayout(LinearLayout.java:1753)
      at android.view.View.layout(View.java:16967)
      at android.view.ViewGroup.layout(ViewGroup.java:5584)
      at android.widget.FrameLayout.layoutChildren(FrameLayout.java:396)
      at android.widget.FrameLayout.onLayout(FrameLayout.java:333)
      at com.android.internal.policy.PhoneWindow$DecorView.onLayout(PhoneWindow.java:2769)
      at android.view.View.layout(View.java:16967)
      at android.view.ViewGroup.layout(ViewGroup.java:5584)
      at android.view.ViewRootImpl.performLayout(ViewRootImpl.java:2547)
      at android.view.ViewRootImpl.performTraversals(ViewRootImpl.java:2250)
      at android.view.ViewRootImpl.doTraversal(ViewRootImpl.java:1321)
      at android.view.ViewRootImpl$TraversalRunnable.run(ViewRootImpl.java:6732)
      at android.view.Choreographer$CallbackRecord.run(Choreographer.java:894)
      at android.view.Choreographer.doCallbacks(Choreographer.java:696)
      at android.view.Choreographer.doFrame(Choreographer.java:631)
      at android.view.Choreographer$FrameDisplayEventReceiver.run(Choreographer.java:880)
      at android.os.Handler.handleCallback(Handler.java:815)
      at android.os.Handler.dispatchMessage(Handler.java:104)
      at android.os.Looper.loop(Looper.java:207)
      at android.app.ActivityThread.main(ActivityThread.java:5740)
      at java.lang.reflect.Method.invoke(Method.java:-1)
      at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:905)
      at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:766)

Вот мой обратный вызов DiffUtil:

    private static final DiffUtil.ItemCallback<CoinTicker> DIFF_CALLBACK = new DiffUtil.ItemCallback<CoinTicker>() {
        @Override
        public boolean areItemsTheSame(
                @NonNull CoinTicker oldTicker, @NonNull CoinTicker newTicker) {
            // User properties may have changed if reloaded from the DB, but ID is fixed
            return oldTicker.getSlug().equals(newTicker.getSlug()); // String
        }
        @Override
        public boolean areContentsTheSame(
                @NonNull CoinTicker oldTicker, @NonNull CoinTicker newTicker) {
            // NOTE: if you use equals, your object must properly override Object#equals()
            // Incorrectly returning false here will result in too many animations.
            return oldTicker.equals(newTicker); // see below
        }

        @Override
        public Object getChangePayload(@NonNull CoinTicker oldTicker, @NonNull CoinTicker newTicker) {
            return newTicker.getComparisonPrice() - oldTicker.getComparisonPrice();
        }
    };


    // equals for CoinTicker:
    @Override
    public boolean equals(Object o) {
        if (this == o) return true;
        if (o == null || getClass() != o.getClass()) return false;
        CoinTicker that = (CoinTicker) o;
        return Double.compare(that.price, price) == 0 && slug.equals(that.slug);
    }

Что здесь может быть не так?

Ответы [ 3 ]

0 голосов
/ 20 марта 2019

В этом разделе

public boolean areContentsTheSame(
            @NonNull CoinTicker oldTicker, @NonNull CoinTicker newTicker) {
        // NOTE: if you use equals, your object must properly override Object#equals()
        // Incorrectly returning false here will result in too many animations.
        return oldTicker.equals(newTicker); // see below
    }

отсутствует (oldTicker.getSomething (). Equals (newTicker.getSomething ()) или попробуйте вернуть oldTicker.getSomething () == newTickers.getSomething ()

если в строке сравнения используется «равно», если целое или двойное значение «==»

0 голосов
/ 26 марта 2019

Рад сказать, что "исправить" очень легко:

recyclerView.setItemAnimator(null);

Мне нет дела до анимации.Я просто хотел цветную анимацию на элементах, которые отображаются на экране, что я делаю в onBindViewHolder (), и она работает очень быстро без задержек!

0 голосов
/ 20 марта 2019

Сравнение двух списков - дорогостоящая операция. Если вы имеете дело с огромным количеством данных, он может использовать много ресурсов. Поэтому запустите ваши операции DiffUtils в фоновом потоке или используйте AsyncListDiffer , который работает в фоновом потоке сам по себе.

...