IndexOutOfBoundsException при добавлении элементов к определенному индексу с помощью обозревателя LiveData

1

Я работаю над фрагментом ViewPager, который использует FragmentStatePagerAdapter в качестве адаптера. Данные адаптера - это объект List, который поступает из базы данных, и я наблюдаю за запросом с помощью MediatorLiveData. MediatorLiveData объединяет шесть разных списков, которые у меня есть на один день, и превращает их в один список, который используется адаптером.

Я хочу добавить элемент в список для определенного индекса и динамически обновлять пользовательский интерфейс. Наблюдатель уведомляет адаптер, когда элемент добавляется и обновление работает нормально, однако, когда я пытаюсь сделать это для определенного индекса, вызов notifyDataSetChanged() вызывает IndexOutOfBoundsError.

Я инициализирую адаптер с помощью следующего списка:

public MealDetailsPagerAdapter(FragmentManager fm, List<Long> list, long date, MealViewModel vm) {
    super(fm);
    this.mealIdList = list;
    this.date = date;
    this.model = vm;
}

Если MealViewModel не имеет отношения к этому вопросу, он используется для фрагмента, который создает адаптер. Изменения списка выполняются другой моделью представления.

Вот код, который работает правильно:

public void changeItems(List<Long> list, long date) {
    if(this.date != date){
        this.mealIdList = list;
        notifyDataSetChanged();
        return;
    }
    mealIdList.addAll(list);
    mealIdList = removeDuplicates(mealIdList);
    System.out.println(Arrays.toString(mealIdList.toArray()));
    notifyDataSetChanged();
}

И наблюдатель, который называет это:

 @Override
public void onActivityCreated(@Nullable Bundle savedInstanceState) {
    super.onActivityCreated(savedInstanceState);
    mViewModel.getMealIdsInDay().observe(getViewLifecycleOwner(), longs -> {
        myAdapter.changeItems(longs, mCurrentIndexDay);
        if(isResumed){
            mViewPager.setCurrentItem(myAdapter.getPageIndexForMealId(mViewModel.getMealId()));
            isResumed=false;
        }
        updateIndicator(mViewPager);
    });
}

Где isResumed имеет значение false по умолчанию, однако, если пользователь добавляет новую Meal, isResumed изменяется на true, и текущая позиция viewPager изменяется на созданную позицию Meal.

Однако с рабочим кодом созданная позиция Meal всегда будет в конце списка адаптеров из-за addAll(). Я хочу добавить блюдо в определенную позицию, но если я получу индекс с помощью mViewPager.getCurrentItem() и отправлю его в метод следующим образом:

    mealIdList.addAll(index, list);

сам addAll работает, но notifyDataSetChanged() вызывает IndexOutOfBoundsError.

Вот полный след стека:

    E/AndroidRuntime: FATAL EXCEPTION: main
Process: fi.seehowyoueat.shye.debug, PID: 14148
java.lang.IndexOutOfBoundsException: Index: 2, Size: 2
    at java.util.ArrayList.set(ArrayList.java:453)
    at androidx.fragment.app.FragmentStatePagerAdapter.destroyItem(FragmentStatePagerAdapter.java:147)
    at androidx.viewpager.widget.ViewPager.populate(ViewPager.java:1212)
    at androidx.viewpager.widget.ViewPager.setCurrentItemInternal(ViewPager.java:669)
    at androidx.viewpager.widget.ViewPager.setCurrentItemInternal(ViewPager.java:631)
    at androidx.viewpager.widget.ViewPager.dataSetChanged(ViewPager.java:1086)
    at androidx.viewpager.widget.ViewPager$PagerObserver.onChanged(ViewPager.java:3097)
    at androidx.viewpager.widget.PagerAdapter.notifyDataSetChanged(PagerAdapter.java:291)
    at fi.octo3.shye.view.viewpagers.MealDetailsPagerAdapter.changeTheItems(MealDetailsPagerAdapter.java:87)
    at fi.octo3.shye.fragments.MealDetailsFragment.lambda$onActivityCreated$0(MealDetailsFragment.java:223)
    at fi.octo3.shye.fragments.-$$Lambda$MealDetailsFragment$XB4Svnx84FE6kVa5Gzle01e8F3o.onChanged(Unknown Source:4)
    at androidx.lifecycle.LiveData.considerNotify(LiveData.java:131)
    at androidx.lifecycle.LiveData.dispatchingValue(LiveData.java:149)
    at androidx.lifecycle.LiveData.setValue(LiveData.java:307)
    at androidx.lifecycle.MutableLiveData.setValue(MutableLiveData.java:50)
    at fi.octo3.shye.models.viewmodel.-$$Lambda$2CouvY7DQv4NA0nk6EMoH6jUavw.onChanged(Unknown Source:4)
    at androidx.lifecycle.MediatorLiveData$Source.onChanged(MediatorLiveData.java:152)
    at androidx.lifecycle.LiveData.considerNotify(LiveData.java:131)
    at androidx.lifecycle.LiveData.dispatchingValue(LiveData.java:149)
    at androidx.lifecycle.LiveData.setValue(LiveData.java:307)
    at androidx.lifecycle.LiveData$1.run(LiveData.java:91)
    at android.os.Handler.handleCallback(Handler.java:873)
    at android.os.Handler.dispatchMessage(Handler.java:99)
    at android.os.Looper.loop(Looper.java:280)
    at android.app.ActivityThread.main(ActivityThread.java:6748)
    at java.lang.reflect.Method.invoke(Native Method)
    at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:493)
    at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:858)

