代码走查总结
纠错   07 Jul 2019

Code Review 实战总结

  • 1、统一编码规范、样式检查(checkstyle)、静态代码检查（PMD或者FindBug）、单元测试
  • 2、代码提交日志必须规范。提交功能与修复BUG必须区分开来
  • 3、代码提交尽量做到原子。
  • 4、代码提交截止时间（目前没有每日构建，定时构建当日代码）
  • 5、提交Code Review请求最好两个人最好主次搭配（主深入理解本次技术），并且备注需要着重阅读代码
  • 6、Code Review 人员不固定
  • 7、Code Review 时间不能太长控制在半个小时内
  • 8、Code Review 审核人应该安排一个固定时间进行审核以免耽误自己工作，而且需要给别人反馈最后审核的时间
  • 9、每一个月对提交的审核结果进行整理，梳理出最佳实践。
  • 10、每周每个人主持一次大型code review，不提代码编写人名字。（伤面子，呵呵！）集体对代码进行点评好的地方、改进的地方
  • 11、需要理解别人代码才能找出设计漏洞

代码审查应该关注问题

  • 1、是否能够正常工作
  • 2、代码是否可读
  • 3、有没有逻辑缺陷
  • 4、是否可以进行重构，是否有更好的实现
  • 5、单元测试代码逻辑是否覆盖

Comments