Как передать управление из routes в node js?

Question

[Abc Edc]

Есть роут

module.exports = function(app) {
app.get('/users', require('../controllers/user').CRUD.findAll);
app.get('/users/:id', require('../controllers/user').CRUD.findById);
app.post('/users/login', require('../controllers/user').handlers.login);
};

В самом ап.джс
вызов такой
require('./routes')(app);
Но в контроллере

exports.handlers = {
    login: function(req, res, next){
К ПРИМЕРУ
req.json(req.params);

нихренаеньки не вернет, в чем соль?
как правильно передавать рек и рес или управление какое то

Answer 1

[kazmiruk]

req.params содержит динамические параметры, которых у Вас судя по урл /users/login для данного запроса попросту нет. Если Вы хотите выводить там то, что после ? (/users/login?q=1), то для этого нужно использовать res.query

Comments

[Abc Edc]

А для чего же тогда парамс?

[Abc Edc]

Теперь падает
Error: Argument passed in must be a single String of 12 bytes or a string of 24 hex characters

[kazmiruk]

ГЛЕБ ГЛЕБОВ: например, если опираться на Ваши роуты и вызвать примерно такое: /users/321, то req.params будет содержать {id: 321}, но есть такой нюанс: у вас /users/:id перекрывает роут /users/login и поэтому вызов всегда попадает в users/:id и там остается (скорее всего именно поэтому Вы ничего и не видите на экране, даже пустой params). Решений здесь 2: в users/:id проверить, что req.params.id != 'login' (или в более общем случае проверить, что id число) и если id не тот, что нужно, то вызвать next (функция, которая передается в качестве последнего параметра у Вас), которая продолжит поиск нужного обработчика в роутах. Второй вариант более правильный - необходимо описывать роуты от частных к общим (т.е. поменять местами 3ий и 2ой роут), но это не умиляет факта использования next, так как позволяет обрабатывать запросы в более общем виде: не подходят параметры под текущий запрос - передали дальше управление. Подходящий обработчик не найден - 404.

[kazmiruk]

ГЛЕБ ГЛЕБОВ: Судя по всему Вы используете монго в качестве базы. Он говорит, что login - плохой ид. Причина в моем предыдущем комментарии.

[Abc Edc]

kazmiruk: ООО ТОЧНО!!ГДЕ Ж ВЫ РАНЬШЕ БЫЛИ

[Abc Edc]

ГЛЕБ ГЛЕБОВ: Пожалуйста подскажите последний раз как же лучше оргназиовать логинилку тогда?

[Abc Edc]

kazmiruk: app.get('/users/', UserController.handlers.login); Это как бы некст будет да например?

[kazmiruk]

ГЛЕБ ГЛЕБОВ: А в чем именно проблема? В принципе не отличается от остальных языков: app.get('/users/login', ...) - отдаем форму, app.post('/users/login', ...) - забираем данные из формы (тут понадобится body-parser), проверяем их в базе, если все ок, то выставляем куку с сессией (для сессий нужна прослойка session). Затем при каждом запросе начинаете проверять наличие куки. Если нет куки или значение в ней не совпадает со значением из базы (или где Вы будете хранить сессии), то пользователь не авторизован. Иначе авторизован. Для задачи проверки можно использовать middleware также, в котором проставлять request.is_anonymous = true\false.

[kazmiruk]

ГЛЕБ ГЛЕБОВ: нет. UserController.handlers.login должен иметь следующие входные параметры function(request, response, next) {} request - объект запроса, response - объект ответа, next - функция для передачи управления дальше в цепочки вызовов. Данный подход используется повсеместно в express. Т. е. расценивайте примерно так: если в текущем контроллере Вы по какой-то причине не можете отдать ответ, то вызывайте next()

[Abc Edc]

kazmiruk: app.get('/users/:id', UserController.CRUD.findById);
app.post('/users/login', UserController.handlers.login);

findById : function(req,res, next) {
var id = new ObjectID(req.params.id)
if (id){
User.findById(id, function (err, users) {
итд..

module.exports.handlers = {
login: function(req, res){
var mail = req.query.mail;
var password = req.query.password;
User.findOne({'mail': mail}, function(err, user){

означает ли это что если в запросе не найдется параметр ИД то он пойдет дальше в логинилку?

[Abc Edc]

else next();

[Abc Edc]

забыл

[kazmiruk]

ГЛЕБ ГЛЕБОВ: да. Чтобы лучше понять как это работает напишите простой пример с разными роутами, которые перекрывают друг друга (типа как /users/:id, /users/login, /users/tratata и т.п.) и посмотрите что будет при вызове next.

[Abc Edc]

kazmiruk: Супер!Вот теперь нода это курче пхп :))