Wie gebe ich das Ergebnis von Versprechen zu Karte Funktion zurück

Question

Ich habe ein JavaScript-Versprechen, wo ich den Wert zur Kartenfunktion, die leichter zu sehen, als das Problem mit zweiten Versprechen zu erklären, zurückgeben möchte. Ich weiß, dass ich keinen Wert innerhalb des Versprechens zurückgeben kann, aber ich frage, ob es eine bessere Lösung dafür gibt. Dank

Mein Code:

const _     = require('lodash') 
    Boom    = require('boom'), 
    When    = require('when'), 
    User    = require('../../models/user'); 

var users; 
/** 
* Contact core functionality 
*/ 
users = { 


    /** 
    * Find user by id 
    * 
    * @param  {id} user id 
    * @returns  {Promise(User)} User 
    */ 
    findById: (id) => { 
     if (_.isEmpty(id)) { 
      return When.reject(Boom.badRequest('User id missing.')) 
     } 
     return User.get(id).run().then((user) => { 
      if (user) { 
       return When.resolve(user) 
      } 
      return When.reject(Boom.notFound('No user registered with provided id.')) 
     }).error((error) => { 
      return When.reject(error) 
     }); 
    }, 

    /** 
    * Returns all contacts of user from database 
    * @returns  [{Promise(Contact)}] contacts 
    */ 
    findContacts: (userId) => { 
     if (_.isEmpty(userId)) { 
      return When.reject(Boom.badRequest('User id missing.')) 
     } 
     return User.get(userId).run().then((user) => { 
      if (user) { 
       if(_.isObject(user.contacts)){ 
        user.contacts = _.map(user.contacts, (contact) => { 
         this.findById.then((user) => { 
          contact.user = user 
         }) 
         return contact 
        }) 
        return When.resolve(user.contacts) 
       } 
       return When.reject(Boom.notFound('No contacts registered with provided user.')); 
      } 
     }).error((error) => { 
      return When.reject(error) 
     }) 
    }, 
} 

Und das Problem hier:

   user.contacts = _map(user.contacts, (contact) => { 
        this.findById.then((user) => { 
         contact.user = user 
        }) 
        return contact 
       }) 

Answer 1

Da Sie die Wenn Versprechen Bibliothek verwenden, sollten Sie eine eigene Iteratoren, wie map und reduce verwenden, um Verarbeite deine Arrays.

Eine Sache, die hier unglaublich wichtig ist, ist, dass in Wenn map läuft "opportunistisch" (läuft alles auf einmal parallel), während reduce läuft in Serie ohne Überschneidungen. Mit anderen Worten, wenn Sie ein Array mit 1000 Benutzern haben, sendet map (wahrscheinlich) 1000 Anfragen gleichzeitig an Ihre Datenbank, während reduce nicht beendet wird, bevor die Datenbankanforderungen nacheinander ausgeführt wurden.

So könnte Ihr Code wie folgt aussehen:

// This mutates user.contacts, so you don't need to worry about the 
// 'initial value' in your reducer function 
When.reduce(user.contacts, (carry, contact) => { 
    return this.findById.then((user) => { 
     contact.user = user 
    }) 
}); 

Wenn Sie eine Tonne Benutzer tun müssen, verarbeiten, könnte es mehr Sinn, chunk machen Sie Ihre Daten in einer Reihe von kleineren, vernünftig bemessenen Arrays; Sie würden reduce die Chunks und map die Benutzer in jedem Stück. Dies würde bedeuten, dass das Array viel schneller verarbeitet werden könnte, als wenn es seriell erfolgt, ohne dass die Gefahr besteht, dass Ihre Datenbank beschädigt wird.

bearbeiten

// This mutates user.contacts, so you don't need to worry about the 
// 'initial value' in your reducer function 
// It also returns the array of users as `allContacts` 
var allContacts = When.reduce(user.contacts, (carry, contact) => { 
    return this.findById.then((user) => { 
     contact.user = user 
     carry.push(user); 
     return carry; 
    }) 
}, []);