Асинхронный вызов на основе ответа от предыдущего вызова, избегая ада обратного вызова

Question

Асинхронный вызов на основе ответа от предыдущего вызова, избегая ада обратного вызова

1

Поэтому я обычно занимаюсь асинхронными функциями, используя цепочку .then. Но на этот раз он этого не решает. Это ситуация.

return Promise.all([
    Banner.findOne(),
    event.findOne({
        status: true
    }),
    Account.find({
        accType: 'employer'
    })
        .then(([banner, event, account]) => {
            res.render('index', {
                title: 'Web Project Studio',
                user: req.user,
                banner: banner,
                event: event,
                employer: account
            });
        }).catch(e => {
        console.error(e);
    })
]);

Так что это сработало, и все было здорово. Но теперь я решил изменить способ выбора активного event. Я вызываю коллекцию, называемую generalInfo и захватываю идентификатор к текущему событию. Вот что у меня есть до сих пор:

return Promise.all([
    Banner.findOne(),
    GeneralInfo.findOne(),
    Account.find({
        accType: 'employer'
    })
]).then(([banner, generalInfo, account]) => {
        Event.findById(generalInfo.activeEventId).then(r => {
            console.log('test');
            var event = r;
            res.render('index', {
                title: 'Web Project Studio',
                user: req.user,
                banner: banner,
                event: event,
                employer: account
            });
        }).catch(e => console.log(e));
    }
}).catch(e => {
    console.error(e);
})

Но этот код начинает выглядеть как аддон.

Я также пробовал что-то вроде этого:

var banner = await Banner.findOne().catch(e => console.log(e));
var currentEvent = await GeneralInfo.findOne().catch(e => console.log(e));
currentEvent.then(async r => {
    var event = await Event.findOneById(r.activeEventId).catch(e => console.log(e));
}).catch(e => console.log(e));

Это не завершено, но это как-то показывает мой образ мыслей. Но опять же, не повезло.

Итак, как я могу ладить с асинксом, не .then навсегда? Мне нужно передать все возвращенные объекты функции render.

Alex Ironside 12 июль 2018, в 14:33

Источник

0

Прежде чем опубликовать ответ, мне любопытно, захотите ли вы создать заметное из вашего обещания (или обещаний). Если это так, вы можете объединиться с map, concat и всеми этими замечательными наблюдаемыми операторами, которые позволяют вам делать подобные вещи процедурным (но, конечно, асинхронным) способом. Если вы не заинтересованы в rxjs, хотя я оставлю это.
Tim Consolazio 12 июль 2018, в 12:19
0

Я понятия не имею, что это такое, но я все за любые ответы
Alex Ironside 12 июль 2018, в 12:20
0

Ах: вам действительно (действительно!) Нужно искать «rxjs» и исследовать «наблюдаемые». По сути, они позволяют вам представлять данные как «поток», который вы можете объединить с другими потоками, заставить ждать, пока другие потоки не будут готовы, и все такое. Обещания хороши для того, для чего они хороши (в основном, мгновенные однократные исполнения), но начинают разваливаться стилистически, когда вам нужно управлять несколькими асинхронными потоками, особенно если ими нужно управлять последовательно, объединять, повторять и скоро. Это требует определенных усилий, но в настоящее время вам действительно нужны rxjs под поясом, если вы выполняете асинхронную работу.
Tim Consolazio 12 июль 2018, в 12:23
0

В любом случае, Observable (вещь, которая представляет асинхронный поток данных) может обернуть обещание, чтобы вы могли использовать операторы Observable и так далее. Обязательно посмотрите «несколько потоков в rxjs», и вы получите всевозможные учебные пособия и примеры кода о том, как это сделать.
Tim Consolazio 12 июль 2018, в 12:24
0

Сделаю. Спасибо!
Alex Ironside 12 июль 2018, в 12:52
0

Примечание: в приведенном выше коде «Вот что у меня есть» есть дополнительный символ } (и несколько странных отступов). Версии в моем ответе исправляют это (в дополнение к их основной цели: ответ на вопрос :-)). FWIW, ваше изменение моего ответа изменило цепочку, поэтому оно не соответствовало вашему первому блоку кода в вопросе, поэтому я отменил редактирование ответа.
T.J. Crowder 12 июль 2018, в 13:02
0

