Express router params(), как передать функцию?

Question

[IDONTSUDO]

Я пытаюсь установить, параметры для обработки маршрута. Но express пишет.

invalid param() call for userId, got undefined

Сам express в официальной документации просто делает функцию обработчик. Это баг или фича? Или я что то написал не так? Если я в дальнешем просто обьявляю функцию то это работает. Но я думаю возможно есть более качественное решение?

Я пытался указать намеренно

const router = express.Router({mergeParams: true});

Но это не помогло

Сode.

------------------> routes user.js //роутер
    const express = require("express")
    
    const router = express.Router()
    const { UserById, allUsers } = require("../controllers/user")
    //при помощи деструктаризации вытаскиваем  две функции
    
    router.get('/users',  allUsers) //функция allUsers обслуживает маршрут  /users
    
    router.param('userId', UserById)//то что не работает UserById обслуживает параметры принемаемых запросов
    
    module.exports = router
    ------------------> controllers user.js
    const User = require('../models/user') //подключение mongoose
    
    exports.userById = (req, res , next , id) =>{
        User.findById(id).exec((err, user)=>{
            if(err || !user){
                return res.status(400).json({
                    error: "User nor found"
                })
            }
            req.profile  = user//Вот так функция которую я деструктуризирую  
            next()
        })
    }//дальше ничего интересного
    exports.hasAuthorization = (req, res, next) =>{
        const  authorized = req.profile && req.auth && req.profile._id  === req.auth._id
        if(!authorized){
            return res.status(403).json({
                error: "User is not authorized to perfom this action"
            })
        }
    }
    exports.allUsers = (req, res)  =>{
        User.find(() =>{
            if(err){
                return res.status(400).json({
                    error: err
                })
            }
            res.json({ users })
        })
    }

Comments

[Lynn «Кофеман»]

userById != UserById

Answer 1

[Дмитрий Гололобов]

router.get('/users', allUsers(req, res))

Comments

[IDONTSUDO]

router.get работает как часы, проблема в route.param()

[IDONTSUDO]

Ты натолкнул меня на мысль, что параметры можно передать в функцию напрямую. И это сработало,спасибо.

[Дмитрий Гололобов]

)))) пожалуйста!

Answer 2

[IDONTSUDO]

IT`S WORK`S!!!

router.param('userId', function(req,res,id){
UserById(req, res, id)
})

Comments

[Роман]

IDONTSUDO, А что мешало сделать так?

router.param( 'userId', UserById );

PS: сдается мне что вы не достаточно глубоко понимаете JavaScript. Вам бы повторить как передать calback функцию в качестве параметра.

[IDONTSUDO]

Роман, это
throw new Error('invalid param() call for ' + name + ', got ' + fn);
^

Error: invalid param() call for userId, got undefined

[IDONTSUDO]

Роман,

exports.userById = (req, res , next , id) =>{
    User.findById(id).exec((err, user)=>{
        if(err || !user){
            return res.status(400).json({
                error: "User nor found"
            })
        }
        req.profile  = user 
        next()
    })
}

Калбэк здесь не нужен в контролере логика ответа.

[Роман]

IDONTSUDO, этот ваш код:

router.param('userId', function(req,res,id){
UserById(req, res, id)
})

и этот мой вариант:
router.param( 'userId', UserById );

ИДЕНТИЧНЫ. Если не работает мой то не работает и ваш, несмотря на то что вы пишете про него:

IT`S WORK`S!!!

так-же вы пишите:

Калбэк здесь не нужен в контролере логика ответа.

Вынужден вас огорчить, функция передаваемая в роутер в качестве параметра и есть callback

[IDONTSUDO]

Роман,потому что, это функционал EXPRESS. Я не пишу на чистом JS. И ошибку выдавал не интерпретатор JS а EXPRESS. И ошибка была из за его внутреннего, функционала.