Как вынести маршруты в отдельные файлы?

Question

[charbet]

Есть файл, в котором прописанно создание сервера на express и куча маршрутов по типу

app.get('/user', function(req, res) {
    if(req.isAuthenticated() && req.user.role == 'user')
        res.render('user');
});

app.post('/changeRole', function(req, res){
    console.log(req.body);
    pg.connect(connectionString, function(err, client, done){
        console.log(req);
        if(err){
            return console.error('error feetching client from pool', err);
        }
        client.query('UPDATE items SET role=($1), change=($2) WHERE name=($3)', [req.body.role, 
        	req.body.role == 'user' ? 'true' : 'false', req.body.name]);
        done();
    });
});

Каким образом можно вынести эти маршруты в отдельные файлы? Или даже не сами маршруты, а их функции-обработчики.

Answer 1

[Алексей Шашенков]

Скажем назовем файл handlers.js с таким содержимым

handlers.js

module.exports.getUser = (req, res) => {
    if (req.isAuthenticated() && req.user.role == 'user')
        res.render('user');
}


module.exports.postChangeRole = (req, res) => {
    console.log(req.body);
    pg.connect(connectionString, function (err, client, done) {
        console.log(req);
        if (err) {
            return console.error('error feetching client from pool', err);
        }
        client.query('UPDATE items SET role=($1), change=($2) WHERE name=($3)', [req.body.role,
        req.body.role == 'user' ? 'true' : 'false', req.body.name]);
        done();
    });
}

А файле роутера его подключим

router.js

var handlers=require('./handlers');

app.get('/user', handlers.getUser);
app.post('/changeRole',handlers.postChangeRole);

Answer 2

[Abcdefgk]

Берёте вот эту штуку

function(req, res) {
    if(req.isAuthenticated() && req.user.role == 'user')
        res.render('user');

и делаете так:

module.expotrts = function(req, res) {
    if(req.isAuthenticated() && req.user.role == 'user')
        res.render('user');
};

, - назвав этоfoo.js

А потом так:
app.get('/user', require('./foo'));
Всего делов

Comments

[Алексей Шашенков]

это не хорошая практика, постоянно вызывать require

[Abcdefgk]

Алексей Шашенков: Платформа Node.js создана так, чтобы постоянно вызывать require, который вызывает require, который снова вызывает require, и ещё раз require... ...
Это не "хорошая" или "нехорошая" практика, это просто практика Node.js

[Алексей Шашенков]

Abcdefgk: Это цепочка вызовов. При запуске, но не при каждом вызове маршрута. Лучше вызвать модуль 1 раз, нежели дергать при каждом запросе.

[Abcdefgk]

Алексей Шашенков: "Дёргать модуль при каждом запросе" - это значит обращаться к нему в оперативной памяти: все модули приложения загружаются при его запуске в оперативную память. Никакого "дёрганья" не происходит - всё уже в оперативке.
А в режиме production - и шаблоны уже все в оперативке. Рекварийте на здоровье сколько вам захочется - лишь бы вам удобно было в своём собственном коде разбираться.
Эту систему Райан Дал заимствовал из великого Lisp