前些日子在github上遇到了一个有趣的问题，是在一个叫“xRecyclerView”的项目里的issue，一哥们说他给列表加两个head就会报错，而我则不会。随后又有两个哥们表示他们也有同样的问题：
java.lang.IllegalArgumentException: called detach on an already detached child ViewHolder{11ae0c3d position=2 id=-1, oldPos=-1, pLpos:-1 scrap [attachedScrap] tmpDetached no parent}...
于是我贴上了我的代码：
View view1 = LayoutInflater.from(activity).inflate(layoutId1, null);
View view2 = LayoutInflater.from(activity).inflate(layoutId2, null);
xRecyclerView.addHeaderView(view1);
xRecyclerView.addHeaderView(view2);
而其中两个哥们的错误代码如下：
headerRecommend=LayoutInflater.from(getActivity()).inflate(R.layout.fragment_find_item_recommend, (ViewGroup) mView.findViewById(android.R.id.content), false);
headerRank=LayoutInflater.from(getActivity()).inflate(R.layout.fragment_find_item_rank, (ViewGroup) mView.findViewById(android.R.id.content), false);
rvTipsRecommendList.addHeaderView(headerRecommend);
rvTipsRecommendList.addHeaderView(headerRank);
headerview = new CustomizeHeaderLayout(this);
headerview1 = new CustomizeHeader1Layout(this);
headerview2 = new CustomizeHeader2Layout(this);
listview.addHeaderView(headerview);
listview.addHeaderView(headerview1);
listview.addHeaderView(headerview2);
public CustomizeHeaderLayout(Context context) {
super(context);
init();
}
private void init() {
LayoutInflater inflater = (LayoutInflater)getContext().getSystemService(Context.LAYOUT_INFLATER_SERVICE);
View view = inflater.inflate(R.layout.layout_customize_headview, null);
...
}
先看源码，recyclerView和XRecyclerView的源码。在recyclerView的源码里抛出这个异常的地方是这样判断的：
public void detachViewFromParent(int offset) {
if (vh != null) {
if (vh.isTmpDetached() && !vh.shouldIgnore()) {
throw new IllegalArgumentException("called detach on an already"
+ " detached child " + vh);
}...
vh.addFlags(ViewHolder.FLAG_TMP_DETACHED);
}
}
这个vh.isTmpDetached()和!vh.shouldIgnore()是一些c里面的二进制运算，先放在一边。我们回过头看log，可以看到vh对象打印出的东西有点不一样，作者应该是重写了toString，一找果然是，
@Override public String toString() { final StringBuilder sb = new StringBuilder("ViewHolder{" + Integer.toHexString(hashCode()) + " position=" + mPosition + "id="+ mItemId + ", oldPos=" + mOldPosition + ", pLpos:" + mPreLayoutPosition); if (isScrap()) { sb.append(" scrap ") .append(mInChangeScrap ? "[changeScrap]" : "[attachedScrap]"); } if (isInvalid()) sb.append(" invalid"); if (!isBound()) sb.append(" unbound"); if (needsUpdate()) sb.append(" update"); if (isRemoved()) sb.append(" removed"); if (shouldIgnore()) sb.append(" ignored"); if (isTmpDetached()) sb.append(" tmpDetached"); if (!isRecyclable()) sb.append(" not recyclable(" + mIsRecyclableCount + ")"); if (isAdapterPositionUnknown()) sb.append(" undefined adapter position"); if (itemView.getParent() == null) sb.append(" no parent"); sb.append("}"); return sb.toString(); }
我的正常的log打出的是
ViewHolder{335f5b75 position=0 id=-1, oldPos=-1, pLpos:-1 no parent} ViewHolder{1e435ef3 position=1 id=-1, oldPos=-1, pLpos:-1 no parent} ViewHolder{f598161 position=2 id=-1, oldPos=-1, pLpos:-1 no parent}
对比可以得出结论是，我的在isScrap()和isTmpDetached()这两个判断是始终没有走的，而他们两个都走到这个判断里面去了。那就是这个isScrap()和isTmpDetached()里有问题！再结合上文判断异常部分的代码，isTmpDetached()这个方法显得诡异了许多。
通过我的有道神器，知道了Detached是‘分离’的意思，那detachViewFromParent就可以理解为‘从父view中分离子view’吧，如果不报错，还会addFlags(ViewHolder.FLAG_TMP_DETACHED)，虽然我看不懂大神的二进制算法，但是现在我好像可以从字里行间里隐约感觉到是怎么回事了。那就是在调用了detachViewFromParent之后，使isTmpDetached()成立，又一次调用了这个方法，就报了这个错误！想明白的瞬间背上似乎有了层凉气，在凌晨2点的夜里还是蛮吓人的...但我还是不明白detachViewFromParent的调用机制，归根结底我还是没有明白这个bug出现的原因，只能明天再做调查了。
ps：有个挪威小哥也有同样的问题：
https://github.com/martijnvdwoude/recycler-view-merge-adapter/issues/4
但我没有看太懂
——2017-3-1 2:32
