问题介绍
Android应用在bugly上偶现调用View.draw()时出现数据越界,根据bugly提供的解决方案如下,
1.遍历数组/字符串等集合前,要判断遍历对象的长度;2.操作数组/字符串等集合前,要检查角标是否在长度允许范围内;3.ListView操作不当也会引起该异常,这种情况下一般是由于List渲染的时候,外面的数据源发生变化导致的。举例如ListView滚动时点击刷新将会报错,解决方法是ListView滚动时将刷新置为不可点击。或者改变数据源之前调用adapter的notifyDataSetInvalidated()方法将原数据源设置为无效。
但由于无项目源码相关的堆栈提示信息,此问题一直被搁置。
分析思路
出错堆栈信息如下
1 java.util.ArrayList.throwIndexOutOfBoundsException(ArrayList.java:255)2 java.util.ArrayList.get(ArrayList.java:308)3 android.widget.HeaderViewListAdapter.isEnabled(HeaderViewListAdapter.java:164)4 android.widget.ListView.dispatchDraw(ListView.java:3329)5 android.view.View.draw(View.java:16206)6 android.widget.AbsListView.draw(AbsListView.java:4166)7 android.view.View.updateDisplayListIfDirty(View.java:15200)8 android.view.View.draw(View.java:15973)//...49 android.view.ViewRootImpl.draw(ViewRootImpl.java:2615)50 android.view.ViewRootImpl.performDraw(ViewRootImpl.java:2434)51 android.view.ViewRootImpl.performTraversals(ViewRootImpl.java:2067)52 android.view.ViewRootImpl.doTraversal(ViewRootImpl.java:1107)53 android.view.ViewRootImpl$TraversalRunnable.run(ViewRootImpl.java:6013)//...60 android.os.Looper.loop(Looper.java:148)61 android.app.ActivityThread.main(ActivityThread.java:5417)62 java.lang.reflect.Method.invoke(Native Method)63 com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:726)64 com.android.internal.os.ZygoteInit.main(ZygoteInit.java:616)
这里重点关注第3~4行信息内容,根据第3行可知是ListView的dispatchDraw()方法异常
通过第3行知道是HeaderViewListAdapter.isEnabled方法抛出的查询Android的ListView源码可知,public void setAdapter(ListAdapter adapter) { if (mAdapter != null && mDataSetObserver != null) { mAdapter.unregisterDataSetObserver(mDataSetObserver); } resetList(); mRecycler.clear(); if (mHeaderViewInfos.size() > 0|| mFooterViewInfos.size() > 0) { //注意这里:只有调用addHeaderView或者addFooterView时候才会构建HeaderViewListAdapter mAdapter = wrapHeaderListAdapterInternal(mHeaderViewInfos, mFooterViewInfos, adapter); } else { mAdapter = adapter; } mOldSelectedPosition = INVALID_POSITION; mOldSelectedRowId = INVALID_ROW_ID; // AbsListView#setAdapter will update choice mode states. super.setAdapter(adapter); //…省略}
根据HeaderViewListAdapter类及堆栈信息可知出问题的肯定是ListView且有调用addHeaderView或者addFooterView的地方
补充说明:1.Bugly的”跟踪数据”模块也能帮忙排除某些页面,大致定位出错的地方2.注意有些页面是针对GridView或者RecyclerView自定义的add(Head/Foot)erView的地方,根据是ListView类报错也可以忽略分析ListView.dispatchDraw方法
@Overrideprotected void dispatchDraw(Canvas canvas) { final int count = getChildCount(); if (!mStackFromBottom) { for (int i = 0; i < count; i++) { if (adapter.isEnabled(itemIndex) && (headerDividers || !isHeader && (nextIndex >= headerCount)) && (isLastItem || adapter.isEnabled(nextIndex) && (footerDividers || !isFooter && (nextIndex < footerLimit)))) { //... } } } } } } // Draw the indicators (these should be drawn above the dividers) and children super.dispatchDraw(canvas);}
可以看到,dispatchDraw是通过getChildCount去遍历
分析HeaderViewListAdapter.isEnable()方法
public boolean isEnabled(int position) { // Header (negative positions will throw an IndexOutOfBoundsException) int numHeaders = getHeadersCount(); if (position < numHeaders) { return mHeaderViewInfos.get(position).isSelectable; } // Adapter final int adjPosition = position - numHeaders; int adapterCount = 0; if (mAdapter != null) { adapterCount = mAdapter.getCount(); if (adjPosition < adapterCount) { return mAdapter.isEnabled(adjPosition); } } // Footer (off-limits positions will throw an IndexOutOfBoundsException) return mFooterViewInfos.get(adjPosition - adapterCount).isSelectable;}
HeaderViewListAdapter.isEnable的方法实现很简单,重点关注该方法使用的count=mAdapter.getCount()
结合上面两个方法的分析可以想到,isEnable()方法触发”java.lang.IndexOutOfBoundsException”只有两种可能:
1.调用remove(Header/Footer)View方法后没有及时触发更新adapter更新2.数据源数据减少(不只有remove这一种可能,也有可能是赋值size更小的集合引用)后没有及时更新结合案例分析
项目全局搜索addFooterView|addHeaderView,排除无关结果后如下:
1.“MenuFragment”中menuClsAdapter.modules有remove、clear或者集合重新赋值的地方均有显式调用notifyDataSetChanged方法通知ListView去更新2.“AirTableFragment”中检查tableViewProcessor.mareaDBModelList在AirTableFragment中有remove、clear或者变更引用的地方2.1 doDeleteArea:该方法中可能有问题,一开始以为是clear之后调用selectionArea方法中在notifyDataSetChanged之前有耗时操作,检查了下代码,排除该猜想2.2 会不会是tableViewProcessor.mareaDBModelList在非AirTableFragment页面中被修改?检查发现loadLocalDatas()确实有传递tableViewProcessor对象引用且内部有变更mareaDBModelList数据。PS:问题并非仅仅如此,具体原因与公司封装的base库相关,就不深入介绍了。