Code Review都需要关注什么?

Code Review

Code Review 主要从三个方面去 Review:

  1. 系统风险识别(死循环,死锁,异步任务,线程池等,资金安全,数据泄露安全,数据一致性,有没有补偿方案等)
  2. 代码质量(代码可读性,方法和类的签名,硬编码,超长方法和超大类)
  3. 代码的可扩展性(代码结构是否合理,是否考虑了扩展性)

Code Review检查清单

  1. 死循环,死锁,异步任务,线程池等
  2. 接口性能,安全性,数据一致性,并发控制
  3. 代码可读性
  4. 重复编码
  5. 硬编码
  6. 超长方法(100行/超过一屏高度)
  7. 超长的参数列表(最多4个)
  8. 超大的类
  9. Switch 分支语句
  10. 通过参数控制逻辑
  11. 流程控制逻辑嵌套超过两层
  12. sonar扫描major级别警告