代码审查那些事儿

代码审查那些事儿

定义

代码审查（英语：Code review）是指对计算机源代码系统化地审查，常用软件同行评审的方式进行，其目的是在找出及修正在软件开发初期未发现的错误，提升软件质量及开发者的技术。代码审查常以不同的形式进行，例如结对编程、非正式的看过整个代码，或是正式的软件检查 from wiki

大概流程

• 时间: 最好每天或每周3 下班前1个小时
• 主持: 会议室,由leader主持
• 内容:
  • 组内每人调400行代码的主逻辑出来查看
  • 每人简述完毕后,QA环节
  • 挑取有效的建议,标注到upsource的代码审查系统

清单

• 审查内容
  • 代码安全性
  • 代码性能
  • 代码注释
  • 单元测试
  • 代码优化
    • 格式化
    • 分类
    • 封装方法

注意

1. 代码审查应该是日常工作
2. 审查内容不要过多,关注主逻辑,行数不要超过400行
3. 代码审查过程要保持心态平和,每个人都是值得尊重的

补充

upsource是jetbrain公司出的一个团队协作的代码审查系统,可以配合idea实时显示评论.安装时免费的,但是团队的成员账号不能超过5个.如果超过则需要给钱买授权;免费的适合小组内的代码审查.目前在用,感觉还行.

联系我: suveng@163.com qq: 1344115844