专栏首页非著名程序员良心推荐:总结 Android 开发中必备的代码 Review 清单

良心推荐:总结 Android 开发中必备的代码 Review 清单

本文收集了我自己工作以来提交代码前的所有检查点。事实证明,这样能有效提高自己的代码质量和功能的稳定性。所以推荐大家以后每次提交代码前,都可以看下这份 Review 清单哈。

此外,可能还有些检查点我并没有发现,欢迎大家踊跃在评论区补充哈~

1

清理操作

1.页面退出时,是否完成必要的清理操作

1) 是否调用Handler的removeCallbacksAndMessages(null)来清空Handler里的消息; 2) 是否取消了还没完成的请求; 3) 在页面里注册的监听,是否反注册; 4) 假如自己用到观察者模式,是否反注册; 5) 假如用了RxJava的话,是否解除订阅;

这五个维度相互之间有着或强或弱的关联,任意两份代码之间可以参考这个体系进行大概的比较,但没有绝对的高下之分。

2.页面退出时,是否完成必要的清理操作

这个点一般人都知道,出问题一般在于,没有考虑到多线程并发时的情况下,Cursor没有被释放。

所以数据库的操作需要加上同步代码块。详细可参考:http://www.2cto.com/kf/201408/329574.html

3.打开过的文件流是否关闭

4.Android 3.0以下的版本,使用完的Bitmap是否调用recycle(),否则会一直占用内存

而Android 3.0及以上的版本不需要调用recycle(),因为这些版本的Bitmap全部放到虚拟机的堆内存中,让GC自动回收。

5.WebView使用完是否调用了其destory()函数

2

是否能进一步优化自己的代码

1.保存在内存中的图片,是否做过压缩处理再保存在内存里,否则可能由于图片质量太高,导致OOM。

2.Intent传递的数据太大,会导致页面跳转过慢。太大的数据可以通过持久化的形式传递,例如读写文件

3.频繁地操作同一个文件或者执行同一个数据库操作,是否考虑把它用静态变量或者局部变量的形式缓存在内存里。用空间换时间

4.放在主页面的控件,是否可以考虑用ViewStub来优化启动速度

3

要小心第三方包

1.build.gradle远程依赖第三方包时,版本号建议写死,不要使用+号

避免由于新版本的第三方包引入了新的问题

2.导入第三方工程时,记得把编码转换成自己工程当前是用的编码

3.调用第三方的包或者JDK的方法时,要跳进他们的源码,看要不要加 try-catch

否则可能会导致自己应用的崩溃

4.使用第三方包时,是否加上其混淆规则

若漏掉加上第三方包的混淆规则,会导致第三方包不该混淆的代码被混淆。在Debug版本没有发现问题,但是Release版本就会出现问题

5.系统应用添加so时,是否在固件对应的Android.mk文件上加入新增的so,否则系统可能编译不过

@lib/armeabi/libcommon.so \
@lib/armeabi/libabcdefg.so \

4

注意要成对出现的地方

1.系统的、自己写的,注册和反注册的方法,是否成对出现

2.在生命周期的回调里,创建和销毁的代码是否对应起来

比如:onCreate()里面创建了Adapter,那么对应Adapter的退出处理操作(比如清空Image缓存),一般就要写在onDestory(),而不能写在onDestoryView()。

类似的生命周期对应的代码有:

onStart()、onStop();onCreate()、
onDestory();onResume()、onPause();
onCreateView()、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;
}

比如以上对mTitleView、mGreenLabelView和mRedLabelView的操作,都是成对出现。否则ListView可能会由于Item复用,导致Item显示错乱问题

5

防内存泄漏

1.内部类,比如Handler、Listener、Callback是否是成static class

因为非静态内部类会持有外部类的引用。

2.假如子线程持有了Activity,要用弱引用来持有

比如Request的Activity就应该用弱引用的形式,防止内存泄漏。

3.要求传入Activity作为参数的函数,是否可以改用getApplicationContext()来作为参数

6

Handler相关

1.使用View.post()是否会有问题

因为在View处于detached状态期间,post()里面的Runnable是不会被执行的。只有在此View处于attached状态时才会被执行。

如果想改Runnable每次肯定会被执行,那么应该是用Handler.post来替代

2.假如程序可能多次在同一个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);

7

其他

1.多思考某些情况下,某变量是否会为空

而且在函数体内,处理参数前,必须加上判空语句

2.回调函数是否处理好

回调函数很容易出问题。比如网络请求的回调,需要判断此时的Aciivity等是否还存在,再进行调用。因为异步操作回来,Activity可能就消失不存在了。

