记一次bug修复 -运用Android的窗口管理机制

前几天,在编写公司普通业务代码的过程中,遇到个bug,修复之后觉得还是挺有意思的,所以跟大家分享下哈~

为什么觉得这个bug比较有意思呢?因为这个bug不是本人最常见的老脸孔nullPointer、outOfIndexBounds了,解决这个bug要了解Android的窗口管理机制,要是我之前没系统地学习过,估计我肯定百思不得其解。。。不过掌握了相关的知识后,解决这个bug那是小菜一碟。所以内心不禁沾沾自喜,毕竟我努力学进阶知识很大的一个动机就是为了从容面对bug啊!!对一个bug毫无头绪甚至连errorMessage都读不懂的羞愧感我真的怕了。。。

具体的Android的窗口管理机制就不在此讲了。如果你了解主线程的消息循坏机制,那会对本文理解得更加清晰哈~

背景是这样的:
公司的APP的业务大多是很常见的信息展示、响应用户操作事件。对于每一个网络请求,都会在等待后台返回时显示个Loading动画,表示正在处理中(很多APP也是这样做的啦)。逻辑大概是这样

showLoading();
//这里进行网络请求
dismissLoading();  //接收到响应时隐藏掉

这样做一直以来都很和谐。直到前几天在写一个业务,是让用户可以上传图片的。写过上传图片业务的朋友都应该比较熟悉,一般的公司处理上传图片都要经过两步:第一步获取token,第二步才上传到七牛。获取token对用户来说是感知不到的,在他们眼中上传图片就是一步到位,所以我们这两个网络请求是第一步紧接着第二步的。当我想都没想就写下以下逻辑代码并测试时,APP闪退了:

showLoading();
//这里获取token
dismissLoading(); 
showLoading();
//这里上传图片到七牛
dismissLoading(); 

Logcat信息如下:


0C291BBF-C833-45F4-A8FE-4F15915AE0D9.png

(好吧还是老脸孔NullPointerException😓)
日志显示loading这个变量为空

因为APP里显示的是统一的Loadind动画,而这个Loading动画是同事写的,我并不熟悉。于是点进去看看,首先是showLoading()和dismissLoading()两个函数

protected void showLoading(){
    loading = Loading.newInstance();
    loading.show(getFragmentManager(),"loading");
}

protected void dismissLoading(){
    if (loading != null) {
        loading.dismiss();
    }
}

这两个函数就是通过loading这个变量来控制Loading动画的了,再点进去这个loading,看看是何方神圣

public class Loading extends BaseDialogFragment{
    private static Loading loading;

    public static Loading newInstance() {
        loading = new Loading();
        Bundle bundle = new Bundle();
        loading.setArguments(bundle);
        return loading;
    }

    @Override
    public void onCreate(@Nullable Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        Bundle bundle = loading.getArguments();
    }

    @Override
    public void onDestroyView() {
        super.onDestroyView();
        loading = null;
    }
}

可以看到,loading是个static变量,而且在每次showLoading()时,loading会被赋予新变量 loading = new Loading(); 每次dismissLoading()时,loading变量会被设为空变量 loading = null;

然后debug一下,发觉奔溃是发生在第二次showLoading()的时候,也就是第一步获取token成功后,第二步上传到七牛前的那个showLoading()。
我觉得很奇怪,showLoading()方法中已经确保了loading变量不为空 loading = Loading.newInstance(); 紧接着下一行代码就是 loading.show(getFragmentManager(),"loading"); 了啊,为什么loading变量突然就变空了???

927FD7C3-40D3-4450-81FC-822FA401CBD4.jpg

冷静下来再看看代码,看到Loading是继承BaseDialogFragment的。条件反射般联想到,Dialog是一种窗口!!对于窗口的管理应该注意!!对于我们APP的主线程来说,这里的创建/销毁一个Dialog其实是一个异步非阻塞任务!!

我们来分析一下:第一次dismissLoading()时其实只是通过binder告诉WMS,我要销毁这个Dialog,征求WMS的允许和调度;然后我们的APP还没来得及接收到WMS的反馈,紧接着就继续执行第二步的showLoading()了,即告诉WMS,我要显示这个Dialog。但注意此时Dialog还没有真的被销毁再显示,我们的APP只是向WMS发出了请求,因为这两个过程都需要WMS的允许和配合。
当我们的APP的主线程收到WMS的返回的msg时,消息队列里必定是销毁Dialog的msg在先,显示Dialog的msg在后的。处理销毁Dialog的msg时,我们会执行onDestroyView()里的 loading = null ,loading变量赋值为null;紧接着再处理显示Dialog的msg,就会执行onCreate()里的loading.getArguments();这时候当然报NullPointerException啦~~

所以,这个bug时由于我没重视到窗口的创建/销毁是个异步任务所造成的~要是不了解这个机制,估计我根本不会意识到这个问题,而会觉得创建/销毁Dialog跟View.setVisiable(boolean)差不多呢。。。这种情况,估计加多少班都修复不了这个bug。。。

既然弄清了问题出在哪,那解决方法很简单,第一步和第二步之间不要操作Loadind动画就好了~

showLoading();
//这里获取token
//dismissLoading(); 
//showLoading();
//这里上传图片到七牛
dismissLoading(); 

感悟:Android的窗口管理机制涉及的东西比较广,他们之间的调用链,类和方法的具体名字,我看了又忘😓。不过整个机制我是理解并且大体掌握的,这些知识不进是进阶必须,而且还是修复某些bug所必须掌握的利器😊解决这个bug比平时解决的那些OutOfIndexException要爽多了😄


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

推荐阅读更多精彩内容