約束の誰かがきれいに失敗した場合、約束全体を拒否したいと思っています。私はこの拒否を「キャッチ」し、エラー通知を送信したいと思います。私はそれに次のコードを実装しました:その中で約束を拒否する方法
let reportMetaData = api.ajaxGet(api.buildV3EnterpriseUrl('reports' + '/' + params.report_id))
.catch(error => {
if (error.status === constants.HTTP_STATUS.GATEWAY_TIMEOUT) {
this.notify.error(this.translate('reports.report_timedout'), this.translate('reports.report_timedout_desc'));
} else {
this.send('error', error);
}
});
let aggregateData = reportMetaData.then(success => {
try {
return api.xmlRequest('GET', success.aggregationUrls.elements[0].url);
} catch (error) {
return Promise.reject();
}
}).then(rawData => {
try {
return JSON.parse('{' + rawData + '}');
} catch (error) {
return Promise.reject();
}
}, error => Promise.reject(error));
let aggregateReport = aggregateData.then(data => {
if (!data || !data.report) {
return Promise.reject();
}
return data.report;
}).catch(error =>{
this.notify.error(this.translate('reports.report_timedout'), error);
});
ご覧のとおり、わかりやすく複雑です。私はこれを単純化する方法はありますか?誰かが失敗すると失敗するという約束をすべて拒否する最も簡単な方法を望んでいます。どのように私はその機能の中からそれをするのですか?また、スローされたエラーが、クロムのコンソールに途切れることなく、誤ったエラーとして表示されるようです。私はそれを捕まえたにもかかわらず、なぜそれは泡立つのですか?
私は間違っているか、あなたが既にPromise.reject()を返すことでこれをやっていませんか?私はあなたが単純にいくつかのエラーキャッチャーを残すことができると思うし、それはすべて最後のものに行くだろう... –
主な問題は、実際にチェーンを持っていないように思われる。お互いに関係のないいくつかの別々の約束があります。すべてのコードを1つの約束のチェーンにまとめるつもりですか?既に複数の場所でそれを行っているので、 '.then()'の内部から拒否する方法を既に知っているので、あなたの質問は今のところ答えることができません。しかし、あなたは 'Promise.all()'のようなものと一緒にあなたの約束を集めるか、実際にそれらを1つのチェーンにしなければなりません。私たちはあなたがコードのために何をしようとしているのか分かりません。 – jfriend00
私は自分の質問を修正しました。今はっきりしていることを願っています。基本的に私は自分のコードをきれいにしたい。 Btw私は約束を一緒に連鎖している。たとえば、aggregateReportはaggregateDataの場合は「then」で、aggregateDataの場合はindividualReportも「then」です。したがって、これらは連鎖しています。 –