如何进行代码审查？

如何review开发人员的代码

前置的一些概念

review级别：（参与人身份和方式不同划分）
● 相关开发自己看代码（非正式会议）
● 开发人员组内
● 相关开发+直接上级
● 相关开发+直接上级+总监

1.团队review制度

团队内根据实际情况规定流程
在我们团队内要求：
正式需求上线前要经过：组内，直接上级参加的，技术总监参加的review
小需求紧急问题视情况简化

2.review会议组织

相关开发人员自己review
找个时间找个会议室就可以了
有上级或非相关开发参加的
确定会议主持人（一般是被review的人）
主持人要做以下事情：
确定谁来参与，约会议日程
提前把要看的代码，相关需求发出去给参加的人

3如何开会议？

时间控制在一个小时内（太长了效果就差了）
做好会议记录
● 基本信息：时间、地点、参与人
● 记录问题：本次review发现了哪些问题，谁的问题，推荐怎么改【后续要跟进】
● 记录共性问题或好的写法，定期在周会上同步整个团队内，制定相应的规范
按照功能点看相关的代码，在看代码前先简单介绍一下代码实现了一个什么需求做了什么事情 让参与的人有认知

4. Review的时候怎么看别人的代码？

● 是否符合团队内规范
● 凭经验看哪里写的有坑
dto类不能有默认值，属性必须是包装类型
数据一致性考虑，执行失败是否有异常处理（日志、补偿）等等…
● 考虑这段代码自己写是否有更好的写法（优雅可读性好）

如果这个项目用了领域驱动设计，还要看
● 代码和设计是否一致
● 聚合、实体、值对象设计是否合理
● 代码的层级调用关系是否合理
● 各个层是否只承担了自己的职责（应用层编排领域服务 ，业务逻辑都在领域层，基础设施层只干技术的事情，只持久化实体 不包含业务逻辑）

不一定要代码彻底写完才进行review，先期可以把主体结构写好，细节写好 todo 如果review时有大的结构变动会很方便调整【要知道写完代码，都测试的没问题，还要大改，重新测试是很痛苦的】