代码review标准

1.代码是否实现了设计文档提出的功能需求(业务需求)

2.代码里面是否做了正确的初始化(变量初始化,数据库,redis,rocketmq等的初始化和配置)

3.代码设计规范(驼峰命名,代码缩进,类首字母大写,方法首字母小写,静态变量全大写,方法参数超过4个抽象成类等)

4.代码的api是否合理,让使用人是否容易理解,并在代码级别做到限制误用

5.代码异常处理原则(底层异常转换为业务异常,采用TinyException->RuntimeException,防止污染api)

6.是否存在大量重复代码(多处调用相同的代码)

7.代码中是否出现潜在性能问题(数据库未加索引,System
.out,printStackTrace,粗暴的锁,重复递归,过滤器频繁操作)

8.代码中是否出现安全隐患(每次请求传输userId,特殊条件下触发死循环)
while (true) {
        if (threads.size() <= counts || flag) {
                break;
        }
}

9.代码是否线程安全(flow,spring管理的bean都要做到线程安全)
@Component("test")
public class Test{
        private int COUNT=0;
        public void action(...){
                //操作COUNT;
        }
}

10.代码健壮性检查(为空判断,数组边界溢出,堆栈溢出)

11.代码bug的检查(流未关闭,未初始化的变量直接使用)

12.代码是否过度设计过度实现

相关推荐
©️2020 CSDN 皮肤主题: 大白 设计师:CSDN官方博客 返回首页