结算Code review

Code review 的好处

好处自然不必多说，主要是：

1. 可以在测试前期发现代码中的逻辑问题，主要是针对核心逻辑。
2. 检查编码风格和代码结构设计问题
3. 边缘性功能检查，如日志打印，埋点功能等是否完善。

---

结算Code review前需要思考以下几个问题：

• 代码的核心逻辑在哪？作为开发者是否能流畅地给别人讲出来。需要自己把其中的核心逻辑逐条罗列出来，针对自己的代码可以有针对性的梳理。
• 代码的整体结构是否符合《代码整洁之道》中提到的要求？每个方法的职责是否单一且代码划分的层次足够清晰？
• 能否可以通过配置切换路由，做到自动路由，单机房路由。
• 降级逻辑是否完善，能否做到不影响以后新增的接口的逻辑。底层共用的降级是否会影响到当前的业务降级？如果会，该如何处理。
• 异常的捕获和处理，有没有出现异常的过度捕获，如一层层的catch，一层层的上抛。 有没有出现可能漏掉异常的可能性。
• 日志打印需要自己手动在程序中写死？还是使用了aop切面。埋点也是同样的道理。埋点是埋了正常的逻辑调用，还是埋了异常逻辑的调用。还是两者都有，所埋下的点的 tp性能和可用率的变化是否可以推测出程序逻辑上出现的问题。
• 新加方法的是否支持单机压测，如果能需要知道压测性能是多高？如果不能，需要排查不能压测的原因。免得备战的时候，不能够压测，导致没有合理的数据进行参考。忌病急乱投医

---

注意：
针对上面每一步代码review的时候，都需要问自己，为什么这样做，这样做的好处是什么？如何还有其他的做法，应该如何取舍？

如果以上的步骤都完成了，那么可以说，代码review的时候，可以查到的问题，也应该不会太多。