怎么进行CodeReview

怎么进行Code review?

需求

• 功能是什么？
• 功能之外的需求
  • 可用性/易用性、
  • 可测试、
  • 目标性能、
  • 安全性、
  • 扩展性、
  • 可用性等
  • 监控、打点

设计

1. 方案设计是否合理

1. 表定义：是否需要分库/分表？根据业务预期增长情况考虑、2～3年
2. 缓存：Redis、本地缓存、
3. Redis数据结构设计是否合理：zset、hash、string、set等

- 大key问题？
- 热key问题？

3. 数据结构/模型定义
4. 相关分层结构？

- 同样的代码逻辑是否集中在同一个服务/模块中，而不是零散在多个微服务、定时任务等多个地方？
- 读写DB、修改缓存是否集中的同一个服务和模块中

5. 是否需要消息队列/异步等？

2. idl设计

1. 接口设计是否合理，接口参数是否合理，
   • 1. 命名规范容易理解、
   • 2. 必要的注释、
   • 3. 是否容易使用，不容易误用、
   • 4. 是否方便测