5

複数のルートミドルウェアを定義しましたが、それらを複数のルート/コントローラー間で共有したいと思います。

これが私の設定です:

app.jsには./routes/index.jsが必要です。

// load fs module
var fs = require('fs');

// import routing files
module.exports = function(app){
    fs.readdirSync(__dirname).forEach(function(file) {
        if (file == "index.js") return;
        var name = file.substr(0, file.indexOf('.'));
        require('./' + name)(app);
    });
};

index.jsは、すべてのルートをディレクトリに自動的にロードします。可能なルートファイルは次のようになります。

module.exports = function(app) {

    app.get('/contacts', function(req, res, next) {
       // routing stuff
    });

};

今、私はルートミドルウェアを手に入れました:

function isAuthenticated(req, res, next) {
    if (!req.session.authenticated) return next(new Error('user not authenticated'));
};

function loadUser(req, res, next) {
    var query = User.findById(req.session.user_id);
    query.populate('contacts');
    query.exec(function(err, user) {
        if (err) return next(err);
        req.user = user;
        next();
    });
}

私が使用したいもの:

var User = require('../models/user');

module.exports = function(app) {

    app.get('/contacts', isAuthenticated, loadUser, function(req, res, next) {
       res.json(req.user.contacts);
    });

};

また、すべてのルーティングファイルでそれらを要求することは避けたいと思います。

考えられる解決策は次のとおりです。

// load fs module
var fs = require('fs');

var routeMiddleware = {
    loadUser: function(req, res, next) { // logic },
    isAuthenticated: function(req, res, next) { // logic },
};


// import routing files
module.exports = function(app){
    fs.readdirSync(__dirname).forEach(function(file) {
        if (file == "index.js") return;
        var name = file.substr(0, file.indexOf('.'));
        require('./' + name)(app, routeMiddleware);
    });
};

しかし、私は最高ではないと思います...

4

1 に答える 1

12

個人的には、コントローラーではなく、アプリで共有ミドルウェアを宣言します。

ルート/home.js:

module.exports = function(req, res, next) { \\ Code }

app.js:

app.get('/', thisMiddleware, thatMiddleware, require('./routes/home'))

スタック(オブジェクトではなく配列)を作成することもできます。

theseMiddlewares = [thisMiddleware, thatMiddleware]
app.get('/', theseMiddlewares, require('./routes/home'))

また、これらのミドルウェアが一部を除くすべてのルートで使用されている場合は、次の操作を実行できます。

theseMiddlewares = function(req, res, next) {
  if (req.url.match(some_regex_for_urls_to_skip)) next()
  else {
    \\ do stuff...
    next()
  }
}

これで、そのミドルウェアを使用できますapp.use(theseMiddlewares)。または、他のミドルウェアと比較して特定の順序で発生する必要がある場合は、次を使用できます。app.all('*', theseMiddlewares)

于 2012-07-28T03:53:51.880 に答える