代码评审常见问题总结【持续更新】

为什么80%的码农都做不了架构师？>>>

1：如果调用的方法返回值是基本数据类型，接收返回值的变量如果不是必须用包装类，请不要使用包装类进行接收，同理：在一个方法内return基本数据类型，方法返回值不要写包装类。

2：方法的参数列表中，如果包含可以为空的入参，请将该参数往后放，把主要的参数放在前面。

3：方法入参进行不要用id、code、name等过于简单的描述，应该用patientId，centerUserId，patientCode等，如果用前者必须写注释

4：一个方法内get()获取一个属性时，重复2次以上一律提成一个变量，不要在多处重复调用get方法

5：DTO或者VO只用于数据传输，不要作为入参进入方法内，以这个方法为例，方法内使用了仅recordDTO的recordId和recordParamList，该方法的参数列表就应该只有这（recordId，recordParamList），这里的思想是：入参是函数体的目录，需要让人一眼就能看到包含什么，除非必要情况，尽量不要传对象，特别的大对象

6：进行修改操作的接口一定要先校验合法性，看该记录可不可以被修改，甚至修改人有没有修改记录的资格（token只能保证用户本身是合法的，不能保证他的行为）

7：引入第三方类库时不要引入不用版本的相同内容，要在import中修改统一的版本，尽量不要在代码段落中书写完整的路径

8：循环超过2层时考虑重构、分支判断层数过多时请考虑使用卫语句或其他方法重构。

9：没有必要的时候，尽量不要写大功能的公用方法。

转载于:https://my.oschina.net/dlam/blog/3002864

代码评审常见问题总结【持续更新】相关推荐

【帆软报表】使用技巧及常见问题汇总-持续更新
[帆软报表]使用技巧及常见问题汇总-持续更新 1.重复与冻结设置,做用:冻结区域模板-重复与冻结设置 2.单元格有效小数设置选中单元格-格式-数字-#0.00 3.图表中有效小数设置图表属性表- ...
Vue3中使用Monaco Editor代码编辑器记录～持续更新
Vue3中使用Monaco Editor代码编辑器记录-持续更新因为毕设需要用到代码编辑器,根据调研,我选择使用monaco-editor代码编辑器前端框架使用Vue3 + elementUI m ...
1个人70万行代码，20年持续更新，这款游戏号称开发到死，永不停更
梦晨博雯发自凹非寺量子位报道 | 公众号 QbitAI 这是一款「开发到死」,「永不停更」的游戏. 兄弟两人,一人开发,一人剧情,共同维持了这款游戏近20年. 现在的玩家刚刚打开它,往往会发 ...
CVPR 2022 论文/代码分类汇总！持续更新中！
关注公众号,发现CV技术之美 CVPR 2022 的论文官方还没有完全公布,但有作者陆续公布出来一些.为方便大家跟进论文,了解最新技术,CV君在Github建了一个仓库,对已经出来的论文(目前是340 ...
CVPR 2021 论文/代码分类汇总！持续更新中！
CVPR 2021 的论文官方还没有完全公布,但有作者陆续公布出来一些.为方便大家跟进论文,了解最新技术,CV君在Github建了一个仓库,对已经出来的论文(目前是340多篇)进行了按类别汇总.对于O ...
2、线程池篇 - 从理论基础到具体代码示例讲解（持续更新中......）
前言暂无. 一.线程篇有关线程部分的知识整理请看我下面这篇博客: 1.线程篇 - 从理论到具体代码案例最全线程知识点梳理(持续更新中-) 二.线程池基础知识线程池优点他的主要特点为: 线程复用 ...
基于PaddleOCR的FCENet论文总结和代码详解（持续更新）
Fourier Contour Embedding for Arbitrary-Shaped Text Detection 文末有总结一些CVPR有关OCR领域的最新论文及代码地址 Contribut ...
ROS与gazebo中常见问题【持续更新中】
ROS中常见问题 [gazebo] Spawn service failed. Exiting gazebo模型与服务器 gazebo中添加动态障碍物 [actor] [ROS] rosdep ins ...
Spring 常见问题（持续更新... ... ）
Spring 常见问题 1. Spring循环依赖相关问题 spring三层缓存的第二层缓存有什么用?循环依赖为什么要使用三级缓存?二级不能解决么?beanB从三级缓存中获取beanA以后提升到二级缓 ...

代码评审常见问题总结【持续更新】

代码评审常见问题总结【持续更新】相关推荐

最新文章

热门文章