目录code review分析一下我的错误代码行为1、写没必要的函数2、Promise传递不明值3、使用没必要try catch4、Promise.all并发限制5、nodejs中使
所谓code review,意思很明确,就是代码回顾,这个环节能帮你发现一些你代码中的不好的习惯,或者一些错误的行为。这个工作一般是团队的老大来做的,但是
我们的团队人均大佬
所以我们都是一起code review的,人多力量大,参加的人越多,越能让你发现自己的错误,从而能及时改正,收益颇丰!!!
注:下方代码都是虚构,主要为了讲解代码错误习惯,请勿较劲~
场景:我需要给某个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>
场景:我需要请求一个接口,请求返回一组数据,我需要把这组数据中的某个参数通过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)
场景:试异步操作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()
})
场景:我需要用Promise.all去进行并行执行一些异步操作,这个操作是在服务端的。。
// promises可能会有几十个,几百个,上千个
await Promise.all(promises)
缺陷:众所周知,服务端有时候是很脆弱的,可能你几十个并发就会把服务端给折腾的不要不要的了,所以控制并发是很重要的
改正:既然承受不住,那就控制并发呗,网上很多控制并发的方案。这里我就不说哪个方案比较好了。。自己实现也行,用库也行,看你们团队需要哪个吧。。
// 控制并发
await promiseAllLimit(promises)
场景:在后端那边进行文件操作,由于比较喜欢用同步方法,所以用了
const readData = fs.readFileSync(filepath);
fs.writeFileSync(targetPath, readData);
const workbook = xlsx.readFile(targetPath);
fs.unlinkSync(targetPath);
缺陷:Nodejs引以为傲的就是他的大部分方法都支持异步,所以它才能在高并发的场景中那么牛,所以尽量少用它的同步方法,性能会好一些
改正:使用它对应的异步方法吧,,不过要嵌套了。。很烦啊
场景:有两个参数,我要对他们进行一系列判断:
if (usename) {
...
}
if (passWord) {
...
}
if (!username || !password) {
...
}
缺陷:这个顺序是不对的,如果两个参数为空,则没必要进行另外两步
改正:改变顺序:
if (!username || !password) {
...
}
if (usename) {
...
}
if (password) {
...
}
以上就是代码规范需要防微杜渐code review6个小错误纠正的详细内容,更多关于代码规范code review错误纠正的资料请关注编程网其它相关文章!
--结束END--
本文标题: 代码规范需要防微杜渐code review6个小错误纠正
本文链接: https://lsjlt.com/news/151380.html(转载时请注明来源链接)
有问题或投稿请发送至: 邮箱/279061341@qq.com QQ/279061341
2024-01-12
2023-05-20
2023-05-20
2023-05-20
2023-05-20
2023-05-20
2023-05-20
2023-05-20
2023-05-20
2023-05-20
回答
回答
回答
回答
回答
回答
回答
回答
回答
回答
0