而且还要对一些可能被回收的变量进行判空。

3.修改数据库后,是否把数据库的版本号+1

4.启动第三方的Activity时,是否判断了该Intent能否被解析

Intent sendIntent = new Intent(mContext, Demo.class);
// 这种方式判断是否存在
if (sendIntent.resolveActivity(getPackageManager()) != null) {
    startActivity(sendIntent);
}

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

5.新注册的Activity、Service或Provider,若AndroidManifest.xml中exported属性为true,要考虑是否会引发安全性问题

<activity android:name="com.inkenka.DemoActivity"
            android:exported="true"/>

因为exported属性为true时,外部应用就可以直接调用起该Activity。

可能导致的问题:

1) 若外部应用直接启动详情页,从而让某些验证页面直接被绕过 2) 若外部应用给该Activity传递乱七八糟的Intent,可能让该应用崩溃。也就是Android中的拒绝服务漏洞

6.除数是否做了非0判断

7.不要在Activity的onCreate里调用PopupWindow的showAsLoaction方法,由于Activity还没被加载完,会报错

8

功能完成后,自测时的检查点

1.思考某些情况下,某个变量是否会造成空指针问题

2.把手机横屏,检查布局是否有Bug

3.在不同分辨率的机型上,检查布局是否有Bug

4.切换到英文等外文字体下,检查外文是否能完整显示

5.从低版本升级上来,会不会有问题,比如可能会出现数据库不兼容的问题

6.按下Home再返回是否正常

7.熄灭屏幕再打开是否正常

8.切换成其它应用再切换回来会怎样

9.利用手机的开发者选项中的 “调试GPU过度绘制” ,“GPU呈现模式分析” 和 “显示FPS和功耗” 功能,看自己的新功能是否会导致过度绘制、是否会掉帧

10.测试看是否影响启动速度:adb shell am start -W 包名/Activity

11.对比看APK大小是否有增大

12.跑1小时Monkey,测试其稳定性

本文分享自微信公众号 - 非著名程序员(non-famous-coder),作者:InKenKa

原文出处及转载信息见文内详细说明,如有侵权,请联系 yunjia_community@tencent.com 删除。

原始发表时间:2017-06-01

本文参与腾讯云自媒体分享计划,欢迎正在阅读的你也加入,一起分享。

我来说两句

0 条评论
登录 后参与评论

相关文章

  • 常见面试第三题之Activity的几种启动模式介绍

    Activity的几种启动模式介绍 ? 今天我们来讲的是Activity有几种启动模式,其实以前我没太在意这方面的知识,但是我在开发过程中发现,其实了解和...

    非著名程序员
  • Activity 的4种启动模式,分析的简单全面

    今天想来说说Android的启动模式,一来自己做一个总结,二来如果能帮助到别人就更好了~ 首先来看一个实际的业务场景。我之前在公司实习的时候,我所在的部门只负责...

    非著名程序员
  • Java程序员月薪达到三万,需要技术水平达到什么程度?

    非著名程序员
  • Golang 入门系列(三)Go语言基础知识汇总

    前面已经了 Go 环境的配置和初学Go时,容易遇到的坑,大家可以请查看前面的文章 https://www.cnblogs.com/zhangweizhong/c...

    架构师精进
  • 《精进》6 努力,是一种最需要学习的才能 不断优化你的“努力”方式

    yeedomliu
  • 一张图读懂结对编程与单人开发的差异

    一张图读懂结对编程与单人开发的差异。结对编程的好处(尤其适合软件模块之间耦合很强的场景):

    吾真本
  • 辞职理由千奇百怪,腾讯的也许最...

    最近本乐看到我们HR的小姐姐都是一脸苦逼,为啥?因为每逢“金三银四”就会有各种层出不穷的奇葩离职信~今儿乐乐就带你们细数一下这些年我见过的奇葩辞职理由。 吃...

    腾讯乐享
  • leetcode399. Evaluate Division

    已知一些字母之间的关系式,问是否能够计算出其它字母之间的倍数关系? 如已知a/b=2.0 b/c=3.0问是否能够计算出a/c, b/a, a/e, a/a, ...

    眯眯眼的猫头鹰
  • 常见面试第三题之Activity的几种启动模式介绍

    Activity的几种启动模式介绍 ? 今天我们来讲的是Activity有几种启动模式,其实以前我没太在意这方面的知识,但是我在开发过程中发现,其实了解和...

    非著名程序员
  • Spark性能优化之道——解决Spark数据倾斜(Data Skew)的N种姿势

    Jason Guo

扫码关注云+社区

领取腾讯云代金券