代码评审都评审些什么呢？

可以分为以下几点进行描述

主讲人需要涉及到的点：

         1、大概的业务讲解
         2、主讲重点实现
                2.1、可能影响到其他人的代码，确认代码的修改范围
                2.2、新的技术使用，自己是要讲解一下这个技术用来干什么的，是在这个业务的初衷是怎样，是否符合
                2.3、风险问题也是要讲解一下，看看新增的代码是否有风险
                2.4、自我讲解代码的亮点

评审人需要涉及到的点：

        1、 方法参数过多（最好是3个，一般是3-5个吧）
        2、方法太长,未进行拆分
        3、各种长事务
        4、sql复杂
        5、方法命名及其不合理
        6、 一个方法调用大量查询
        7、 日志瞎比打
        8、 没有事务注解