前言
本文收集了我自己工作以来提交代码前的所有检查点。事实证明，这样能有效提高自己的代码质量和功能的稳定性。所以推荐大家以后每次提交代码前，都可以看下这份 Review 清单哈。
清理操作
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() 函数
是否能进一步优化自己的代码
1.保存在内存中的图片，是否做过压缩处理再保存在内存里
否则可能由于图片质量太高，导致 OOM
2.Intent 传递的数据太大，会导致页面跳转过慢。太大的数据可以通过持久化的形式传递，例如读写文件
3.频繁地操作同一个文件或者执行同一个数据库操作，是否考虑把它用静态变量或者局部变量的形式缓存在内存里。用空间换时间
4.放在主页面的控件，是否可以考虑用 ViewStub 来优化启动速度
要小心第三方包
1.build.gradle 远程依赖第三方包时，版本号建议写死，不要使用+号
避免由于新版本的第三方包引入了新的问题
2.导入第三方工程时，记得把编码转换成自己工程当前是用的编码
3.调用第三方的包或者 JDK 的方法时，要跳进他们的源码，看要不要加 try-catch
否则可能会导致自己应用的崩溃
4.使用第三方包时，是否加上其混淆规则
若漏掉加上第三方包的混淆规则，会导致第三方包不该混淆的代码被混淆。在 Debug 版本没有发现问题，但是 Release 版本就会出现问题
5.系统应用添加 so 时，是否在固件对应的 Android.mk 文件上加入新增的 so，否则系统可能编译不过
注意要成对出现的地方
1.系统的、自己写的，注册和反注册的方法，是否成对出现
2.在生命周期的回调里，创建和销毁的代码是否对应起来
比如：onCreate() 里面创建了 Adapter，那么对应 Adapter 的退出处理操作(比如清空Image 缓存)，一般就要写在 onDestory()，而不能写在 onDestoryView()。
类似的生命周期对应的代码有：
onStart()、onStop();
onCreate()、onDestory();
onResume()、onPause();
onCreateView()、onDestoryView()
3.若 ListView 的 item 复用了，对 Item 里 View 的操作是否成对出现
比如：
比如以上对 mTitleView、mGreenLabelView 和 mRedLabelView 的操作，都是成对出现。否则 ListView 可能会由于 Item 复用，导致 Item 显示错乱问题
防内存泄漏
1.内部类，比如 Handler、Listener、Callback 是否是成 static class
因为非静态内部类会持有外部类的引用。
2.假如子线程持有了 Activity，要用弱引用来持有
比如 Request 的 Activity 就应该用弱引用的形式，防止内存泄漏。
3.要求传入 Activity 作为参数的函数，是否可以改用 getApplicationContext() 来作为参数
Handler相关
1.使用 View.post() 是否会有问题
因为在 View 处于 detached 状态期间，post() 里面的 Runnable 是不会被执行的。只有在此 View 处于 attached 状态时才会被执行。
如果想改 Runnable 每次肯定会被执行，那么应该是用 Handler.post 来替代
2.假如程序可能多次在同一个 Handler 里 post 同一个 Runnable，每次 post 之前都应该先清空这个 Handler 中还没执行的该 Runnable
如：
其他
1.多思考某些情况下，某变量是否会为空
而且在函数体内，处理参数前，必须加上判空语句
2.回调函数是否处理好
回调函数很容易出问题。比如网络请求的回调，需要判断此时的 Aciivity 等是否还存在，再进行调用。因为异步操作回来，Activity 可能就消失不存在了。
而且还要对一些可能被回收的变量进行判空。
3.修改数据库后，是否把数据库的版本号+1
4.启动第三方的 Activity 时，是否判断了该 Intent 能否被解析
若 Activity 不存在，会出现 ActivityNotFoundException 的异常
5.新注册的 Activity、Service 或 Provider，若 AndroidManifest.xml 中 exported 属性为 true，要考虑是否会引发安全性问题
因为 exported 属性为 true 时，外部应用就可以直接调用起该 Activity。
可能导致的问题：
1）若外部应用直接启动详情页，从而让某些验证页面直接被绕过
2）若外部应用给该 Activity 传递乱七八糟的 Intent，可能让该应用崩溃。也就是 Android中的拒绝服务漏洞
5.除数是否做了非 0 判断
6.不要在 Activity 的 onCreate 里调用 PopupWindow 的 showAsLoaction 方法，由于Activity 还没被加载完，会报错
功能完成后，自测时的检查点
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，测试其稳定性
注：非原创。来源自掘金code小生
网友评论