Promise.all ответ

Question

Promise.all ответ

1

Могу ли я добавить Promise.resolve(value) и Promise.reject(error) в ответ Promise.all().

Например,

function transferFRQ(fromUserId, fromCompanyDetails, toUserDetails, toCompanyDetails,) {
  return Promise.all([
    transferRFQCompany(fromCompanyDetails, toCompanyDetails),
    replaceRFQCreatedBy(fromUserId, toUserDetails)
  ])
    .then(result => Promise.resolve(result))
    .catch(error => Promise.reject(error));
}

function transferRFQCompany (fromCompanyDetails, toCompanyDetails) {
  return new Promise((resolve, reject) => {
    Request.updateMany({
      "company.id": fromCompanyDetails._id
    }, {
      $set: {
        company: {
          id: toCompanyDetails._id,
          name: toCompanyDetails.name,
          logo: toCompanyDetails.logo
        }
      }
    }).then(result => resolve(result))
      .catch(error => reject(error));
  });
}

function replaceRFQCreatedBy (fromUserId, toUserDetails) {
  return new Promise((resolve, reject) => {
    Request.updateMany({
      "createdBy.id": fromUserId
    }, {
      $set: {
        createdBy: {
          id: toUserDetails._id,
          firstName: toUserDetails.firstMame,
          lastName: toUserDetails.lastName
        }
      }
    }).then(result => resolve(result))
      .catch(error => reject(error));
  });
}

Я не знаю, правильно ли это или нет, но мне нужно обработать ответ transferRFQ должным образом, потому что мне нужно будет добавить transferRFQ в другой Promise.all() для правильной обработки ошибки.

Я делаю не так? если да, то как это сделать правильно

Любые дополнительные советы приветствуются!

Ali Razmjo 13 авг. 2018, в 11:09

Источник

0

да, вы делаете это неправильно. Request.updateMany уже возвращает обещание, а затем вы заключаете его в другое обещание. вы должны просто return Request.updateMany... вместо
Theo 13 авг. 2018, в 09:08
0

Таким образом, простой return result и return error решат проблему. Вы уверены, что?
Ali Razmjo 13 авг. 2018, в 09:10
0

на transferFRQ , нет необходимости , чтобы обернуть его в другое обещание, поскольку Promise.all уже возвращает обещание, уменьшить сложность есть
Theo 13 авг. 2018, в 09:14
0

нет, я имею в виду, вы обрабатываете результат, где вы использовали transferFRQ
Theo 13 авг. 2018, в 09:15
0

Это возвращает обещание или текст?
Ali Razmjo 13 авг. 2018, в 09:16
0

здесь, посмотрите на этот пример gist.github.com/shierro/d14b1de029237ae2a67265439e930a70
Theo 13 авг. 2018, в 09:18
0

имеет ли смысл @ali Razmjo? дайте мне знать, как вы идете
Theo 13 авг. 2018, в 09:23
0

пожалуйста, проверьте ваш GitHub
Ali Razmjo 13 авг. 2018, в 09:36
0

Давайте продолжим эту дискуссию в чате .
Theo 13 авг. 2018, в 09:39

Показать ещё 7 комментариев

Теги:

javascript

promise

1 ответ

Ещё вопросы

да, вы делаете это неправильно. Request.updateMany уже возвращает обещание, а затем вы заключаете его в другое обещание. вы должны просто return Request.updateMany... вместо
Таким образом, простой return result и return error решат проблему. Вы уверены, что?
на transferFRQ , нет необходимости , чтобы обернуть его в другое обещание, поскольку Promise.all уже возвращает обещание, уменьшить сложность есть
нет, я имею в виду, вы обрабатываете результат, где вы использовали transferFRQ
Это возвращает обещание или текст?
здесь, посмотрите на этот пример gist.github.com/shierro/d14b1de029237ae2a67265439e930a70
имеет ли смысл @ali Razmjo? дайте мне знать, как вы идете
Давайте продолжим эту дискуссию в чате .

Theo · Accepted Answer · 2018-08-13T07-49-00.000Z

Я советую вам не использовать ненужные обертки Promise и избегать подъема javascript.

Вы должны попытаться сделать что-то подобное

// Note that this is declared before used, to avoid javascript hoisting
function transferRFQCompany (fromCompanyDetails, toCompanyDetails) {
    return Request.updateMany({ // updateMany already returns a promise right? no need to wrap it in another promise
      "company.id": fromCompanyDetails._id
    }, {
      $set: {
        company: {
          id: toCompanyDetails._id,
          name: toCompanyDetails.name,
          logo: toCompanyDetails.logo
        }
      }
    })
  });
}

// Note that this is declared before used, to avoid javascript hoisting
function replaceRFQCreatedBy (fromUserId, toUserDetails) {
  return Request.updateMany({ // updateMany already returns a promise right? no need to wrap it in another promise
    "createdBy.id": fromUserId
  }, {
    $set: {
      createdBy: {
        id: toUserDetails._id,
        firstName: toUserDetails.firstMame,
        lastName: toUserDetails.lastName
      }
    }
  })
}

function transferFRQ(fromUserId, fromCompanyDetails, toUserDetails, toCompanyDetails,) {
  return Promise.all([
    transferRFQCompany(fromCompanyDetails, toCompanyDetails),
    replaceRFQCreatedBy(fromUserId, toUserDetails)
  ])
}
// Sample usage async/await style
(async () => {
  try {
      // put your params, of course
      const result = await transferFRQ(...params); 
      // 'result' is result of .then()
  } catch (e) {
    // 'e' is result of .catch()
  }

  // or use it in promise-style
  transferFRQ(...params)
    .then(console.log)
    .catch(console.error)
})()