记一次代码评鉴

前言

近期公司组织了一次代码评鉴，在这边记录下学习到的一些规范吧

案例

案例1

• 参数过多，改为对象好一些

const start = (filename, version, isFirst, branch, biz) => {
    // ....
}

案例2

• query不应该直接透传

• 对象解构可能导致覆盖，可以调下顺序

// ...
await axios.post('xxx', {
    data: {
        host: 'xxx'
        ...getQuery()
    }
})

案例3

• 超过三个条件的判断抽出为表达式或者函数

• 魔法数字用变量代替

• 与和非不一起使用

if (bottom < boxMaxH && topRemain < boxMax || top > 20) {
}