Android 代码Review清单

自测清单

开发结束后,开发人员做好以下自测点,保证提测质量

  1. 对照需求文档,测试逻辑是否吻合完整
  2. 对可能有问题的地方进行简单的压力测试,是否会出现闪退Bug
  3. 对照Umeng或者Bugly开发阶段出现的闪退bug是否都已解决
  4. 检测UI是否有变形,比如常见的TextView,是否需要行数限制,文字太多,是否或出现页面显示异常
  5. 在不同分辨率的机型上,检查布局是否有Bug,特别是现在全面屏刘海屏,需要有全屏操作是,在有虚拟导航栏的手机上的适配问题
  6. 版本升级,从低版本升级上来,会不会有问题 比如可能会出现数据库不兼容的问题
  7. 利用手机的开发者选项中的 “调试GPU过度绘制” ,“GPU呈现模式分析” 和 “显示FPS和功耗” 功能,看自己的新功能是否会导致过度绘制、是否会掉帧
  8. app业务断网,再联网,相关业务是否正常。

Review清单

清理操作

  1. Activity退出销毁操作
     * 是否调用Handler的removeCallbacksAndMessages(null)来清空Handler里的消息;
     * 是否取消了还没完成的请求;
     * 在页面里注册的监听,是否反注册;
     * 假如自己用到观察者模式,是否反注册;
     * 假如用了RxJava的话,是否解除订阅;
  1. 数据库的游标是否已经关闭
  2. 打开过的文件流是否关闭
  3. WebView使用完是否调用了其destory()函数
  4. Bitmap是否调用recycle
    Android 3.0以下的版本,使用完的Bitmap是否调用recycle(),否则会一直占用内存 而Android 3.0及以上的版本不需要调用recycle(),因为这些版本的Bitmap全部放到虚拟机的堆内存中,让GC自动回收。

代码性能优化

  1. 检查是否使用多余的for循环,UI线程是否有耗时操作
  2. 是否可以用String代替StringBuilder,是否可以用StringBuilder代替StringBuffer
  3. 保存在内存中的图片,是否做过压缩处理再保存在内存里 否则可能由于图片质量太高,导致OOM
  4. Intent传递的数据太大,会导致页面跳转过慢。太大的数据可以通过持久化的形式传递,例如读写文件或者是否实现Parcelable
  5. 频繁地操作同一个文件或者执行同一个数据库操作,是否考虑把它用静态变量或者局部变量的形式缓存在内存里。用空间换时间
  6. Listview或者RecycleView的item中,bindData的时候,避免SharePreference的读取或者存储操作,是否可以采用全局变量来解决
  7. Sqlite数据库增删改查是否处理了多线程访问的并非操作,建议采用事务处理,高效,尽量用synchronized,效率太慢
  8. 页面布局,是否可以考虑用ViewStub来优化启动速度
  9. xml中的布局文件是否有多余的嵌套,影响性能

gradle第三方依赖包

  1. build.gradle远程依赖第三方包时,版本号建议写死,不要使用+号 避免由于新版本的第三方包引入了新的问题
  2. 调用第三方的包或者JDK的方法时,要跳进他们的源码,看要不要加 try-catch 否则可能会导致自己应用的崩溃
  3. 使用第三方包时,是否加上其混淆规则 若漏掉加上第三方包的混淆规则,会导致第三方包不该混淆的代码被混淆。在Debug版本没有发现问题,但是Release版本就会出现问题

检查成对的方法是否同时出现

  1. 系统的、自己写的,注册和反注册的方法,是否成对出现
  2. 在生命周期的回调里,创建和销毁的代码是否对应起来 比如:onCreate()里面创建了Adapter,那么对应Adapter的退出处理操作(比如清空Image缓存),一般就要写在onDestory(),而不能写在onDestoryView()。
  3. 若ListView的item复用了,对Item里View的操作是否成对出现 比如:
switch (type) {    
case ArticleListItem.TYPE_AD:        ......        
   mTitleView.setText(tencentAdBean.title);        
   mGreenLabelView.setVisibility(VISIBLE);        
   mRedLabelView.setText("");        
   mRedLabelView.setVisibility(GONE);    
   break;    
case ArticleListItem.TYPE_ARTICLE:        ......        
   mTitleView.setText(mzAdBean.adData.getTitle());        
   mGreenLabelView.setVisibility(GONE);        
   mRedLabelView.setText("ABC");        
   mRedLabelView.setVisibility(VISIBLE);        
   break;
}

内存泄漏

  1. 内部类,比如Handler、Listener、Callback是否是成static class 因为非静态内部类会持有外部类的引用。或者在页面销毁的时候,是否设置listener,CallBack等为null
  2. 要求传入Activity作为参数的函数,是否可以改用getApplicationContext()来作为参数,比如数据库,SharePreference,getSystemService等
//Kotlin代码
private val mAudioManager = context.applicationContext.getSystemService(Context.AUDIO_SERVICE) as AudioManager
private val sp=context.applicationContext.getSharedPreferences(name, Context.MODE_PRIVATE)
  1. 单例模式不要持有Activity的引用,同时要记得注册和反注册的监听,如下:
