已经调用了回调！在loopback中，在updateAll函数中

Question

我正在使用环回，这里使用数组中的对象列表进行更新调用。

我已经调用了回调！

场景是，我已经定义了循环内部的回调，并且在第一个循环中，它实际上被调用了。

我正在寻找方式

我应该在查询MySQL计划调用中更新所有对象列表。

    Inward.updateIsActiveDetails = function(data, callback) {
        var id = _.map(data, 'id');
        if (id.length > 0) {
          _.forEach(id, id => {
            console.log('id....:', id)
            Inward.updateAll({id}, {
              isActive: 0,
            }).then(updateresult => {
              console.log(updateresult);
   // callback(error); showing err with it... (callback already called)
            }).catch(function(error) {
              callback(error);
            });
          });
        } else {
          callback(null, {
            success: true,
            msg: 'No records to update',
          });
        }
      };

输出：

id....: 3
id....: 4
{ count: 1 }
{ count: 1 }

欣赏正确的解决方案

Answer 1

回调应该被调用一次，你在循环中调用它，所以它将在循环的每次迭代中被调用。不止一次。如果由于某种原因您无法使用async / await，以下内容将是正确的。

Inward.updateIsActiveDetails = function(data, callback) {
    var id = _.map(data, 'id');
    var len = id.length;
    var resultList = [];

    // When you call this function we add the results to our list
    // If the list of updates is equal to the number of updates we had to perform, call the callback.
    function updateResultList(updateResult) {
      resultList.push(updateResult);
      if (resultList.length === len) callback(resultList);
    }
    if (len > 0) {
      _.forEach(id, id => {
        Inward.updateAll({id}, {
          isActive: 0,
        })
        .then(updateResult);
      });
    } else {
      callback(null, {
        success: true,
        msg: 'No records to update',
      });
    }
  };

使用async / await会更短。

Inward.updateIsActiveDetails = async function(data) {
  const results = [];
  for(let i = 0; i < data.length; i++) {
    results.push(await Inward.updateById(data[i].id));
  }
  return results;
}

Answer 2

这是我的最终和工作答案。

基本上，updateAll查询运行一次，它将作为内置查询运行

  id: {
        inq: _.map(data, 'id'),
      }

因此，在运行之后它将仅更新相应的行！很有意思。

 Inward.updateIsActiveDetails = function (data, callback) {
    Inward.updateAll({
      id: {
        inq: _.map(data, 'id'),
      },
    }, {
        isActive: 0,
      }, function (error, resultDetails) {
        if (error) {
          console.log('error', error);
          callback(error);
        } else {
          console.log('resultDetails', resultDetails);
          callback(null, resultDetails);
        }
      });
  };