我是 JavaScript 的新手。我不知道在一个承诺中我可以在哪里使用两个拒绝;谁能告诉我,在一个承诺中有两个拒绝是不好的做法?如果是,可选解决方案是什么?
这是我的代码
async handler(ctx) {
/*
* sheet_to_json method converts excel file to JSON format
*/
let { s3Url, header } = ctx.params;
return new Promise((resolve, reject) => {
request({method: 'GET', uri: s3Url,encoding: null}, async function(err, res, data) {
if(err || res.statusCode !== 200) reject(err);
try{
const workbook = XLSX.read(data, {type:'buffer'});
const sheet_name_list = workbook.SheetNames;
let result = await XLSX.utils.sheet_to_json(workbook.Sheets[sheet_name_list[0]], {header});
resolve(result);
}
catch(error){
reject("Invalid File Cointaint");
}
})
});
}
您可以在 promise 回调中调用多个reject(err)
位置。 只有第一个执行的才重要。 对reject()
的后续调用将被忽略,因为一旦设置了 Promise 状态,就无法通过进一步调用 reject()
或 resolve()
来更改它。
拒绝时使用return
或使用else
语句来结束函数执行,因为继续执行其他无法解决或拒绝的事情通常是没有意义的,在某些情况下是有害的。
因此,在您的特定情况下,您可以添加如下所示的return
:
async handler(ctx) {
/*
* sheet_to_json method converts excel file to JSON format
*/
let {s3Url,header} = ctx.params;
return new Promise((resolve, reject) => {
request({method: 'GET', uri: s3Url, encoding: null}, async function(err, res, data) {
if (err || res.statusCode !== 200) return reject(err); // <==== return added here
try {
const workbook = XLSX.read(data, {type: 'buffer'});
const sheet_name_list = workbook.SheetNames;
let result = await XLSX.utils.sheet_to_json(workbook.Sheets[sheet_name_list[0]], {header});
resolve(result);
} catch (error) {
reject("Invalid File Cointaint");
}
})
});
}
或者,您可以添加其他:
async handler(ctx) {
/*
* sheet_to_json method converts excel file to JSON format
*/
let {s3Url, header} = ctx.params;
return new Promise((resolve, reject) => {
request({method: 'GET', uri: s3Url, encoding: null}, async function(err, res, data) {
if (err || res.statusCode !== 200) {
reject(err);
} else { // <======== else added here
try {
const workbook = XLSX.read(data, {type: 'buffer'});
const sheet_name_list = workbook.SheetNames;
let result = await XLSX.utils.sheet_to_json(workbook.Sheets[sheet_name_list[0]], {header});
resolve(result);
} catch (error) {
reject("Invalid File Cointaint");
}
}
})
});
}
是的,您可以在承诺中使用任意数量的reject
。只是废品需要在不同的流程上。意味着一个接一个地调用reject
reject
是行不通的。因为一个接一个地打电话给resolve
resolve
不会。
此外,为了确保其他代码不会运行,最好在reject
之前添加一个return
。
例如:
const checkIfRightPermission = new Promise((res, rej) =>{
//let's say we are doing an http request.
request.get('/getPermission?id=someid', function(err, res, body){
if(err){
//the request itself couldn't be completed SO
return rej();
}else{
if(sufficientPermission(body)){
}else{
//again
return rej()
}
}
})
});
您应该尝试将每个解决方案和拒绝保留在 if else 块中。
if (condition1) {
resolve(true);
} else if (condition2) {
reject(false);
} else {
reject(false);
}
例如,如果您要这样做:
if (condition1) {
resolve(true);
}
reject(false);
相反,您应该:
if (condition1) {
resolve(true);
return;
}
reject(false);
使用许多拒绝没有问题,这完全取决于您的逻辑,并且以这种方式使用它。
我唯一建议你改变的是 Promise 中的 try/catch 块,当你抛出异常时,下一个 promise.catch 将处理它。
看一看:
https://medium.com/datafire-io/es6-promises-patterns-and-anti-patterns-bbb21a5d0918