Ой. Да, я все испортил. Прости за это! Исправлено сейчас.
Alex Ironside 12 июль 2018, в 13:15

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

Теги:

javascript

node.js

mongodb

mongoose

asynchronous

2 ответа

2

Это должно сработать.

return Promise.all([
  Banner.findOne(),
  GeneralInfo.findOne(),
  Account.find({
    accType: 'employer'
  })
]).then(async ([banner, generalInfo, account]) => {
    res.render('index', {
      title: 'Web Project Studio',
      user: req.user,
      banner: banner,
      event: await Event.findById(generalInfo.activeEventId),
      employer: account
    });
}).catch(e => {
    console.error(e);
})

Moonjsit 12 июль 2018, в 10:43

Ещё вопросы

Прежде чем опубликовать ответ, мне любопытно, захотите ли вы создать заметное из вашего обещания (или обещаний). Если это так, вы можете объединиться с map, concat и всеми этими замечательными наблюдаемыми операторами, которые позволяют вам делать подобные вещи процедурным (но, конечно, асинхронным) способом. Если вы не заинтересованы в rxjs, хотя я оставлю это.
Я понятия не имею, что это такое, но я все за любые ответы
Ах: вам действительно (действительно!) Нужно искать «rxjs» и исследовать «наблюдаемые». По сути, они позволяют вам представлять данные как «поток», который вы можете объединить с другими потоками, заставить ждать, пока другие потоки не будут готовы, и все такое. Обещания хороши для того, для чего они хороши (в основном, мгновенные однократные исполнения), но начинают разваливаться стилистически, когда вам нужно управлять несколькими асинхронными потоками, особенно если ими нужно управлять последовательно, объединять, повторять и скоро. Это требует определенных усилий, но в настоящее время вам действительно нужны rxjs под поясом, если вы выполняете асинхронную работу.
В любом случае, Observable (вещь, которая представляет асинхронный поток данных) может обернуть обещание, чтобы вы могли использовать операторы Observable и так далее. Обязательно посмотрите «несколько потоков в rxjs», и вы получите всевозможные учебные пособия и примеры кода о том, как это сделать.
Примечание: в приведенном выше коде «Вот что у меня есть» есть дополнительный символ } (и несколько странных отступов). Версии в моем ответе исправляют это (в дополнение к их основной цели: ответ на вопрос :-)). FWIW, ваше изменение моего ответа изменило цепочку, поэтому оно не соответствовало вашему первому блоку кода в вопросе, поэтому я отменил редактирование ответа.
Ой. Да, я все испортил. Прости за это! Исправлено сейчас.

T.J. Crowder · Accepted Answer · 2018-07-12T10-23-00.000Z

Вы можете просто гнездиться раньше, что имеет преимущество перекрытия больше (см. *** комментарий (ы)):

return Promise.all([
    Banner.findOne(),
    GeneralInfo.findOne()
        .then(generalInfo => Event.findById(generalInfo.activeEventId)), // ***
    Account.find({
        accType: 'employer'
    })
}).then(([banner, event, account]) => {
    res.render('index', {
        title: 'Web Project Studio',
        user: req.user,
        banner: banner,
        event: event,
        employer: account
    });
}).catch(e => {
    console.error(e);
});

Если вы делаете это событие generalInfo => много, вы можете обернуть его в служебную функцию.

С другой стороны, если они не должны пересекаться, как это, вы можете свести к минимуму вложенности, добавив еще один, then:

return Promise.all([
    Banner.findOne(),
    GeneralInfo.findOne(),
    Account.find({
        accType: 'employer'
    })
]).then(([banner, generalInfo, account]) => {
    return Event.findById(generalInfo.activeEventId).then(event => { // ***
        return [banner, event, account];                             // ***
    });                                                              // ***
}).then(([banner, event, account]) => {
    res.render('index', {
        title: 'Web Project Studio',
        user: req.user,
        banner: banner,
        event: event,
        employer: account
    });
}).catch(e => {
    console.error(e);
})