SwipeRefreshLayout引发的一场血案

关于下拉刷新这件事,无论是普通用户还是开发者都再熟悉不过了,过去的某段时间无论下拉刷新的设计还是开源控件都异常火爆,火爆到惊动了党中央(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逐步排查,总归会豁然开朗的。

最后编辑于
©著作权归作者所有,转载或内容合作请联系作者
  • 序言:七十年代末,一起剥皮案震惊了整个滨河市,随后出现的几起案子,更是在滨河造成了极大的恐慌,老刑警刘岩,带你破解...
    沈念sama阅读 206,839评论 6 482
  • 序言:滨河连续发生了三起死亡事件,死亡现场离奇诡异,居然都是意外死亡,警方通过查阅死者的电脑和手机,发现死者居然都...
    沈念sama阅读 88,543评论 2 382
  • 文/潘晓璐 我一进店门,熙熙楼的掌柜王于贵愁眉苦脸地迎上来,“玉大人,你说我怎么就摊上这事。” “怎么了?”我有些...
    开封第一讲书人阅读 153,116评论 0 344
  • 文/不坏的土叔 我叫张陵,是天一观的道长。 经常有香客问我,道长,这世上最难降的妖魔是什么? 我笑而不...
    开封第一讲书人阅读 55,371评论 1 279
  • 正文 为了忘掉前任,我火速办了婚礼,结果婚礼上,老公的妹妹穿的比我还像新娘。我一直安慰自己,他们只是感情好,可当我...
    茶点故事阅读 64,384评论 5 374
  • 文/花漫 我一把揭开白布。 她就那样静静地躺着,像睡着了一般。 火红的嫁衣衬着肌肤如雪。 梳的纹丝不乱的头发上,一...
    开封第一讲书人阅读 49,111评论 1 285
  • 那天,我揣着相机与录音,去河边找鬼。 笑死,一个胖子当着我的面吹牛,可吹牛的内容都是我干的。 我是一名探鬼主播,决...
    沈念sama阅读 38,416评论 3 400
  • 文/苍兰香墨 我猛地睁开眼,长吁一口气:“原来是场噩梦啊……” “哼!你这毒妇竟也来了?” 一声冷哼从身侧响起,我...
    开封第一讲书人阅读 37,053评论 0 259
  • 序言:老挝万荣一对情侣失踪,失踪者是张志新(化名)和其女友刘颖,没想到半个月后,有当地人在树林里发现了一具尸体,经...
    沈念sama阅读 43,558评论 1 300
  • 正文 独居荒郊野岭守林人离奇死亡,尸身上长有42处带血的脓包…… 初始之章·张勋 以下内容为张勋视角 年9月15日...
    茶点故事阅读 36,007评论 2 325
  • 正文 我和宋清朗相恋三年,在试婚纱的时候发现自己被绿了。 大学时的朋友给我发了我未婚夫和他白月光在一起吃饭的照片。...
    茶点故事阅读 38,117评论 1 334
  • 序言:一个原本活蹦乱跳的男人离奇死亡,死状恐怖,灵堂内的尸体忽然破棺而出,到底是诈尸还是另有隐情,我是刑警宁泽,带...
    沈念sama阅读 33,756评论 4 324
  • 正文 年R本政府宣布,位于F岛的核电站,受9级特大地震影响,放射性物质发生泄漏。R本人自食恶果不足惜,却给世界环境...
    茶点故事阅读 39,324评论 3 307
  • 文/蒙蒙 一、第九天 我趴在偏房一处隐蔽的房顶上张望。 院中可真热闹,春花似锦、人声如沸。这庄子的主人今日做“春日...
    开封第一讲书人阅读 30,315评论 0 19
  • 文/苍兰香墨 我抬头看了看天上的太阳。三九已至,却和暖如春,着一层夹袄步出监牢的瞬间,已是汗流浃背。 一阵脚步声响...
    开封第一讲书人阅读 31,539评论 1 262
  • 我被黑心中介骗来泰国打工, 没想到刚下飞机就差点儿被人妖公主榨干…… 1. 我叫王不留,地道东北人。 一个月前我还...
    沈念sama阅读 45,578评论 2 355
  • 正文 我出身青楼,却偏偏与公主长得像,于是被迫代替她去往敌国和亲。 传闻我的和亲对象是个残疾皇子,可洞房花烛夜当晚...
    茶点故事阅读 42,877评论 2 345

推荐阅读更多精彩内容