对 Review 的评论进行分级，不同级别的结果打上不同的 Tag

  • blocker: 表示这个代码行的问题必须要修改！
  • optional：表示这个代码行的问题可改可不改！
  • question：表示对这个代码行不理解，有问题需要问，被审查者需要针对问题进行回复澄清！

示例

每次代码审查完成后, 以示例中的形式发送到群里进行同频汇报。
英文缩写: CMNT; 中文全称: v. 注释, 评论; 英文全称: Comment.
英文缩写: CVR；中文全称: v. 覆盖, 涉及, 包含；英文全称: Cover .

CR: [blocker]【团队管理】{业务员管理}-业务员绑定客户时，增加客户标签筛选
CMNT: 接口请求重复，在产品模块下有 getProduct 方法，与此次新增的 getProductList 方法 注册的接口是一样的.
CVR: [VBOM]
gbcloud-admin/src/views/dataCenter/components/PanelGroup.vue
gbcloud-admin/src/views/dataCenter/coupon/index.vue
gbcloud-admin/src/views/dataCenter/staffs/check.vue

审查历史记录

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
2021-05-20

CR: [question]【活动模块】编辑活动面向客户时增加对筛选结果全部选中
CMNT: 全部选中没有问题, 但是已选客户有点疑问, 为什么默认显示数量为1, 切换到已选客户却又没有数据, 需要进一步确认是否为bug.
CVR: [VBOR]
Gbcloud/gbcloud-admin/src/views/activity/components/SelectCustomer.vue

CR:【企业管理模块】企业资质优化：资质类型优化、申请客商关系优化
CMNT: 企业资质列表已更换独立组件使用并维护, 没有问题.
CVR: [VBOR]
Gbcloud/gbcloud-admin/src/views/enterprise/qualifications/index
Gbcloud/gbcloud-admin/src/views/enterprise/basicInfo/index
Gbcloud/gbcloud-admin/src/views/login/index

CR: [optional]【商城】供应商logo加载后，做缓存处理
CMNT: static 在 webpack 配置了路径别名，url('../../../../static/images/avatar.jpg'); 这种路径可以直接写成 url('static/images/avatar.jpg'); 在banner.vue组件 19行处，比较is_open == 0可能会导致意外的类型强制，使用类型安全的等于运算符===和！==代替常规的对等运算符==和 != 被认为是一种好习惯。
CVR: [VBOQ]
Gbcloud/gbcloud-shop/src/components/headers/index.vue
Gbcloud/gbcloud-shop/src/views/homePage/components/banner/index.vue
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
2021-04-27

CR: [optional]  【前端】【商城】侧边工具栏UI优化
CMNT: 样式固定单位推荐使用双数, 无意义的注释代码最好删除掉，如果有用的代码注释应该写清楚注释的目的，整体没有什么影响，可以直接通过审查。
CVR: [VBOM]
gbcloud-shop/src/components/toolbar/index.vue	Line: 657
gbcloud-shop/src/components/toolbar/index.vue	Line: 40
gbcloud-shop/src/components/toolbar/index.vue	Line: 550


CR: [optional]  【前端】【后台-订单管理】订单详情新增标签展示
CMNT: 
可以适当的使用 switch 替代 if else（效率更高）;
多个同样的代码片段出现在一个页面的时候就可以考虑有没有办法优化（这个涉及整个页面的设计，跟本次增加一个标签的任务没有直接关系）;
不变的变量赋值优先使用 const 关键字（const  效率优于 let、var）;
For 循环体内的变量使用let（更加契合循环体制）;
var 要么不用，要么当全局变量使用（现在变量有的块级作用域，已经不适用var，let const足够应付所有的场景，所以没必要使用了）。


CVR: [VBON]
gbcloud-admin/src/views/settings/parameters/valetOrder/index.vue
gbcloud-admin/src/views/orderMg/market/details.vue
gbcloud-admin/src/views/orderMg/market/index.vue