Bluebird de la Promesse.tous les() la méthode quand une promesse est dépendante d'une autre

Je suis en train d'écrire un peu de code qui ressemble actuellement ce parce que j'ai des dépendances dans mon code. Je me demandais si il y avait une manière plus propre de le faire avec la Promesse.tous les()? Voici mon pseudo code:

        return someService.getUsername()
            .then(function(username) {
                user = username;
            })
            .then(function() {
                return someService.getUserProps(user);
            })
            .then(function(userProps) {
                userProperties = userProps;
                return someService.getUserFriends(user);
            })
            .then(function(userFriends) {
                friends = userFriends;
            })
            .catch(error)
            .finally(function(){
                //do stuff with results
            });

La chose importante est que j'ai besoin de l'utilisateur avant que je puisse faire les deux appels pour getUserProps() et getUserFriends(). Je pensais que je pouvais utiliser Promesse.tous les() de cette façon:

var user = someService.getUsername()
    .then(function(username) {
        user = username;
    })
var getUserProps = someService.getUserProps(user);
var getUserProps = someService.getUserFriends(user);

return Promise.all(user, getUserProps, getUserFriends, function(user, props, friends) {
    //do stuff with results
})

Mais je ne peux pas obtenir que cela fonctionne. Est-ce le bon cas d'utilisation .tous?

InformationsquelleAutor Jeff | 2014-09-17
  1. 9

    Promise.all() est conçu pour fonctionnement en parallèle lorsque vous lancez un tas d'opérations asynchrones pour s'exécuter en même temps et puis il vous dit quand ils sont tous fait.

    Il n'a pas d'ordre un rapport à la fin de l'autre en quelque sorte. Donc, vous ne pouvez pas l'utiliser pour attendre que l'utilisateur soit prêt et ensuite les autres opérations utilisation de cet utilisateur. Il n'est tout simplement pas conçu pour faire.

    Vous pourriez obtenir de l'utilisateur en premier et ensuite quand il est complet, vous pouvez utiliser Promise.all() avec vos deux autres opérations qui je pense peut être exécuté en même temps et ne dépendent les uns des autres.

    var user;
someService.getUsername().then(function(username) {
    user = username;
    return Promise.all(getUserProps(user), getUserFriends(user));
}).then(function() {
    //do stuff with results array
}).catch(function() {
    //handle errors
});
    InformationsquelleAutor jfriend00
  2. 4

    Vous pouvez utiliser .all mais vous allez avoir pour s'assurer qu'elles sont exécutées de manière séquentielle votre code, vous pouvez le faire en .thening eux comme vous l'avez fait. Si vous faites cela, vous devez utiliser .join qui est une abréviation pour .all([...]).spread(....

    var user = someService.getUsername();
var props = user.then(getUserProps)
var friends = user.then(getUserFriends)
Promise.join(user, props, friends, function(user, props, friends) {

    //everything is available here, everything is synchronized
});

    Si ce que vous cherchiez à résoudre est la fermeture/l'imbrication question - alors, c'est la façon de le faire.

    • Cela ne semble pas fonctionner pour moi parce que j'ai besoin de passer une variable à getUserProps() et je ne l'ai pas, lorsque j'ai mis les accessoires variable sur la deuxième ligne.
    • la variable utilisateur est automatiquement transmise à l'getUserProps fonction, c'est la promesse de la résolution de la valeur, tout comme dans votre exemple. Vous pouvez utiliser .bind si vous avez besoin de passer des paramètres supplémentaires ou une fonction anonyme.
    • ohhh.... donc, les accessoires de définition aurait pu être: var props = utilisateur.ensuite(getUserProps(utilisateur)); ?
    • Non, puisque l'utilisateur est une promesse que déjà résout par le nom d'utilisateur. C'est tout simplement user.then(getUserProps) qui fait la même chose que user.then(function(user){ return getUserProps(user); }) seulement plus court et plus concis. Le point ici est que la promesse agit comme un proxy pour la valeur directement.
    InformationsquelleAutor Benjamin Gruenbaum
  3. 1

    Promesse.tous les() est une façon d'exécuter une liste de promesses en parallèle, mais si nous voulons exécuter une liste de promesses dans une série où l'on dépend de l'autre, nous avons à résoudre un peu différemment

    //Promise returning functions to execute
function doFirstThing(){ return Promise.resolve(1); }  
function doSecondThing(res){ return Promise.resolve(res + 1); }  
function doThirdThing(res){ return Promise.resolve(res + 2); }  
function lastThing(res){ console.log("result:", res); }

var fnlist = [ doFirstThing, doSecondThing, doThirdThing, lastThing];

//Execute a list of Promise return functions in series
function pseries(list) {  
  var p = Promise.resolve();
  return list.reduce(function(pacc, fn) {
    return pacc = pacc.then(fn);
  }, p);
}

pseries(fnlist);  
//result: 4
    InformationsquelleAutor talal7860