代码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.代码是否过度设计过度实现

  • 0
    点赞
  • 0
    收藏
    觉得还不错? 一键收藏
  • 0
    评论

“相关推荐”对你有帮助么?

  • 非常没帮助
  • 没帮助
  • 一般
  • 有帮助
  • 非常有帮助
提交
评论
添加红包

请填写红包祝福语或标题

红包个数最小为10个

红包金额最低5元

当前余额3.43前往充值 >
需支付:10.00
成就一亿技术人!
领取后你会自动成为博主和红包主的粉丝 规则
hope_wisdom
发出的红包
实付
使用余额支付
点击重新获取
扫码支付
钱包余额 0

抵扣说明:

1.余额是钱包充值的虚拟货币,按照1:1的比例进行支付金额的抵扣。
2.余额无法直接购买下载,可以购买VIP、付费专栏及课程。

余额充值