Code Review Sop

1. 日志输出是否合理，关键业务要输出详细的日志。

2. DB中不能存储带有自己系统服务器中的url，因为服务器上的文件可能移动位置或者更换域名，不可靠。 微信等大厂域名可存。

3. API拆分原则：如商品详情页API，商品详情页包含商品评价信息，应拆分成多个API，不应该放在一个API中，

 a. 因为如果商品评价报错，商品应当可以正常浏览购买，仅评价不能查看。

 b. 压力大时，一个接口做的事太多，扛不住并发。

4. 所有捕获到异常的逻辑都要把异常的堆栈信息打印出来，不然无法分析异常原因。尤其是捕获的是较大的异常，如：catch(Exception e)

5. ReviewSQL语句，查看SQL索引是否被使用到了？（同时完善丰富SQL索引优化方法库）

6. 所有发送MQ，消费MQ，不消费MQ都必须打日志，不然线上分析时无法跟踪Message的生命周期

7. 方法尽可能的让作用域小，能写private，protected的就不要写public，如果写大了作用域，后续不确定是否有人调用无法修改。

 

 

 

Review代码太难懂时把写代码的大佬叫过来让他给你逐行解释一下。