【代码评审总结-2021.10.21】

1.代码规范性

代码风格要求统一，项目中尽可能地使用相同的方法。
遍历数组的时候不要使用forEach() , 统一使用map()方法；

修改前：

 // 企业信息模糊查询
    getCompanyInfo(callback) {
      companyInfo(this.ruleForm.dwmc).then(res => {
        this.common.CheckCode(
          res,
          null,
          () => {
            let companyData = []
            res.data.forEach(item => {
              companyData.push({
                value: item.dwmc,
                zzjgdm: item.zzjgdm,
                dwxz: item.dwxz,
                glbm: item.glbm,
                glbmmc: item.glbmmc,
                qybh: item.qybh
              })
              this.companyData = companyData
            })
            callback(this.companyData)
          },
          () => {}
        )
      })
    }

修改后：

 // 企业信息模糊查询
    getCompanyInfo(callback) {
      companyInfo(this.ruleForm.dwmc).then(res => {
        this.common.CheckCode(
          res,
          null,
          () => {
            this.companyData = res.data.map(item => {
              return {
                value: item.dwmc,
                zzjgdm: item.zzjgdm,
                dwxz: item.dwxz,
                glbm: item.glbm,
                glbmmc: item.glbmmc,
                qybh: item.qybh
              }
            })
            callback(this.companyData)
          },
          () => {}
        )
      })
    }

2.用户体验性

校验提示要求设计合理和用户体验性好
注意添加前端校验，减少接口请求，能前端进行校验的就不要通过接口去校验
例如：至少上传一张图片添加前端校验处理。

修改之前效果：
接口提示校验信息，请求参数为空后端返回错误msg提示

正确实现：
前端添加校验，提示图片为必填项
先不要请求接口，如果照片校验为空则前端进行提示，校验成功之后才能请求接口

修改代码：

<template>
    <el-form :rules="rules">
        <el-form-item label="组织机构代码证明照片" prop="dmzp"></el-form-item>
    </el-form>
</template>
<script>
    dmzp:[
        { required: true, message: '请至少上传一张照片'}
    ]
</script>

3.代码尽量简化

默认为True的属性可以省略赋值

修改前：

    <!-- 搜索框 -->
    <cdt-search
      :formList="searchFormList"
      :defaultData="defaultData"
      :isSearch = "true"
      :isReset = "true"
      @handleSearch="$_SearchFormMixin_handleSearch"
      @handleBtnClick="handleBtnClick()"
      @timeChange="timeChange"
    >
    </cdt-search>

修改后：

    <!-- 搜索框 -->
    <cdt-search
      :formList="searchFormList"
      :defaultData="defaultData"
      isSearch
      isReset
      @handleSearch="$_SearchFormMixin_handleSearch"
      @handleBtnClick="handleBtnClick()"
      @timeChange="timeChange"
    >
    </cdt-search>

4.减少代码冗余

重复多次使用的操作尽量封装成方法来进行调用

5.过于复杂的三元表达式改为if…else…判断

6.代码注释要尽量简明阐述方法功能

7.数据处理中要注意进行非空判断

8.方法判断条件中应使用值类型判断 “ === ”