QmPayHelper.getInstance().setPayListener(object: onPayListener{
 override fun onWxPay(){
 }
})

一定要在页面销毁的时候,设置为null,不然因为onPayListener持有Activity的引用而内存泄漏

QmPayHelper.getInstance().setPayListener(null)
  1. 每次Activity关闭的时候,Handler相关的操作是否需要移除:
 mHandler.removeCallbacksAndMessages(null)
  1. 页面关闭的时候,如果有动画在执行,是否清除相关动画,比如:
@Override
 protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_leak);
        textView = (TextView)findViewById(R.id.text_view);
        ObjectAnimator objectAnimator = ObjectAnimator.ofFloat(textView,"rotation",0,360);
        objectAnimator.setRepeatCount(ValueAnimator.INFINITE);
        objectAnimator.start();
    }

@Override
 protected void onDestroy(){
        super.onDestroy();
        //取消动画
        objectAnimator.cancel();
    } 

Handler操作

  1. Handler使用的时候是否采用静态内部类以及当前类弱引用的方式出现,如下:
//Kotlin代码
inner class KpcHandler(controller: QmPlayerController) : Handler() {
        private var mWrf = WeakReference(controller)
        override fun handleMessage(msg: Message?) {
            super.handleMessage(msg)
            val controller = mWrf.get()
            controller?.hide()
        }
  }
  1. 使用View.post()是否会有问题 因为在View处于detached状态期间,post()里面的Runnable是不会被执行的。只有在此View处于attached状态时才会被执行。
    如果想改Runnable每次肯定会被执行,那么应该是用Handler.post来替代:
 Handler(Looper.myLooper()).post {  } //Kotlin代码
  1. 假如程序可能多次在同一个Handler里post同一个Runnable,每次post之前都应该先清空这个Handler中还没执行的该Runnable 如:
if (mCloudRun != null) {    
     mHandler.removeCallbacks(mCloudRun);    
     mCloudRun = null;}
mCloudRun = new Runnable() {    
     @Override   
     public void run() {        
            CloudAccelerateSwitchRequest request = new CloudAccelerateSwitchRequest();   
            request.setPriority(RequestTask.PRIORITY_LOW);     
            RequestQueue.getInstance().addRequest(request);   
     }
};
mHandler.post(mCloudRun);   

高概率闪退点以及逻辑健全Review

  1. 除数是否做了非0判断
  2. 某些情况下,某变量是否会为Null,而且在函数体内,处理参数前,必须加上判空语句
  3. 异步回调函数是否处理好,回调函数很容易出问题。比如网络请求的回调,需要判断此时的Aciivity或者Fragment等是否还存在,再进行调用。因为异步操作回来,Activity可能就消失不存在了。 而且还要对一些可能被回收的变量进行判空。
  4. 修改数据库后,是否把数据库的版本号+1,升级逻辑是否处理合理
  5. 启动第三方的Activity时,是否判断了该Intent能否被解析
Intent sendIntent = new Intent(mContext, Demo.class);
// 这种方式判断是否存在
if (sendIntent.resolveActivity(getPackageManager()) != null) {  
  startActivity(sendIntent);
}

若Activity不存在,会出现ActivityNotFoundException的异常

  1. 检查if()逻辑后是否处理了else()的相关操作,比如常见的操作,listview中的item复用的时候,如下:
if(hasTitle){
   mTitleView.setText(tencentAdBean.title);        
   mGreenLabelView.setVisibility(VISIBLE);        
}else{
  //如果不处理,则mTitleView可能会不显示
   mTitleView.setText("");        
   mGreenLabelView.setVisibility(GONE);    
}
  1. 不要在Activity的onCreate里调用PopupWindow的showAsLoaction方法,由于Activity还没被加载完,会报错
  2. 系统6.0的动态权限申请,7.0的文件访问,8.0的通知栏渠道channel等适配操作是否完善

参考:https://juejin.im/entry/5ba998f6f265da0aa74f2a76

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

推荐阅读更多精彩内容

  • Android 自定义View的各种姿势1 Activity的显示之ViewRootImpl详解 Activity...
    passiontim阅读 172,288评论 25 707
  • 用两张图告诉你,为什么你的 App 会卡顿? - Android - 掘金 Cover 有什么料? 从这篇文章中你...
    hw1212阅读 12,744评论 2 59
  • 一 朗读者第一季的时候,自小学业优秀,耶鲁大学毕业的秦玥飞没有选择做都市白领,而是投身乡村,做了湖南一个村子的村官...
    唐夕阅读 897评论 0 2
  • 已经安装 composer,写好 composer.bat,并且设置好了 path,在 cmd 下可以正常使用,但...
    guanguans阅读 2,617评论 0 2
  • sentinel的一个缺点是存储容量限于单台机器,而集群提供了一种分布式的解决方案。 redis有两种运行模式1)...
    packet阅读 412评论 0 0