快结款 PC 管理后台

2021-04-09

• 记录者：朝润
• 代码审评者：静文，遗森，朝润
• 代码编写者：国宇

建议意见

1. key 不用 index
2. vue 中属性书写顺序，components -> props -> data -> computed -> watch -> 生命周期 -> methods
3. 组件标签使用时用小写
4. props 的默认值属性名为 default，而非 value。必传的属性无须设置 default
5. props 和 data 全部要备注
6. props 只有类型时不需要用对象声明
7. data 中无数据时可删掉
8. colums 单词拼写错误
9. methods 中类似 list 这种方法名改成“动词 + 名词”的形式
10. 每个方法前换行
11. api 中只有一个实例的 class 可以直接写成对象
12. axios 可封装 get/post 等方法，统一使用方式
13. jsdoc 注释对齐
14. route.meta 对象一定存在，后面不需要问号
15. 没有对错误进行额外处理时不需要 catch 又 throw
16. 一条语句的代码放 template 中
17. 常量中单词之间用下划线分隔
18. 变量用驼峰，不用下划线
19. (new Date()).getTime() 改成 Date.now()
20. import api 的地方统一用小写
21. v 这种名字改掉
22. 清空图表改用 changeData 方法

2021-03-25

• 记录者：朝润
• 代码审评者：静文，遗森，朝润
• 代码编写者：国宇

建议意见

1. 常量要用下划线分隔
2. onDeleteEmpty 参数名
3. localStorage 中基本数据类型不需要 JSON.stringify
4. error.response.data.error.message 太多点
5. webpackChunkName 值改成小写
6. * 重定向的配置写在 home.js 外
7. actions 没必要
8. 登录页面层级过多
9. i 标签自闭合
10. 属性的顺序，无冒号 -> 指令 -> 有冒号 -> 事件
11. 用 scoped 后不用写没必要的选择器嵌套
12. html，js 和 css 写在同一个 vue 文件中
13. 登录校验规则过于简单
14. vue-particles 的引入方式，局部注册更好
15. home 组件应叫 layout
16. 类名命名用小写 + 横线
17. 类名 p24 w200 等要改掉
18. 组件标签使用时用小写 + 横线
19. 组件注册不需要自己写成小写
20. localStorage 中 userInfo 的获取应该统一
21. visible 要具体一些
22. data/props/methods 等地方所有值都要写备注
23. menu 中 props 要加类型和注释
24. parseNum 过滤器可以处理没有值的情况
25. moment 的引入没必要
26. statistics/index.js 等文件中，e 这种变量名要改
27. JS 变量命名，用驼峰，不要下划线
28. year*num 什么意思，备注一下
29. tabKeys/key 不要用数字
30. 对象属性统一换行
31. 方法不要用名次
32. 不必要的三目运算符改成 ||
33. 标签换行时缩进不对
34. methods 换行
35. waitingForReview 119 行 pagination 无需浅拷贝
36. 单行 return 可放在前一句语句前
37. 颜色用 6 位大写
38. 没用的已注释代码删掉
39. 简单的 switch 语句优化成映射的方式
40. 模板中不必要的箭头函数改成普通函数调用
41. 箭头函数中的 e 可换成 $event
42. order-detail 组件 moment.diff 后面的参数改成 new Date() 会更好
43. order-detail 组件 76 和 82 行的 map 改成 forEach
44. order-detail 组件 89 行 catch 又 throw 没必要
45. 转换后端字段时用 $ + 驼峰

2021-03-19

• 记录者：朝润
• 代码审评者：静文，朝润
• 代码编写者：国宇

建议意见

1. 常量文件夹不要放在 common 文件夹中
2. localstorage 文件不需要封装
3. deleteEmpty 是否每次请求都有必要
4. 请求拦截中 Object.assign 赋值和返回不需要分两行
5. 是否需要统一显示接口报错信息
6. 重写 router 对象 push 方法处要加备注
7. router 变量的定义没必要，可直接传参
8. 路由跳转统一使用 path
9. 路由 404 页面配置
10. case 后的逻辑一样时可合并
11. routerTo 的导出应该由更好的方式
12. 登录失败的提示不应该时成功的样式

2021-03-12

• 记录者：朝润
• 代码审评者：静文，朝润
• 代码编写者：国宇

建议意见

1. import 语句放最顶部
2. 尽量减少 dom 的嵌套
3. style 和 template 标签之间添加空行
4. 判断空对象的方式
5. 注释位置导致的换行问题
6. 应该在 axios 拦截器中设置 token
7. localStorage 暂时没有设置过期时间的需求
8. 路由 push 忽略警告的代码去掉
9. routes 数组的 concat 多余
10. 判断路由不存在时跳转回首页改设置
11. 路由的 path 不能有大写字母
12. 页面跳转尽量使用 path 而不是 name
13. 没有 token 和 token 失效时通过事件通知来跳转到登录页
14. 路由的 webpackChunkName 不能设为同一个名字
15. 引入路径用别名，不用相对路径
16. 路由引入不用定义变量
17. meta 中没有值的参数删掉
18. 路由中 component 为空组件时需要写注释
19. 图片放 images 目录中
20. 公共 scss 文件放 common/css 中
21. _publicStyle 文件名不规范
22. filters.js 中不必要的定义变量
23. filters.js 中不应该引入 vue
24. 嵌套的 if 语句合并
25. class 中不要定义不必要的实例属性
26. local-storage.js 类改成导出函数
27. models 文件夹不要放在 common 文件夹中
28. axios 路径中参数的传参方式统一用对象
29. style 标签添加 scoped 属性
30. css 属性的顺序
31. 文件命名，bj.jpg -> bg.jpg

2021-03-05

• 记录者：朝润
• 代码审评者：静文，朝润
• 代码编写者：国宇

建议意见

1. gzip 的配置
2. 别名必须以 @ 开头
3. alias vue$ (旧版的webapck引入方式不支持es6模块化，现在已删除)
4. eslint 配置同步和修改
5. eslint 配置重复
6. .editorconfig
7. index.html 需要加 seo
8. noscript 提示
9. 图片轮播没用到
10. 有些库不需要全局加载
11. 引入的东西要写备注
12. 去除无用组件，如轮播，可用 ant 的
13. vue-axios 是做什么的(已删除)
14. 路由守卫不应该写在 axios 中
15. lodash 不需要引入，或者按需加载
16. localStorage.cookie 文件名不规范
17. 变量减少不必要的重命名
18. 命名尽量用全称，不用缩写
19. setTimeout 的延时时间
20. .vue 文件不同块的顺序
21. css 中 import 放最上面
22. 无用代码直接删掉