Q Promise Nodejs, как решить в цикле

у меня есть код, написанный в nodejs, что меня сбивает с толку, используя Q Promises

theFunction()
.then(function(data) {
    var deferred = Q.defer()
    var result = [];
    for(i=0; i < data.length; i++) {

        secondFunc(data.item)
        .then(function(data2) {
            data.more = data2.item
        });
        result.push(data);

    }

    deferred.resolve(result);
    deferred.promise();

});

я хочу, чтобы данные во второй функции внутри цикла могли вставляться в результат

поэтому мои предыдущие данные такие

[
    {
        id: 1,
        item: 1,
        hero: 2
    },
    {
        id: 1,
        item: 1,
        hero: 2
    }
]

и так вот

[
    {
        id: 1,
        item: 1,
        hero: 2,
        more: {
            list: 1
        }
    },
    {
        id: 1,
        item: 1,
        hero: 2,
        more: {
            list: 4
        }

    }
]

Я пробовал несколько способов начать с ввода команды deferred.resolve(); оператор в цикле и показывает только 1 данные есть какое-либо решение?


node.js loops promise q
person Nazril Faudi    schedule 21.08.2014    source источник

Ответы (2)


arrow_upward
10
arrow_downward

Вместо deferred.resolve() для массива, который будет разрешен немедленно, используйте Q.all, который ожидает массив промисов:

theFunction()
.then(function(data) {
    var result = [];
    for(var i=0; i < data.length; i++) (function(i){
        result.push(secondFunc(data[i].item)
        .then(function(data2) {
            data[i].more = data2.item;
            return data[i];
        }));
    })(i); // avoid the closure loop problem
    return Q.all(result)
});

Или еще лучше:

theFunction()
.then(function(data) {
    return Q.all(data.map(function(item)
        return secondFunc(item)
        .then(function(data2) {
            item.more = data2.item;
            return item;
        });
    });
});
person Benjamin Gruenbaum    schedule 21.08.2014
comment
thnaks Второй код работает отлично, и есть небольшая ошибка на result.map, должно быть data.map - person Nazril Faudi; 22.08.2014

arrow_upward
0
arrow_downward

Я знаю, что это старый пост, но у меня та же проблема, и я не нашел решения. Может быть, кто-то здесь найдет хорошее решение очень быстро.

function CompareTeamspeakClients(forumUsers) {
  var promises = [];
  var tsClient = new TeamSpeakClient("127.0.0.1", 10011);

  tsClient.send("login", {
    client_login_name: "serveradmin",
    client_login_password: "M+h8YzUA"
  }, function(err, response){
    if (err) deferred.reject(err);
  });

  tsClient.send("use", {
    port: 9987
  }, function(err, response){
    if (err) deferred.reject(err);
  });

  forumUsers.forEach(function(user, index){
    var deferred = Q.defer();

    tsClient.send("clientdbfind", ["uid"], {
      pattern: user.tsid
    }, function(err, response){
      if (err) deferred.reject(err);
      if (response) {
        tsClient.send("clientdbinfo", {
          cldbid: response.cldbid
        }, function(err, response){
          if (err) deferred.reject(err);

          forumUsers[index]['tsdbid'] = response.client_database_id;
          forumUsers[index]['tsnickname'] = response.client_nickname;
          forumUsers[index]['tslastconnected'] = response.client_lastconnected;

          deferred.resolve(forumUsers);
        });
      }
    });

    promises.push(deferred.promise);
  });

  console.log(promises);

  return Q.all(promises);
}
person muuvmuuv    schedule 05.11.2016