壁虎找票

2021-09-23

• 记录者：任演
• 代码评审者：国宇、朝润、遗森、任演
• 代码编写者：惠贤

建议意见

接单页面

index.vue

0. 样式名尽量少用属性名
1. 组件名没内容自闭合
2. 组件属性在前，方法在后， 属性 -> : -> 指令
3. 空标签可使用伪类代替
4. 组件传空对象（283行）
5. 组件props可设置默认值，单个类型不需要使用数组
6. 组件传number，布尔类型的值时，使用双引号，使用冒号
7. 属性值不使用单引号，使用双引号
8. html嵌套代码比较长时需要换行
9. import 不是常用的需要写注释
10. data 变量需要写注释
11. putAway 默认应当为1或2,值只有1，2时使用布尔类型（379行）
12. TODO后面添加冒号 可高亮
13. 方法、生命周期之间换行
14. 全局事件名用常量，大写（393行）
15. 1,2这些数字使用常量，统一写到constant.js里面去（407行）
16. 判断等于true false时不需要 === true
17. 一些魔法数字提示可以加上忽略（421行）
18. if判断可以写成一行，30写成常量（447行等）
19. showRuleList可以使用计算属性
20. toBlack和toFront多余
21. {{}}、? 等左右需要空格
22. 前两个if判断执行的内容是一样的（515行）
23. 能使用现有的变量就不要新建变量
24. 页面顶部注释使用html的注释
25. 颜色使用变量

custom-list.vue

26. 样式名称使用中划线连接
27. length 使用中文括号，不需要span（137行）
28. 竖线分隔使用伪类
29. props的变量注释写上有可能的值
30. await后面去掉then（238行）
31. .catch 换成try catch
32. 一些方法可以合并成一个

2021-10-11

• 记录者：遗森
• 代码评审者：国宇、朝润、遗森、任演
• 代码编写者：惠贤

pages/order-receiving/index.vue

0. 样式不用写前缀
1. 标签属性统一换行
2. i 等标签自闭合
3. 组件的数据操作可以尽量放在组件里面操作
4. ref名字统一用驼峰式命名
5. class不要用中文命名（i标签的class）
6. span标签加className（161行）

custom-list.vue

0. try/catch 错误码的使用（是否写死或使用接口返回的错误码）

notion-card.vue

0. css颜色尽量不要用red这些单词
1. margin等大小尽量取整
2. v-deep 样式可以写在一起
3. 多余的代码，如（241行）删掉
4. className尽量语义化，少用c、b这些简写
5. 缩进要对齐（407、412等）
6. import多个属性时可换行并分别写注释（558行）
7. 金额计算要使用BigNumber.js来处理（centToTenThousand方法）
8. 方法名用驼峰（666行）
9. return 值可以缩写回一行（getDiscountBankType方法）
10. $event.emit 的方法常量要写在固定的文件里面
11. catch里面的nextTick是否需要
12. catch 应该用error而不是res