Глядя на это, кажется, что проблема как-то вызвана LiveData, но я не уверен, как это исправить. Если вы хотите посмотреть на метод, который обновляет MediatorLiveData, он у меня здесь:

 private void refresh(long key){
    if(groupsAndMealsMap.get(key) != null){
        //remove source and then value from map
        liveDataMerger.removeSource(groupsAndMealsMap.get(key));
        groupsAndMealsMap.remove(key);
        //add new value to map and add it as a source
        groupsAndMealsMap.append(key, mealIdsInGroup);
        liveDataMerger.addSource(groupsAndMealsMap.get(key), liveDataMerger::setValue);
    }
}

Функция refresh() вызывается методом addItem(), который получает обновленный список блюд из базы данных (список - едыIdsInGroup), а liveDataMerger состоит из шести объектов LiveData>.

Любая помощь будет оценена. Спасибо!

РЕДАКТИРОВАТЬ

Здесь метод addItem(), как вы можете видеть, исполнитель службы ожидает выполнения операции, прежде чем перейти к методу refresh().

public void addItem(Meal meal, long mealGroupId){
    ExecutorService service = Executors.newCachedThreadPool();
    service.execute(() -> {
        setMealId(db.mealDao().insertMealIntoGroup(meal, mealGroupId));
        mealIdsInGroup = db.mealDao().loadMealsWithinGroup(mealGroupId);
    });
    service.shutdown();
    try {
        service.awaitTermination(Long.MAX_VALUE, TimeUnit.NANOSECONDS);
    } catch (InterruptedException e) {
        e.printStackTrace();
    }

    refresh(mealGroupId);
}
Теги:
android-architecture-components
android-livedata
fragmentstatepageradapter

2 ответа

0

Эта проблема возникает независимо от того, в какой позиции вы находитесь?

Потому что, если вы находитесь в позиции 0 адаптера, адаптер еще не создал фрагмент, который находится в третьей позиции.

Помните, что одно из основных различий между ViewPagerAdapter и FragmentStatePagerAdapter заключается в том, что последний создает только 3 фрагмента за раз, и если вы находитесь на первой позиции или на последней, адаптер будет содержать только 2 экземпляра фрагментов.

Дайте мне знать, помогло ли вам это решение!

  • 0
    Это происходит, если позиция меньше, чем последний элемент в списке. Я заявил в вопросе, что AddAll () без указанного индекса работает полностью нормально, поэтому добавление в конец списка работает, даже если индекс указан ... Кроме того, что такое ViewPagerAdapter? Это загружаемая библиотека?
  • 0
    Извините, я имел в виду PagerAdapter. Да, это имеет смысл. Дело в том, что addAll () будет добавлять данные к каждому фрагменту, который в данный момент существует в этом адаптере. Например: если количество адаптеров равно 10, и вы в настоящее время находитесь в позиции 1 (фрагмент 2), если вы добавите addAll (index = 3, list), он выдаст вам IndexOutOfBoundsError, поскольку адаптер еще не создал этот фрагмент. Это значит, что в этот момент ваш адаптер имеет только фрагменты 1, 2 и 3.
Показать ещё 2 комментария
0

Я подозреваю, что проблема возникает, потому что операция вставки еще не закончена.

Используйте Handler чтобы поставить задержку

       new Handler(getMainLooper()).postDelayed(new Runnable() {
            @Override
            public void run() {
                notifyDataSetChanged();
            }
        }, 500);

500 мс только для тестирования.

  • 0
    Вставка завершена, иначе у метода addItem не было бы списка для добавления. В моем методе addItem () я запускаю ExecutorService, чтобы выполнить операции вставки и выбора в БД, а затем выполнить цикл try-catch, чтобы дождаться завершения службы.
  • 0
    Попробовал это в любом случае только сейчас, получил исключение: PagerAdapter приложения изменил содержимое адаптера без вызова PagerAdapter # notifyDataSetChanged! Ожидаемое количество элементов адаптера: 2, найдено: 3
Показать ещё 2 комментария

Ещё вопросы

Сообщество Overcoder
Наверх
Меню