说到code review，每个程序员都必须面临，这一块到底有无什么技巧经验呢？
简单聊下自己在代码review的一些看法:
稳定性
- 稳定性检查: 是否会空指针,添加判空
- 格式转换异常: 针对数字转换类型,比如str-> int等,尤其要关注,就算认为不可能,也要添加catch, 主要是很多数据是服务器来配置下发,客户端必须要考虑容错方案
- 强转类型判断, 对应部分强转类型要用instant of
- 弱引用的空指针隐患
- 线程安全类型检查
- 系统api的坑
业务逻辑
- 针对业务使用是否妥当
- 异常场景业务是否正常
性能
- 使用部分api是否有性能问题
- 第三方介入的api检测等
- 低端机下的考虑
- 不同android rom下是否有兼容性问题
- 不同机器分辨率布局是否存在问题,考虑不周等
- 包size是否有优化空间.图片是否经过预压缩
- 内存泄漏风险
格式
- 格式换行等是否符合规范
- 静态变量\全局变量命名是否规范
- 业务命名变量是否清晰
- 方法命名是否足够清晰
- 部分特殊的场景或者坑是否没有添加注释
架构设计
- 有无重复造轮子,部分模块可直接使用,而不是重新提交一份
- 模块设计是否清晰合理
- 模块是否解耦\独立\具备组件化
- 是否影响其他模块
兼容性
- 覆盖安装场景
- 全新安装场景
- …
综上,会发现,一个review将会涉及到方方面面,而每一方面在不同人review都会有不同的侧重点,有些偏重了解你的业务,有些则会注重稳定性或者格式等,感觉每一次reivew,只要有人抽时间,都是对你的一次提高和大家的碰撞.
转载声明: 本文转载前需与作者联系并标明文章出处