【问题标题】:IndexOutOfBoundsException when adding items to specific index through LiveData observer通过 LiveData 观察者将项目添加到特定索引时出现 IndexOutOfBoundsException
【发布时间】:2019-04-12 08:21:50
【问题描述】:

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

我想将一个项目添加到 List 到特定索引并动态更新 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() 调用,它从数据库中获取更新的膳食列表(列表为 mealIdsInGroup),并且 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);
}

【问题讨论】:

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


    【解决方案1】:

    我怀疑这个问题是因为插入操作还没有结束。

    使用Handler 延迟

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

    500 ms 仅用于测试。

    【讨论】:

    • 插入结束,否则 addItem 方法将没有要添加的列表。在我的 addItem() 方法中,我正在运行 ExecutorService 以在数据库中执行插入和选择操作,然后使用 try-catch 循环等待服务终止。
    • 刚才试过了,出现异常:应用程序的PagerAdapter更改了适配器的内容,而没有调用PagerAdapter#notifyDataSetChanged!预期的适配器项目数:2,找到:3
    • addAll 之后,removeDuplicates 之前调用notifyDataSetChanged() 怎么样。如果使用 Handler() 放一小段时间,例如 50ms.Same 结果?
    • 同样的结果,是的。
    【解决方案2】:

    无论您处于什么职位,都会出现这个问题吗?

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

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

    如果此解决方案对您有帮助,请告诉我!

    【讨论】:

    • 如果位置小于列表中的最后一项,就会发生这种情况。我在问题中指出,没有指定索引的 AddAll() 完全可以正常工作,因此即使指定了索引,添加到列表末尾也可以工作......另外,什么是 ViewPagerAdapter?它是一个可下载的库吗?
    • 对不起,我的意思是 PagerAdapter。是的,这是有道理的。问题是,addAll() 会将数据添加到该适配器中当前存在的每个片段中。例如:如果您的适配器计数为 10 并且您当前位于位置 1 (fragment 2) ,如果您执行 addAll(index = 3, list) 它将给您 IndexOutOfBoundsError 因为适配器尚未创建该片段。也就是说,在那一刻,您的适配器只有片段 1、2 和 3。
    • 适配器知道计数比这个大,但出于性能原因(这就是为什么 FragmentStatePagerAdapter 在有很多片段时很棒的原因)它最多只创建 2 或 3 个。
    • 但问题是,我没有尝试添加到不存在的索引。通过调试,我看到 addAll(index, list) 行中的 index 为 1 而 size 为 3。它正确地完成了该行,错误来自 notifyDataSetChanged() 本身!我在那条线上放了一个断点,然后崩溃发生了。
    猜你喜欢
    • 2018-06-05
    • 2020-09-03
    • 1970-01-01
    • 2020-11-11
    • 1970-01-01
    • 1970-01-01
    • 2014-06-10
    • 1970-01-01
    • 1970-01-01
    相关资源
    最近更新 更多