Umgang mit mehreren Fängen in einer Versprechenkette

.catch funktioniert wie die try-catch-Anweisung, was bedeutet, dass Sie am Ende nur einen Haken brauchen:

repository.Query(getAccountByIdQuery)
        .then(convertDocumentToModel)
        .then(verifyOldPassword)
        .then(changePassword)
        .then(function(){
            res.status(200).send();
        })
        .catch(function(error) {
            if (/*see if error is not found error*/) {
                res.status(404).send({ error: "No account found with this Id" });
            } else if (/*see if error is verification error*/) {
                res.status(406).send({ OldPassword: error });
            } else {
                console.log(error);
                res.status(500).send({ error: "Unable to change password" });
            }
        });

Ich frage mich, ob es eine Möglichkeit gibt, die Kette irgendwie zu zwingen, an einem bestimmten Punkt anzuhalten, basierend auf den Fehlern

Nein. Sie können eine Kette nicht wirklich "beenden", es sei denn, Sie werfen eine Ausnahme, die bis zum Ende sprudelt. Siehe Benjamin Gruenbaums Antwort , wie das geht.

Eine Ableitung seines Musters wäre, die Fehlertypen nicht zu unterscheiden, sondern Fehler zu verwenden, die statusCode- und body-Felder haben, die von einem einzigen generischen .catch-Handler gesendet werden können. Abhängig von Ihrer Anwendungsstruktur kann seine Lösung jedoch sauberer sein.

oder wenn es einen besseren Weg gibt, dies zu strukturieren, um eine Form von Verzweigungsverhalten zu erhalten

Ja, Sie können verzweigen mit Versprechen . Dies bedeutet jedoch, die Kette zu verlassen und zum Verschachteln zurückzukehren - genau wie Sie es in einer verschachtelten if-else- oder try-catch-Anweisung tun würden:

repository.Query(getAccountByIdQuery)
.then(function(account) {
    return convertDocumentToModel(account)
    .then(verifyOldPassword)
    .then(function(verification) {
        return changePassword(verification)
        .then(function() {
            res.status(200).send();
        })
    }, function(verificationError) {
        res.status(406).send({ OldPassword: error });
    })
}, function(accountError){
    res.status(404).send({ error: "No account found with this Id" });
})
.catch(function(error){
    console.log(error);
    res.status(500).send({ error: "Unable to change password" });
});

Ich habe diesen Weg gemacht:

Am Ende verlassen Sie Ihren Fang. Und werfen Sie einfach einen Fehler, wenn dies in der Mitte Ihrer Kette geschieht.

    repository.Query(getAccountByIdQuery)
    .then((resultOfQuery) => convertDocumentToModel(resultOfQuery)) //inside convertDocumentToModel() you check for empty and then throw new Error('no_account')
    .then((model) => verifyOldPassword(model)) //inside convertDocumentToModel() you check for empty and then throw new Error('no_account')        
    .then(changePassword)
    .then(function(){
        res.status(200).send();
    })
    .catch((error) => {
    if (error.name === 'no_account'){
        res.status(404).send({ error: "No account found with this Id" });

    } else  if (error.name === 'wrong_old_password'){
        res.status(406).send({ OldPassword: error });

    } else {
         res.status(500).send({ error: "Unable to change password" });

    }
});

Ihre anderen Funktionen würden wahrscheinlich so aussehen:

function convertDocumentToModel(resultOfQuery) {
    if (!resultOfQuery){
        throw new Error('no_account');
    } else {
    return new Promise(function(resolve) {
        //do stuff then resolve
        resolve(model);
    }                       
}

Wahrscheinlich etwas spät zur Party, aber es ist möglich, .catch wie hier gezeigt zu nisten:

Mozilla Developer Network - Versprechen verwenden

Bearbeiten: Ich habe dies eingereicht, weil es die gewünschte Funktionalität im Allgemeinen bietet. Dies ist jedoch in diesem speziellen Fall nicht der Fall. Denn wie bereits von anderen ausführlich erläutert, soll .catch den Fehler beheben. Sie können beispielsweise keine Antwort an den Client in mehrfachen .catch Rückrufen senden, da ein .catch ohne explizite return löst es in diesem Fall mit undefined auf , wodurch das Fortfahren von .then ausgelöst wird, obwohl Ihre Kette nicht wirklich aufgelöst ist, was möglicherweise zu einem folgenden .catch führt um eine weitere Antwort an den Client auszulösen und zu senden, die einen Fehler verursacht und wahrscheinlich ein UnhandledPromiseRejection auf Ihre Weise wirft. Ich hoffe, dieser verschachtelte Satz hat für Sie einen Sinn ergeben.

Ich denke, die Antwort von Benjamin Gruenbaum oben ist die beste Lösung für eine komplexe Logiksequenz, aber hier ist meine Alternative für einfachere Situationen. Ich verwende einfach ein errorEncountered-Flag zusammen mit return Promise.reject(), um nachfolgende then- oder catch-Anweisungen zu überspringen. So würde es so aussehen:

let errorEncountered = false;
someCall({
  /* do stuff */
})
.catch({
  /* handle error from someCall*/
  errorEncountered = true;
  return Promise.reject();
})
.then({
  /* do other stuff */
  /* this is skipped if the preceding catch was triggered, due to Promise.reject */
})
.catch({
  if (errorEncountered) {
    return;
  }
  /* handle error from preceding then, if it was executed */
  /* if the preceding catch was executed, this is skipped due to the errorEncountered flag */
});

Wenn Sie mehr als zwei Then/catch-Paare haben, sollten Sie wahrscheinlich die Lösung von Benjamin Gruenbaum verwenden. Dies funktioniert jedoch für eine einfache Einrichtung.

Beachten Sie, dass das letzte catch nur return; und nicht return Promise.reject(); enthält, da es kein nachfolgendes then gibt, das wir überspringen müssen, und dies würde als eine nicht behandelte Promise-Zurückweisung gelten, die dem Knoten nicht gefällt. Wie oben geschrieben, wird das letzte catch ein friedlich gelöstes Versprechen zurückgeben.

Anstelle von .then().catch()... können Sie .then(resolveFunc, rejectFunc) ausführen. Diese Versprechenkette wäre besser, wenn Sie die Dinge auf dem Weg erledigen würden. So würde ich es umschreiben:

repository.Query(getAccountByIdQuery)
    .then(
        convertDocumentToModel,
        () => {
            res.status(404).send({ error: "No account found with this Id" });
            return Promise.reject(null)
        }
    )
    .then(
        verifyOldPassword,
        () => Promise.reject(null)
    )
    .then(
        changePassword,
        (error) => {
            if (error != null) {
                res.status(406).send({ OldPassword: error });
            }
            return Promise.Promise.reject(null);
        }
    )
    .then(
        _ => res.status(200).send(),
        error => {
            if (error != null) {
                console.error(error);
                res.status(500).send({ error: "Unable to change password" });
            }
        }
    );

Hinweis: Die if (error != null) ist eine Art Hack, um mit dem neuesten Fehler zu interagieren.