Как передать обработчик Express?

Question

[Kappy]

Есть обработчик get

router.get('/', function(req, res, next) {
...Тут действия с бд 
next();
}

Далее я передаю обработчик на следующий уровень

router.get('/', function(req, res, next) {
...Тут следующие действия
next();
}

Но почему то промис не срабатывает, и сразу выполняется 2-ой блок, только потом 1-ый. Делаю так же как в документации, не помогает, все еще понемногу обучаюсь и вот тут встрял. Хочу в одном обработчике заполнить json, во втором передать .

Comments

[Роман]

Abcdefgk, да, ладно!

А вот так значит уже нельзя?

router.get('/', someObrabotchik);
router.get('/', moreSomeObrabotchik);
router.get('/', function(req, res, next) {...});

Может быть вы не понимаете суть вопроса, заданного пользователем?
Как правильно написали люди в ответах проблема в том что он вызывал next() не дождавшись ответа от БД.

[Abcdefgk]

Роман, http - "протокол без сохранения состояния" (в переводе на обычный язык - "запрос - ответ - я вас не помню"). Учите матчасть.

[Роман]

Abcdefgk, мне почему то кажется что Вы сильно запутались сами и путаете людей.

PS: если Вы все еще сомневаетесь, офдока Вам в помощь.
Если в двух словах, то я могу навешивать сколько угодно последовательных (get, post и пр.) обработчиков на один и тот же путь. При этом я могу делать это так как показали вы, так и отдельными вызовами. Если Вы все еще сомневаетесь - проверьте сами прежде чем путать людей.

[Роман]

Abcdefgk, эта Ваша фраза ясно дает понять что Вы совсем не понимаете как работает express.

http - "протокол без сохранения состояния" (в переводе на обычный язык - "запрос - ответ - я вас не помню"). Учите матчасть.

[Abcdefgk]

Роман, Вы бы лучше у меня что-нибудь спросили уважительно - я б объяснил. Но теперь уж не буду, конечно.

[Роман]

Abcdefgk, я нисколько не сомневаюсь в Вашем понимании работы http протокола, Вы верно написали что http - "протокол без сохранения состояния". Но касаемо вопроса заданного автором про роутеры expressjs это не имеет никакого отношения.

Для понимания сути происходящего предлагаю следующее описание процесса:
1. экспресс принимает get запрос от браузера для пути '/' (к примеру)
2. экспресс вызывает первый обработчик зарегистрированный как get или all для этого пути
3. если обработчик производит вызов next() то экспресс берет следующий в очереди обработчик зарегистрированный как get или all для этого пути и вызывает его (передавая ему объект req со всеми изменениями, внесенными предыдущими обработчиками). Если же обработчик вызывает один из следующих методов (res.end(), res.json(), res.jsonp(), res.redirect(), res.render(), res.send(), res.sendFile, res.sendStatus()) то обработка запроса прекращается и последующие обработчики не вызываются.

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

[Abcdefgk]

Роман, Я там напейсал решение этой "суперзадачи" - посмотри в ответах. Если аффтар вопроса не отреагирует, то удалю через два часа - так что, поторопись.

Answer 1

[shmatuan]

next(); должен в колбеке/then асинхронной функции вызываться

БД ещё не вернуло значения (только отправило), а next() уже сработал

router.get('/', function(req, res, next) {
  someaction({query}, (error, data) => { // тут от БД зависит, может быть через промисы
    if(data) next();
  })

}

Comments

[Kappy]

Боже, точно, внутри next тоже же асинхронно передается, так получается внутри промиса должна быть функция которая колбеком вызывает next()?

[Kappy]

Да, понял, сейчас попробую сделать, спасибо!

[Kappy]

Так в голове бардак уже, чем дальше пытаюсь разобраться тем становится запутаннее.

router.get('/', function(req, res, next) {
    var json={};
    json.industry = [];
    pool.query(query.getIndustryNoReq).then((res) => {
        for (var i = 0; i < res.rows.length; i++)
            json.industry.push(res.rows[i]);
        console.log("Запрос 1")
        next();
    })

pool.query тот же промис, результатом которого мы выполняем функцию, с результатом из бд, и надо по окончанию результату передать router дальше , если next убираю, страница не получает ответа, значит тут промис работает . Тут я уже разобрался, в голове каша, но как передать next после then промиса запроса?

[shmatuan]

Николай Капитанов,

как передать next после then промиса запроса?

На вид всё правильно)

pool.query(query.getIndustryNoReq).then((res) => {
       // ... код
        next();
    })

[shmatuan]

Николай Капитанов, Если не срабатывает, скорее всего нужно именно с видом middleware исправить как в комментах к вопросу или

var requestTime = function (req, res, next) {
  req.requestTime = Date.now()
  next()
}
app.use(requestTime) // добавить

[Kappy]

shmatuan, Проблема была в библиотеке , а точнее во мне. Там пишется, что не рекомендуется использовать промисы в пуле запросов. Их нужно привязать к клиенту. Переделал структуру, все работает как надо.

client.query(query.overIndustry)
        .then(res => {
            console.log(res.rows);
            console.log('2');
            return res;
        })
        .then(res=> console.log(res.rows))


    })

Большое спасибо за помощь!

Answer 2

[Александр Дроздов]

Ну ты же внутри промиса вызываешь next()?

Answer 3

[Егор Петров]

router.get('/', async function(req, res, next) {
    try {
        await someaction(query);
        next();
    }
    catch(error){
        console.error(error);
    }
  })
}