关于下拉刷新这件事,无论是普通用户还是开发者都再熟悉不过了,过去的某段时间无论下拉刷新的设计还是开源控件都异常火爆,火爆到惊动了党中央(Google),所以党中央就自己在支持包里面增加了一个下拉刷新的控件——SwipeRefreshLayout,这个下拉的效果非常的别致,堪称低调的华丽,先随便来张图吧,相信大家都不陌生。至于SwipeRefreshLayout怎么用就不讲了,不知道的自觉面壁去。
Swiperefreshlayout.gif
那作为Material Design的坚决拥护者,实战中我也大刀阔斧的用起了SwipeRefreshLayout,简单的写了个BaseSwipeRefreshLayout类初始化了一些基本属性,因为本身就很好用,所以也没做太多封装。有需要用到下拉刷新的,就直接在布局里引用这个Base类,但没想到却因此埋了一个坑,引发了一场血案。当然并不是说Base类本身代码有什么错误,而是因为一些奇妙的组合引起了一些狗血的bug,且听我继续往下八。
实际的开发中,基本上Activity、Fragment都用上了这个下拉刷新。我的首页是一个Activity通过ViewPager维护4个Fragment的这种经典设计,其中第一页和第二页有用到下拉刷新,Fragment采用懒加载的方式,关于懒加载可以看我的另一篇文章 ViewPager+Fragment LazyLoad最优解。这代码绝壁不会有问题,一切看起来都是那么美好!飘柔,就是这么自信。然而忽然有一天发现了一个无法解释的现象:当我启动App停留在第一页的时候,即便静止不动,cpu利用率还是很高,而且非常线性,几乎没什么波动。切到第二页,数据加载出来后,cpu利用率立马就下去了。我当时就呵呵了。
于是乎开始排查优化,经过简单的分析,基本上可以确定问题出在第二页上,不巧的是我的第二页布局炒鸡复杂,顶部是个自动轮播大图,然后是两个横向的RecyclerView,最后还有一个纵向的RecyclerView,当然这中间还嵌套夹杂着一些小的视图。再来分析下问题:进入App停在首页,因为懒加载,所以第二页的View已经初始化完成,但是还没有loadData,这个时候cpu利用率很高,再切到第二页loadData完成,cpu利用率马上恢复正常。而且如果我不采用懒加载的方式去加载Fragment就不会有这个问题,那首先我就怀疑是不是我这个懒加载写的有问题,debug跟了一遍,发现一切正常,并没有发现不合理的地方。
根据老司机的经验判断,既然cpu一直居高不下,那很有可能是某个view一直在测量计算。那这里嫌疑最大的就是RecyclerView,但是我这里有三个RecyclerView,只能通过排除法,一个一个注掉然后再观察cpu情况,然而意外的是即便我把他们全都注掉也没有什么用,那看来并不是View反复测量引起的问题。那这个时候矛头就直指顶部的轮播大图了,轮播图是可以自动滚动,并且无限循环的,那有可能是哪里控制的不太合理,或者timer用的有问题。似乎看见曙光了,问题应该就在这里,于是乎我把轮播图也注释掉再看。我去,仍然没什么卵用。
问题似乎陷入了僵局,按照正常的剧情发展,这个时候我应该下楼点根烟,边抽烟边和同事交流交流,然后深吸一口,吐出淡蓝色的烟雾,看着青烟徐徐上升冥思苦想,忽然大喊一声:我知道了。然后狠狠的掐灭烟头飞奔上楼,留下同事在烟雾缭绕中凌乱不堪。但实际情况是:我并不抽烟。既然这个时候不能凭主观经验迅速定位问题,那就只能采用笨办法了。依然是排除法,我把所有有嫌疑的代码都一行行注释掉,到最后我几乎把整个类都注释掉了,debug进来后已然没有什么代码需要执行了,只是加载了一个布局。但是,我已经不想再说但是了。
JAVA代码排查个遍,依然没有定位到问题,老司机已经有点不淡定了,难道是布局的问题?OverDraw?一切都是猜测,只能硬着头皮一个View一个View的去排除,然而让我万万没想到的是最后揪出来元凶,居然是上面提到的我自己写的那个BaseSwipeRefreshLayout引起的。自己挖的坑,把自己埋进去也要给填上。
- public class BaseSwipeRefreshLayout extends SwipeRefreshLayout {
- public BaseSwipeRefreshLayout(Context context) {
- super(context);
- init();
- }
- public BaseSwipeRefreshLayout(Context context, AttributeSet attrs) {
- super(context, attrs);
- init();
- }
- private void init() {
- this.setProgressViewOffset(false, DensityUtil.dip2px(getContext(), -50), DensityUtil.dip2px(getContext(), 30));
- this.setColorSchemeColors(getContext().getResources().getColor(R.color.primary_green));
- setRefreshing(true);
- }
- @Override
- public boolean onStartNestedScroll(View child, View target, int nestedScrollAxes) {
- return !isRefreshing() && super.onStartNestedScroll(child, target, nestedScrollAxes);
- }
- }
上面这个类了了数行,只是做了些统一的初始化操作,却引起了一些问题,给你三分钟你能看出问题所在吗?
问题就出在init()方法中的setRefreshing(true);这一句。写的时候是这么想的,界面一进去就要loading了,那我干脆把loading加在Base里了。这样就不用每个界面再写一遍了。但是由于我上述的场景里用了懒加载,所以问题就来了:虽然我停留在第一个页面,但是第二个页面的View已经初始化完成,那么自然SwipeRefreshLayout的那个loading的圈圈已经在不停的转动了,所以cpu就开始非常线性的居高不下了,切换到第二页,数据加载完成之后setRefreshing(false),那个loading的圈圈消失,cpu又恢复了正常。费了一番功夫,好在最后还是把坑填上了。另外没预料到的一点是,原来SwipeRefreshLayout也不怎么省油。
实际开发过程中难的不是如何解决问题,而是如何排查和定位问题。大多数情况下,我们可以凭借自己的积累和经验迅速定位问题。而这次自己挖的坑着实隐蔽,费了好大一番功夫,从JAVA代码到Xml几乎是一行一行去排查,最后还是把坑填上了。那开发中还是要考虑的全面一些,少挖坑,那如果真的发现有坑,也是有套路可寻的,仔细分析问题,从JAVA代码到xml逐步排查,总归会豁然开朗的。