分享几个我平时写异步代码的小技巧以及我之前看到过得一些不好的写法。
在 reject Promise
时强制使用Error
对象,方便浏览器底层更容易的分配堆栈以及查找堆栈。
// bad
Promise.reject('error reason');
// good
Promise.reject(new Error('error reason'))
不要把async
函数传递给Promise
构造函数,因为没有必要,其次如果async
函数异常,那么你的promise
并不会reject
。
// bad
new Promise(async (resolve, reject) => {})
// good
new Promise((resolve, reject) => { async function(){coding....}()})
尽可能将这写异步的任务改为并发,可以大幅提高代码执行效率
// bad
for(const apiPath of paths) {
const { data } = await request(apiPath)
}
// good
const results = []
for(const apiPath of paths) {
const res = resquest(apiPath)
results.push(res)
}
await Promise.all(results)
// bad
new Promise((resolve, reject) => {
if(isOK) return 'ok'
return 'not ok'
})
// good
new Promise((resolve, reject) => {
if(isOK) resolve('ok')
reject(new Error('not ok'))
})
// bad
p1((err, res1) => {
p2(res1, (err, res2) => {
p3(res2, (err, res3) => {
p4(res3, (err, res4) => {
console.log(res4)
})
})
})
})
// good
const res1 = await p1()
const res2 = await p1(res1)
const res3 = await p1(res2)
const res4 = await p1(res3)
console.log(res4)
// bad
asyncPromise().then(() => {})
// good
asyncPromise().then(() => {}).catch(() => {})
// bad
const result = await asyncPromise()
// good
try {
const result = await asyncPrmise()
} catch() {
// do something
}
// bad
function getUserInfo() {
return userInfo
}
await getUserInfo()
// good
async function getUserInfo() {
return userInfo
}
await getUserInfo()
async function getUserInfo() {
return userInfo
}
// bad
if(getUserInfo()) {}
// good
if(await getUserInfo()) {}
// better
const { userInfo } = await getUserInfo()
if(userInfo) {}