Вернуть ответ после функции async await в nodejs

Я разрабатываю небольшое приложение, у меня проблема. После того, как я жду запуска запроса из Mongodb, я использую метод карты для выполнения некоторой задачи, затем я возвращаю свой ответ, но ответ всегда возвращает обещание, и у меня нет необходимых данных. Вот мой код

    exports.getListChallenges = async function (req, res) {
    var status_code = Consts.STATUS_CODE.ERROR.UNKOWN_ERROR;
    try {
        var page_number = parseInt(req.params.page_number);
        var page_size = parseInt(req.params.page_size);
        var challenges = await Challenge.find()
            .skip(page_size * (page_number - 1))
            .limit(page_size);            
        status_code = Consts.STATUS_CODE.SUCCESS.DATA_FOUND;

        challenges = challenges.map(async function (o) {
            var challenger_club = await Club.findById(o.challenger_club_id);
            var challege = {
                challenge: o,
                challenger_club_name: challenger_club.club_name,
                challenger_club_avatar: challenger_club.avatar
            };
            return challege;
        });

        res.json({
            "status": true,
            "message": "Challenges found !",
            "status_code": status_code,
            "challenges": challenges
        });

    } catch (error) {
        status_code = Consts.STATUS_CODE.ERROR.UNKOWN_ERROR;
        res.json({
            "status": false,
            "status_code": status_code,
            "error": error
        })
    }
}

"вызовы" в ответ всегда пустые, как я могу это решить? благодарить !

challenges = await Promise.all( challenges.map( async ...
Paul 27.10.2018 08:17
Поведение ключевого слова "this" в стрелочной функции в сравнении с нормальной функцией
Поведение ключевого слова "this" в стрелочной функции в сравнении с нормальной функцией
В JavaScript одним из самых запутанных понятий является поведение ключевого слова "this" в стрелочной и обычной функциях.
Концепция локализации и ее применение в приложениях React ⚡️
Концепция локализации и ее применение в приложениях React ⚡️
Локализация - это процесс адаптации приложения к различным языкам и культурным требованиям. Это позволяет пользователям получить опыт, соответствующий...
Улучшение производительности загрузки с помощью Google Tag Manager и атрибута Defer
Улучшение производительности загрузки с помощью Google Tag Manager и атрибута Defer
В настоящее время производительность загрузки веб-сайта имеет решающее значение не только для удобства пользователей, но и для ранжирования в...
Безумие обратных вызовов в javascript [JS]
Безумие обратных вызовов в javascript [JS]
Здравствуйте! Юный падаван 🚀. Присоединяйся ко мне, чтобы разобраться в одной из самых запутанных концепций, когда вы начинаете изучать мир...
Система управления парковками с использованием HTML, CSS и JavaScript
Система управления парковками с использованием HTML, CSS и JavaScript
Веб-сайт по управлению парковками был создан с использованием HTML, CSS и JavaScript. Это простой сайт, ничего вычурного. Основная цель -...
JavaScript Вопросы с множественным выбором и ответы
JavaScript Вопросы с множественным выбором и ответы
Если вы ищете платформу, которая предоставляет вам бесплатный тест JavaScript MCQ (Multiple Choice Questions With Answers) для оценки ваших знаний,...
0
1
879
1
Перейти к ответу Данный вопрос помечен как решенный

Ответы 1

Ответ принят как подходящий

Вы очень близки - вы сопоставили каждый элемент в исходном challenges с Promise, который разрешается в желаемый объект challenge, теперь вам нужно вызвать Promise.all в этом массиве Promises, чтобы получить значения, которые они разрешают. Чтобы код оставался ясным, лучше не переназначать challenges, если это не необходимо - назовите новый массив обещаний как-нибудь еще, например challengePromises:

exports.getListChallenges = async function(req, res) {
  var status_code = Consts.STATUS_CODE.ERROR.UNKOWN_ERROR;
  try {
    var page_number = parseInt(req.params.page_number);
    var page_size = parseInt(req.params.page_size);
    var challenges = await Challenge.find()
      .skip(page_size * (page_number - 1))
      .limit(page_size);
    status_code = Consts.STATUS_CODE.SUCCESS.DATA_FOUND;

    var challengePromises = challenges.map(async function(o) {
      var challenger_club = await Club.findById(o.challenger_club_id);
      var challege = {
        challenge: o,
        challenger_club_name: challenger_club.club_name,
        challenger_club_avatar: challenger_club.avatar
      };
      return challege;
    });
    var challengeObjs = await Promise.all(challengePromises);

    res.json({
      "status": true,
      "message": "Challenges found !",
      "status_code": status_code,
      "challenges": challengeObjs
    });

  } catch (error) {
    status_code = Consts.STATUS_CODE.ERROR.UNKOWN_ERROR;
    res.json({
      "status": false,
      "status_code": status_code,
      "error": error
    })
  }
}

Кроме того, просто предложение, но если вы используете async и await, вы можете подумать о том, чтобы полностью использовать синтаксис ES6 +, такой как const вместо var, функции стрелок, когда это возможно, сокращенные свойства объектов, среди прочего - они могут сделать код более лаконичный и читаемый. Например, я бы реорганизовал ваш код, чтобы:

exports.getListChallenges = async function(req, res) {
  try {
    const page_number = parseInt(req.params.page_number);
    const page_size = parseInt(req.params.page_size);
    const challengeData = await Challenge.find()
      .skip(page_size * (page_number - 1))
      .limit(page_size);
    const status_code = Consts.STATUS_CODE.SUCCESS.DATA_FOUND;
    const challengePromises = challengeData.map(async (o) => {
      const {
        club_name: challenger_club_name,
        avatar: challenger_club_avatar
      } = await Club.findById(o.challenger_club_id);
      return {
        challenge: o,
        challenger_club_name,
        challenger_club_avatar
      };
    });
    const challenges = await Promise.all(challengePromises);
    res.json({
      status: true,
      message: "Challenges found !",
      status_code,
      challenges
    });
  } catch (error) {
    const status_code = Consts.STATUS_CODE.ERROR.UNKOWN_ERROR;
    res.json({ status: false, status_code, error })
  }
}

Обратите внимание, что status_code всегда будет Consts.STATUS_CODE.ERROR.UNKOWN_ERROR при возникновении ошибки, учитывая логику исходного кода - нет необходимости переназначать его.

Да, это сработало, большое спасибо, ваш ответ действительно подробный и заставляет меня лучше понимать asyn / await / prom.

every Bit 27.10.2018 08:59

Другие вопросы по теме