如何将代码规范需防微杜渐，code review纠正6个小错误改写为一个长尾？

本文共计1296个文字，预计阅读时间需要6分钟。

目录 + code review + 分析我的错误代码行作为：

1.写了不必要的函数 2. Promise传递不明值 3. 使用了不必要的try+catch 4. Promise.all并发限制 5. Nodejs中使用过度的Async函数 6. 判空要放前面 code review 所说

目录

• code review
• 分析一下我的错误代码行为
• 1、写没必要的函数
• 2、Promise传递不明值
• 3、使用没必要try catch
• 4、Promise.all并发限制
• 5、Nodejs中使用过多sync函数
• 6、判空要放前面

code review

所谓code review，意思很明确，就是代码回顾，这个环节能帮你发现一些你代码中的不好的习惯，或者一些错误的行为。这个工作一般是团队的老大来做的，但是

我们的团队人均大佬

所以我们都是一起code review的，人多力量大，参加的人越多，越能让你发现自己的错误，从而能及时改正，收益颇丰！！！

注：下方代码都是虚构，主要为了讲解代码错误习惯，请勿较劲~

分析一下我的错误代码行为

1、写没必要的函数

场景：我需要给某个html根标签设置一个属性，而这个行为是需要在项目初始化的时候去做的，我当时的做法是：

// utils export const setXXXyyy = () => { const tag = document.tag; const xxx = tag.getAttribute('xxx'); if (xxx || xxx === null) { tag.setAttribute('xxx', 'yyy'); } }; // app.tsx import { setXXXyyy } from 'utils' // 初始化时 setXXXyyy()

缺陷：可能我有考虑到以后会在此函数里去做更多事，但是现阶段这么写确实有点多余

改正：直接在模板html里把此属性加到标签上即可

// public/index.html <tag xxx="yyy"></tag>

2、Promise传递不明值

场景：我需要请求一个接口，请求返回一组数据，我需要把这组数据中的某个参数通过Promise带出去

// 请求方法 const request = () => { return new Promise(async (resolve) => { const res = await axios(...) resolve(res) }) } // 使用 const res = await request() console.log(res.data.answer)

缺陷：直接把res传递出去了，如果没有ts的限制，那么下一个使用request的开发人员根本不知道这个res里有什么，也自然就不知道有answer这个目标数据了

改正：直接把目标数据answer传递出去

const request = () => { return new Promise(async (resolve) => { const res = await axios(...) // 改正 resolve(res.data.answer) }) } const answer = await request() console.log(answer)

3、使用没必要try catch

场景：试异步操作1，如果失败的话试异步操作2，如果失败进行操作3

action1().catch(() => { try { await action2() } catch(e) { throw e } }).catch(() => { action3() })

错误：没必要使用try catch，await失败之后，会自动返回一个失败的Promise，通过链式调用，会执行下一个catch

改正：去除try catch

action1().catch(() => { await action2() }).catch(() => { action3() })

4、Promise.all并发限制

场景：我需要用Promise.all去进行并行执行一些异步操作，这个操作是在服务端的。。

// promises可能会有几十个，几百个，上千个 await Promise.all(promises)

缺陷：众所周知，服务端有时候是很脆弱的，可能你几十个并发就会把服务端给折腾的不要不要的了，所以控制并发是很重要的

改正：既然承受不住，那就控制并发呗，网上很多控制并发的方案。这里我就不说哪个方案比较好了。。自己实现也行，用库也行，看你们团队需要哪个吧。。

// 控制并发 await promiseAllLimit(promises)

5、Nodejs中使用过多sync函数

场景：在后端那边进行文件操作，由于比较喜欢用同步方法，所以用了

const readData = fs.readFileSync(filepath); fs.writeFileSync(targetPath, readData); const workbook = xlsx.readFile(targetPath); fs.unlinkSync(targetPath);

缺陷：Nodejs引以为傲的就是他的大部分方法都支持异步，所以它才能在高并发的场景中那么牛，所以尽量少用它的同步方法，性能会好一些

改正：使用它对应的异步方法吧，，不过要嵌套了。。很烦啊

6、判空要放前面

场景：有两个参数，我要对他们进行一系列判断：

• 1、判断数据表里有没有这两个参数
• 2、判断这两个参数是否重叠
• 3、判断这两个参数是否为空

if (usename) { ... } if (password) { ... } if (!username || !password) { ... }

缺陷：这个顺序是不对的，如果两个参数为空，则没必要进行另外两步

改正：改变顺序：

• 1、这两个参数是否为空
• 2、判断数据表里有没有这两个参数
• 3、判断这两个参数是否重叠

if (!username || !password) { ... } if (usename) { ... } if (password) { ... }

以上就是代码规范需要防微杜渐code review6个小错误纠正的详细内容，更多关于代码规范code review错误纠正的资料请关注自由互联其它相关文章！