代码审查那些事儿
定义
代码审查(英语:Code review)是指对计算机源代码系统化地审查,常用软件同行评审的方式进行,其目的是在找出及修正在软件开发初期未发现的错误,提升软件质量及开发者的技术。代码审查常以不同的形式进行,例如结对编程、非正式的看过整个代码,或是正式的软件检查 from wiki
大概流程
- 时间: 最好每天或每周3 下班前1个小时
- 主持: 会议室,由leader主持
- 内容:
- 组内每人调400行代码的主逻辑出来查看
- 每人简述完毕后,QA环节
- 挑取有效的建议,标注到upsource的代码审查系统
清单
- 审查内容
- 代码安全性
- 代码性能
- 代码注释
- 单元测试
- 代码优化
- 格式化
- 分类
- 封装方法
注意
- 代码审查应该是日常工作
- 审查内容不要过多,关注主逻辑,行数不要超过400行
- 代码审查过程要保持心态平和,每个人都是值得尊重的
补充
upsource是jetbrain公司出的一个团队协作的代码审查系统,可以配合idea实时显示评论.安装时免费的,但是团队的成员账号不能超过5个.如果超过则需要给钱买授权;免费的适合小组内的代码审查.目前在用,感觉还行.
联系我: suveng@163.com qq: 1344115844