Когда используется next() в Express?

Question

[Григорий Васильков]

Согласно документации - для того чтобы перейти к обработке следующего ммм контроллера чтоли...
То есть там так:

req.get("/", function (req, res, next) {
  // ... code
 next();
}, function (req, res) {
 // ... code
});

Смысл как работает понятно, другой вопрос - при каких обстоятельствах реально нужно перейти к следующему действию? Что мешает вызвать это действие внутри первого, для чего сделана поддержка нескольких подряд? Или тут смысл в задержке вызова?

Кто-нибудь использует?

Answer 1

[Алексей Уколов]

Здесь речь не о "контроллерах", а о middleware. И в этом контексте цепочка вызова становится понятной. Например:

1. нужно разобрать тело запроса,
   
2. потом обработать куки,
   
3. потом проверить аутентификацию пользователя,
   
4. потом получить данные пользователя из БД (асинхронная операция),
   
5. потом выполнить собственно контроллер роута (асинхронная операция),
   
6. потом залогировать время выполнения скрипта,
   
7. потом подчистить какие-то артефакты.
   

Главная причина такой архитектуры - попытка избавления от callback-hell. В Express всё на колбэках, которые плохо приспособлены к асинхронности, на самом деле. Вот и придумали такие "промисы для бедных" (тут я сильно упростил, конечно).

Comments

[Григорий Васильков]

Что мешает сделать эти действия внутри одного по очереди? Я просто не совсем понимаю почему именно так

[Алексей Уколов]

То, что это плохой код - у функции должна быть только одна обязанность.

[Алексей Уколов]

Где-то порядок может быть иным, опять же, или не нужно выполнять какие-то шаги. А ещё стандартные мидлы можно выносить в отдельные пакеты и использовать из проекта в проект. Короче говоря, такой подход даёт гибкость.

[Григорий Васильков]

Полностью согласен, но простите мою назойливость, что мешает ну примерно так сделать:
function (req, res) {
action1();
action2();
action3();
action4();
}

или тут именно особенность JS что функции имеют свойство запускаться одновременно, а заканчиваться - кто как пойдет, там же запросы и тд

[Григорий Васильков]

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

[Григорий Васильков]

это могло быть реализовано потому что "что-то по-другому не сделать" или потому что "кому-то так может быть удобно" - пытаюсь понять

[Алексей Уколов]

Мешает то, что этот код плохо пахнет и нарушает в конце концов соглашения фреймворка.
В js функции не имеют свойства запускаться одновременно, запускаются и заканчиваются они по очереди, но, действительно, внутри могут запускаться какие-то асинхронные операции и ваш список из функций превратится в лесенку колбэков.

[Григорий Васильков]

А именно такой подход получается позволяет функции запускать одну за другой... мне кажется все равно тут так не выйдет, потому что просто экземпляр функции передается, и он также будет вызван по очереди основного кода, если не поставить что-то вроде when.then

[Алексей Уколов]

Главная причина этого как раз попытка избавления от callback-hell. В Express всё на колбэках, которые плохо приспособлены к асинхронности, на самом деле. Вот и придумали такие "промисы для бедных" (тут я сильно упростил, конечно).

[Алексей Уколов]

он также будет вызван по очереди основного кода, если не поставить что-то вроде when.then

Тут уже от разработчика зависит, когда он будет вызван - в том и гибкость. Если операция асинхронная, то да, вызывать next() нужно по её завершении, а не просто в конце родительской функции.

[Григорий Васильков]

Ааа все воткнул. По умолчанию вызывается по келлбеку, но если хочешь, можно вызвать по ходу, отменив ее назначение в качестве коллбэка