安卓使用RecyclerView的notifyItemInserted()的坑

Posted 2022-01-03 心脏dance

大多数新手经常会碰到：

java.lang.IndexOutOfBoundsException: Inconsistency detected. Invalid view holder adapter positionViewHolder**** position=13 id=-1, oldPos=1 ******

我是在这样的场景下出现的这个问题：

scrollToPosition(0) // 快速定位到列表顶部
notifyItemInserted(0) // 插入一条数据

为什么会出现呢？
        是因为在你的 adapter 的 item 改变后，没有及时 notifyItemInserted() 的事情，导致 adapter 里面的数据，和 recyclerView 里面已有的数据出现不一致，就会导致这个问题出现 （此时你的adapter 和 recyclerView 一定总是持有同一个 List ）。比如你是用liveData来监听数据的变化，你需要先notify再给adapter赋值，但是我们大多数情况下都是先赋值再notify。正常我们调用 notifyDataSetChanged() 或者 notifyItemRangeChanged() 是不会发生闪退的，但是当我们调用 notifyItemInserted() 这个方法的时候就会出现上述这个奇葩的问题。

怎么解决呢？

        网上大部分的解决方法是catch住这个错误，从而让App不会crash，但是这种方法很明显并没有真正解决这个问题，只是把问题给catch住了，并没有去处理它。

网上的方案：

class DataCheckLinearLayoutManager(
    context: Context?
) : LinearLayoutManager(context) 

    override fun onLayoutChildren(recycler: RecyclerView.Recycler?, state: RecyclerView.State?) 
        try 
            super.onLayoutChildren(recycler, state)
         catch (e: IndexOutOfBoundsException) 
            ALog.e("DataCheckLinear", e.message)
        
    

        继承 LinearLayoutManager 重写 onLayoutChildren() 方法，去catch住越界的问题。这样写功能上是可以实现的，但是仔细看我们的列表的话，会有一丢丢闪一下的效果。这样就会影响体验。

我认为的正确解决方案：（因为在我的项目里面是可以的）

        我们在给 adpater 的 item 赋值的时候，我们创建一个新的 List ，然后把旧的 List 里面的数据copy进去，这样 adapter 里面的数据和 recyclerView 里面的数据就不是同一个对象了（这里的对象说的是 List 这个对象）这样上面数组越界的问题就解决了。（保证不持有同一个 List 对象，就不会出现数据不一致的情况，因为 List 都不一样了，（数据不一致，指的是同一个 List 对象里面的数据不一致））

总结：当 recyclerView 和 adapter 总是持有同一个对象 List 的时候，最好是保证 adapter 和recyclerView 里面的数据是一致的。建议使用的时候，每次创造一个新的 List 来存储数据（在不影响性能的情况下～）