Corriger Try...Catch De La Syntaxe À L'Aide Async/Await

J'aime la planéité de la nouvelle Async/Await fonctionnalité disponible en caractères d'imprimerie, etc. Cependant, je ne suis pas sûr que j'aime le fait que je dois déclarer la variable, je suis awaiting à l'extérieur d'un try...catch bloc afin de l'utiliser plus tard. Comme:

let createdUser
try {
    createdUser = await this.User.create(userInfo)
} catch (error) {
    console.error(error)
}

console.log(createdUser)
//business
//logic
//goes
//here

S'il vous plaît corrigez-moi si je me trompe, mais il semble être la meilleure pratique pas de placer plusieurs lignes de la logique métier dans le try corps, donc je suis reste que l'alternative de déclarer createdUser à l'extérieur du bloc, l'affectant dans le bloc, puis à l'utiliser après.

Quelles sont les meilleures pratiques dans ce cas?

Les "meilleures pratiques" est d'utiliser ce qui fonctionne et est compréhensible et maintenable, etc.. Comment pourrions-nous répondre à cette question "correctement"? Je venais d'utiliser var, sachant que la variable serait hissé. C'est que "le mal"?
try/catch doit contenir exactement ce que vous voulez capturer une exception. Si vous êtes à la recherche explicitement pour les erreurs provenant de this.User.create() alors vous ne serait pas mettre autre chose à l'intérieur de la try/catch. Mais, il est parfaitement raisonnable de conception pour mettre tout un tas de logique à l'intérieur d'un bloc try. Tout dépend de comment/où vous voulez gérer une erreur et comment vous souhaitez concevoir votre code de gestion des exceptions et ce qui fait sens pour une opération donnée. Il n'y a pas de générique de meilleures pratiques. L'UN des génériques de la meilleure pratique est de s'assurer que vous attrapez et de gérer toutes les erreurs d'une manière appropriée.
async/await fait partie de ES2017 (de cette année), pas ES6 (qui a été publié il y a deux ans).

OriginalL'auteur Spencer Allen Gardner | 2017-06-20

  1. 20

    Il semble préférable de ne pas placer plusieurs lignes de la logique métier dans le try corps

    En fait, je dirais que c'est. Vous voulez généralement à catch tous exceptions de travailler avec la valeur:

    try {
    const createdUser = await this.User.create(userInfo);

    console.log(createdUser)
    //business logic goes here
} catch (error) {
    console.error(error) //from creation or business logic
}

    Si vous voulez attraper et à manipuler que les erreurs de la promesse, vous avez trois choix:

    • Déclarer la variable en dehors, et de la direction générale selon qu'il y a une exception ou pas. Qui peut prendre diverses formes, comme

      • attribuer une valeur par défaut à la variable dans l' catch bloc
      • return tôt ou re-throw une exception de la catch bloc
      • définir un indicateur de savoir si la catch bloc pris une exception, et de tester dans un if condition
      • tester la valeur de la variable d'avoir été affecté
      let createdUser; //or use `var` inside the block
try {
    createdUser = await this.User.create(userInfo);
} catch (error) {
    console.error(error) //from creation
}
if (createdUser) { //user was successfully created
    console.log(createdUser)
    //business logic goes here
}

    • Test l'exception interceptée à son type, et la poignée ou le renvoyer sur la base que la.

      try {
    const createdUser = await this.User.create(userInfo);
    //user was successfully created
    console.log(createdUser)
    //business logic goes here
} catch (error) {
    if (error instanceof CreationError) {
        console.error(error) //from creation
    } else {
        throw error;
    }
}

      Malheureusement, JavaScript standard (encore) n'ont pas de charge de la syntaxe pour des exemptions conditionnelles.

    • Utilisation avec deux rappels au lieu de try/catch. C'est vraiment le moins laid et ma recommandation personnelle aussi pour sa simplicité et sa justesse, en ne s'appuyant pas sur tagged des erreurs ou des regards de la valeur du résultat de la distinction entre la réalisation et le rejet de la promesse:

      await this.User.create(userInfo).then(createdUser => {
    //user was successfully created
    console.log(createdUser)
    //business logic goes here
}, error => {
    console.error(error) //from creation
});

      Bien sûr, il est livré avec l'inconvénient d'introduire des fonctions de rappel, ce qui signifie que vous ne peut pas aussi facilement break/continue de boucles ou de faire le début du returns à partir de l'extérieur de la fonction.

    Votre dernier exemple utilise .then() pour résoudre la promesse et de fournir un rappel, peut-être await n'a aucun effet.
    Il est awaiting la promesse renvoyé par la .then(…) appel.
    ne les lambdas renvoyé par la .then() appel doivent async mot-clé?
    Que s'ils utilisent await à l'intérieur.

    OriginalL'auteur Bergi