通过LiveData观察器将项目添加到特定索引时出现IndexOutOfBoundsException

时间:2019-04-12 08:21:51

标签: java android android-architecture-components android-livedata fragmentstatepageradapter

我正在研究一个使用FragmentStatePagerAdapter作为适配器的ViewPager片段。适配器的数据是一个来自数据库的List对象,我正在用MediatorLiveData观察查询。 MediatorLiveData合并了我在一天中拥有的六个不同列表,并将它们转换为适配器使用的一个列表。

我想将项目添加到列表中的特定索引并动态更新UI。添加项目后,观察者会通知适配器,并且更新工作正常,但是,当我尝试在特定索引上进行操作时,调用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,但是,如果用户添加新的餐,则isResumed更改为true,并且viewPager的当前位置将更改为创建的餐的位置。

但是,由于有addAll(),使用工作代码,创建的Meal的位置将始终位于适配器列表的末尾。我想将餐点添加到特定位置,但是如果我使用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);
}

2 个答案:

答案 0 :(得分:0)

我怀疑是因为插入操作尚未结束,所以出现了问题。

使用Handler进行延迟

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

500毫秒,仅用于测试。

答案 1 :(得分:0)

无论您处于什么位置,都会发生此问题吗?

因为如果您位于适配器的位置0,则适配器尚未创建位于第三位置的片段。

请记住,ViewPagerAdapter和FragmentStatePagerAdapter之间的主要区别之一是后者一次仅创建3个片段,并且如果您位于第一个位置或最后一个位置,则适配器将仅包含2个片段实例。

让我知道此解决方案是否对您有帮助!