Как я могу вернуть статус внутри обещания?

Я начал изучать обещание с loopback и jsforce и не мог справиться с этой проблемой; Я не смог вернуть статус var внутри обещания функции cb(). В основном я хочу подключить Salesforce и получить данные через JSforce и записать его в db через loopback. Затем вы хотите вернуть созданные/обновленные/записи ошибок клиенту после вызванного удаленного вызова.

Я разрабатываю Loopback с помощью Node.JS и Express.js Я использую библиотеку JSforce для подключения salesforce

Как я могу это исправить?

Вот мой код:

module.exports = function(Contact) {
  var jsforce = require('jsforce');
  var async = require("async");
  var lr = require('lr.js');

  Contact.ImportContacts = function(cb) {
    // Salesforce Projects List
    var sf_projects = [];
    //Salesforce Conn String
    var conn = lr.SalesforceConn();
    conn.apex.get("/Contact/", function(err, res) {
      var status = {
        "Created": [],
        "Updated": [],
        "Error": ""
      };
      if (err) console.log(err);

      sf_projects = res;
      // Clear result
      status.Created.length = 0;
      status.Updated.length = 0;
      status.Error = "";

      if (sf_projects != undefined) {
        async.eachSeries(sf_projects, function(contact, callback) {
          Contact.findOrCreate({
              where: {
                co_SalesforceID: contact.Id
              }
            }, {
              co_Name: contact.FirstName,
              co_Surname: contact.LastName,
              co_Salutation: contact.Salutation,
              co_Title: contact.Title,
              co_Department: contact.Department,
              co_Email: contact.Email,
              co_PhonePersonal: contact.HomePhone,
              co_PhoneWork: contact.Phone,
              co_PhoneCell: contact.MobilePhone,
              co_Description: contact.Description,
              co_SalesforceID: contact.Id
            },
            function(err, cntct, created) {
              if (err) console.log(err);
              if (created) {
                status.Created.push(cntct.id);
                console.log("Contact created. SalesForeID: " +
                  cntct.co_SalesforceID +
                  " ContactName: " +
                  lr.isDefined(cntct.co_Salutation) + " " +
                  lr.isDefined(cntct.co_Name) + " " +
                  lr.isDefined(cntct.co_Surname));
              } else {
                Contact.replaceById(cntct.id, {
                    co_Name: contact.FirstName,
                    co_Surname: contact.LastName,
                    co_Salutation: contact.Salutation,
                    co_Title: contact.Title,
                    co_Department: contact.Department,
                    co_Email: contact.Email,
                    co_PhonePersonal: contact.HomePhone,
                    co_PhoneWork: contact.Phone,
                    co_PhoneCell: contact.MobilePhone,
                    co_Description: contact.Description,
                    co_SalesforceID: contact.Id
                  },
                  false,
                  function(err, obj) {
                    if (err) console.log(err);
                    status.Updated.push(obj.id);
                    console.log("Contact updated. SalesForeID: " +
                      obj.co_SalesforceID + " ContactName: " +
                      lr.isDefined(obj.co_Salutation) + " " +
                      lr.isDefined(obj.co_Name) + " " +
                      lr.isDefined(obj.co_Surname));
                  });
              }
            });
          callback(err);
        }, function(err) {
          if (err) console.error(err);
        });
      } else {
        console.log("Salesforce Connection Error!");
        status.Error = "Salesforce Connection Error";
      }
      return Promise.resolve(status);
    }).then(function(end) {
      cb(null, end);

    }).catch(function(err) {
      if (err) console.log(err);
    });
  };
  Contact.remoteMethod(
    'ImportContacts', {
      returns: {
        arg: 'result',
        type: 'string'
      },
      http: {
        path: '/importContacts',
        verb: 'get'
      }
    }
  );
};

Ответ 1

Не совсем понятно, о чем спрашивают, и вы не включаете свою функцию solve(), которая может быть здесь важна, поэтому я могу дать вам несколько общих советов.

У вас есть что-то вроде этого:

}).then(function(end) {
  cb(null, end);
}).catch(function(err) {
  if (err) console.log(err);
});

Первая часть (then) предполагает, что обратный вызов cb() принимает ошибку как первый аргумент и значение как второй аргумент, следуя обычному соглашению Node обратных вызовов.

Но тогда во второй части (catch) вы не вызываете обратный вызов с ошибкой. Кроме того, if (err) является избыточным, поскольку в обработчике catch всегда будет ошибка, если только функция solve() не возвращает отклоненное обещание с false или null, указанное в качестве причины отклонения, - и даже тогда, независимо от причины отклонения, обратный вызов всегда должен вызываться в случае ошибки:

}).then(function(end) {
  cb(null, end);
}).catch(function(err) {
  console.log(err);
  cb(err);
});

Таким образом, вы не получите ситуации, когда callback никогда не вызывается и ждет навсегда. Когда вы смешиваете promises с традиционными обратными вызовами, вам нужно помнить несколько вещей:

Любая функция, которая получает обратный вызов в качестве аргумента, должна удостовериться, что этот обратный вызов вызван и что он вызывается ровно один раз. Это ваша ответственность как автор функции для обеспечения этого. В случае ошибки вы должны запустить:

callback(error);

и в случае успеха вы должны позвонить:

callback(null, data);

Таким образом, callback может знать, когда операция завершена и завершилась ли она с успехом или неудачей, проверив свой первый аргумент:

function (err, data) {
  if (err) {
    console.log('Error:', err);
  } else {
    console.log('Success:', data);
  }
}

Весь вызов функции, выполняющей обратный вызов, обычно:

functionTakingCallback('some', 'arguments', function (err, data) {
  if (err) {
    console.log('Error:', err);
  } else {
    console.log('Success:', data);
  }
});

С другой стороны, если функция возвращает обещание, вы используете ее следующим образом:

functionReturningPromise('some', 'arguments')
.then(function (data) {
  console.log('Success:', data);
})
.catch(function (err) {
  console.log('Error:', err);
});

Нет необходимости тестировать err в этом случае.

Обратные вызовы всегда следует вызывать ровно один раз. promises всегда должен быть либо разрешен, либо отклонен в конце концов. Использование различно, и ответственность как вызывающего, так и вызываемого абонента различна. Когда вы смешиваете эти два стиля - функции, которые используют традиционные Node -образные обратные вызовы и функции, которые возвращают promises - тогда вы должны быть осторожны с этими различиями.

Иногда вы можете преобразовывать функции, возвращающие обратные вызовы, в функции, возвращающие promises, используя библиотеки, такие как Bluebird и promisify() и promisifyAll(), чтобы иметь согласованный API для всех ваших асинхронных функций во всей кодовой базе. См:

Вы можете увидеть некоторые другие ответы, где я объясню разницу между обратными вызовами и promises и как их использовать вместе более подробно, что может быть полезно для вас в этом случае: