为什么需要codeReview ？
- 提测前发现bug
- 分享代码和知识交流
- 形成互补
- 没有人是团队内唯一的专家
- 我们是一个团队，不是为自己写代码
发起CR
- 准备
自己维护的checklist 自己当作reviewer来对自己代码进行cr 预估代码可能出问题的地方 进行充分的自测 不要指望别人帮你找问题
前置检查
- 单元测试
- 新增单元测试
- 方法行数过多
- 圈复杂度过高
- 代码规范检查
合理的规模
- 完整的单一功能变更
- feature拆分
- 容易理解
- 行数少于200行
- 30分钟内完成审查
基本审查
- 编码规范
- 潜在的bug
- 文档和注释
- 重复代码
- 复杂度
- 测试覆盖率
- 设计与架构
尽量疑问，不要太肯定
这个地方是否？ 对事不对人
要多称赞
没必要力求完美
nit-pick
