代码评审都评审些什么呢？
可以分为以下几点进行描述
主讲人需要涉及到的点：
1、大概的业务讲解
2、主讲重点实现
2.1、可能影响到其他人的代码，确认代码的修改范围
2.2、新的技术使用，自己是要讲解一下这个技术用来干什么的，是在这个业务的初衷是怎样，是否符合
2.3、风险问题也是要讲解一下，看看新增的代码是否有风险
2.4、自我讲解代码的亮点
评审人需要涉及到的点：
1、 方法参数过多（最好是3个，一般是3-5个吧）
2、方法太长,未进行拆分
3、各种长事务
4、sql复杂
5、方法命名及其不合理
6、 一个方法调用大量查询
7、 日志瞎比打
8、 没有事务注解