第二次代码评审

Posted THISISPAN

tags:

篇首语:本文由小常识网(cha138.com)小编为大家整理,主要介绍了第二次代码评审相关的知识,希望对你有一定的参考价值。

1.判断list是否为空,
最好用list.isempty(),不用list.size()
2.枚举是单例的,单例可以直接用==判断相等
3.for循环里面的代码,可以封装成方法
4.update table(a = a+1),不要在代码里相加。对于Oracle
5.for update
6.圈复杂度不能大于15
7.for(i;i<条件;i++),条件不能是表达式,否则会每次循环都计算一次
8.应用日期和数据库日期,选用数据库日期
9.空行,一个空行就可以,没必要两个


接口规范会议:
1.上下文,header头里放入版本号
2.复杂对象返回类型,耦合依赖。
3.面向对象与面向过程的互相转换
4.调用方式,
httpclient,spring rest template,rpc
5.RPC客户端负责抛出异常
6.throwable
7.DTO成对儿出现

以上是关于第二次代码评审的主要内容,如果未能解决你的问题,请参考以下文章

1702-03组-小组第二次会议

gitlab如何实现代码评审机制(通过角色管理控制)

软件项目简明代码评审流程

团队管理中的代码评审

6.我们真的做了代码评审

实验四 代码评审