RecycleView报错Java.lang.IllegalArgumentException: Called attach on a child which is not detached

最近项目的一个需求，一个银行卡列表页，银行卡初始状态是折叠，点击银行卡展开当前卡片，折叠上一张打开的银行卡，仅此而已，如下图效果：



实现方式有很多种，我实现的一种思路就是：列表用Recyclerview，每个银行卡实体定义一个属性

boolean isopen = true/false //银行卡是否展开

根据这个打开或折叠银行卡，每次点击记录点击的

position

,并将它记录在Adapter的成员变量

int preIndex //上一个点击的item的索引

中，并设置另一个标记

boolean hasItemOpened //当前是否有卡片展开

每次点击时，首先根据当前item的

isopen

属性来执行卡片的折叠或展开动作，然后再通过判断

if (preIndex != position && hasItemOpen)//说明当前点击的不是上一个卡片，并且上一张开片是展开状态

来处理上一张卡片的状态。

当然，这里重点不是如何来实现这个功能，而是在实现这个功能过程中发现的问题：

在处理上一张卡片的折叠动作时，我当时想，只需要改变数据源

isopen

的值，再调用RecyclerView.Adapter的notifyItemChanged（int position）方法即可，但是程序出乎意料的挂了：

错误如下：

E/AndroidRuntime: FATAL EXCEPTION: mainProcess: com.paicaifu.riches, PID: 8502

java.lang.IllegalArgumentException: Called attach on a child which is not detached: ViewHolder{adb21588 position=0 id=-1, oldPos=-1, pLpos:-1}

at android.support.v7.widget.RecyclerView$5.attachViewToParent(RecyclerView.java:654)

at android.support.v7.widget.ChildHelper.attachViewToParent(ChildHelper.java:239)

at android.support.v7.widget.RecyclerView.addAnimatingView(RecyclerView.java:1107)

at android.support.v7.widget.RecyclerView.animateChange(RecyclerView.java:3270)

at android.support.v7.widget.RecyclerView.dispatchLayoutStep3(RecyclerView.java:3088)

at android.support.v7.widget.RecyclerView.dispatchLayout(RecyclerView.java:2917)

at android.support.v7.widget.RecyclerView.onLayout(RecyclerView.java:3283)

…

at android.view.Choreographer.doCallbacks(Choreographer.java:574)

at android.view.Choreographer.doFrame(Ch

也没报哪一行，这个错误也没见过，不过这么多错误，这一行才是最主要的：

Called attach on a child which is not detached: ViewHolder

,它的意思大概就是，操作的这个viewholder当前不是被绑定的（detached），这是为啥呢，经过查询得知，这才是RecyclerView缓存机制啊，未在屏幕上显示的item会被暂时收回，就是detached，所以当调用

notifyItemChanged()

时，如果item不在屏幕中，就会导致这个错误，另外相关的还有一个知识点：

mLayoutManager.getChildCount();

这个方法返回的并不是adapter所有item的数量，而是当前显示的item个数，详见源码：

/**
* Return the current number of child views attached to the parent RecyclerView.
* This does not include child views that were temporarily detached and/or scrapped.
*
* @return Number of attached children
*/
public int getChildCount() {
return mChildHelper != null ? mChildHelper.getChildCount() : 0;
}

好了，了解到这个问题以后，就开始解决吧，那么只需要判断当前position是否在屏幕中显示即可，哦了~不得不说Recyclerview是如此的强大，LayoutManager给我们提供了应有尽有的方法来判断position是否在屏幕中，通过LayoutManager我们可以获得当前屏幕显示的第一个item位置，以及最后一个item位置，这个位置是整个list中的位置 ，当满足

position<=lastindex && position>=firstIndex

时，item即是现在在屏幕中，好了，这下可以安心的使用

notifyItemChanged()

了，但是令人崩溃的是，程序又挂了，问题还是

Called attach on a child which is not detached

， 我的天，这次又是为何，经过debug发现，我的RecyclerView可以下啦刷新，position = 0的位置，是下拉刷新头，这个view肯定没有绑定ViewHolder啊，所以导致了以上的错误，好了，问题发现了，就解决吧，根据存储到成员变量

preindex

中的值来计算要需要更新item的位置：

int updatePos = preIndex - firstVisibleItemPosition+1

， Ok，计算出了正确位置便可以通过以下方式更改上一个卡片的状态了：

int firstVisibleItemPosition = mLayoutManager.findFirstVisibleItemPosition();
int lastVisibleItemPosition = mLayoutManager.findLastVisibleItemPosition();
if (preIndex-firstVisibleItemPosition>=0&&preIndex<=lastVisibleItemPosition){
//获取上一个卡片的view
View pre = mLayoutManager.getChildAt(preIndex - firstVisibleItemPosition+1);
closeCard(pre);//折叠卡片
}

OK，完成。虽然是很普通的一个功能，但是踩坑n多啊，总结一下，给自己也给各位android朋友一个提醒。如有错误之处，请不吝赐教，多多交流！

QQ：294958260