Code Review都需要关注什么？

Code Review

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

1. 代码整体结构设计是否合理，比如设计模式的使用，线程池的使用，异常使用等。
2. 代码质量（代码可读性；代码规范，如：方法和类的签名，硬编码，超长方法和超大类；单元测试）
3. 系统风险识别（死循环，死锁，异步任务，线程池等，资金安全，数据泄露安全，数据一致性，有没有补偿方案等）
4. 代码的可扩展性（代码结构是否合理，是否考虑了扩展性）
5. 可运维性（日志，异常，监控）

Code Review检查清单

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