【代码评审总结-2022.05.26】

组内成员代码评审暴露问题总结,开发时需要注意:

P1:
  • 页面搜索栏搜索项内容需要对齐
  • 样式不要写行内样式,在css里面写
  • 没有涉及到this指向问题的方法里,不需要定义 _this
  • 判断接口返回状态使用统一方法 this.common.CheckCode( )或者项目里其他类似方法
  • 判断条件中使用 === ,!== 不要使用 ==,!=
  • page,rows不要重复声明,查询方法有问题,需要改一下
  • 多余方法删除或者注释掉
  • 对象赋值可以使用解构赋值,代码简洁
  • this.marker 定义清除数据问题
  • 分页查询方法使用混合方法 SearchFormMixin, PaginationTableMixin (避免每个页面都要分别赋值,减少代码冗余)
  • 三元表达式 判断情况考虑全面一些
P2:
  • 按钮选中效果,边框问题优化
  • 汉化方法可以处理为公用方法
  • 不需要的的async,await去掉
  • 布尔值默认为true的去掉===true
  • formData赋值代码冗余部分优化
  • 多余this指向赋值去掉 const that = this ×
  • 使用mapState取状态值
  • 汉化方法中做非空判断
  • 0
    点赞
  • 0
    收藏
    觉得还不错? 一键收藏
  • 0
    评论

“相关推荐”对你有帮助么?

  • 非常没帮助
  • 没帮助
  • 一般
  • 有帮助
  • 非常有帮助
提交
评论
添加红包

请填写红包祝福语或标题

红包个数最小为10个

红包金额最低5元

当前余额3.43前往充值 >
需支付:10.00
成就一亿技术人!
领取后你会自动成为博主和红包主的粉丝 规则
hope_wisdom
发出的红包
实付
使用余额支付
点击重新获取
扫码支付
钱包余额 0

抵扣说明:

1.余额是钱包充值的虚拟货币,按照1:1的比例进行支付金额的抵扣。
2.余额无法直接购买下载,可以购买VIP、付费专栏及课程。

余额充值