Expresso: Como passar instância de aplicativo para rotas de um arquivo diferente?

Quero dividir minhas rotas em arquivos diferentes, onde um arquivo contém todas as rotas e o outro as ações correspondentes. Atualmente tenho uma solução para conseguir isso, porém preciso tornar a instância do aplicativo global para poder acessá-la nas ações. Minha configuração atual é assim:

app.js:

var express   = require('express');
var app       = express.createServer();
var routes    = require('./routes');

var controllers = require('./controllers');
routes.setup(app, controllers);

app.listen(3000, function() {
  console.log('Application is listening on port 3000');
});

routes.js:

exports.setup = function(app, controllers) {

  app.get('/', controllers.index);
  app.get('/posts', controllers.posts.index);
  app.get('/posts/:post', controllers.posts.show);
  // etc.

};

controladores / index.js:

exports.posts = require('./posts');

exports.index = function(req, res) {
  // code
};

controladores / posts.js:

exports.index = function(req, res) {
  // code
};

exports.show = function(req, res) {
  // code
};

No entanto, essa configuração tem um grande problema: eu tenho um banco de dados e uma instância de aplicativo que preciso passar para as ações (controladores / *. Js). A única opção que eu poderia pensar é tornar as duas variáveis ​​globais, o que não é realmente uma solução. Quero separar as rotas das ações porque tenho muitas rotas e as quero em um local central.

Qual é a melhor maneira de passar variáveis ​​para as ações, mas separar as ações das rotas?

Use req.app,req.app.get('somekey')

A variável do aplicativo criada pela chamada express()é definida nos objetos de solicitação e resposta.

Veja: https://github.com/visionmedia/express/blob/76147c78a15904d4e4e469095a29d1bec9775ab6/lib/express.js#L34-L35

Node.js oferece suporte a dependências circulares.
Usar dependências circulares em vez de require ('./ routes') (app) limpa uma grande quantidade de código e torna cada módulo menos interdependente de seu arquivo de carregamento:

var app = module.exports = express(); //now app.js can be required to bring app into any file

//some app/middleware setup, etc, including 
app.use(app.router);

require('./routes'); //module.exports must be defined before this line

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

app.get('/', function(req, res, next) {
  res.render('index');
});

//require in some other route files...each of which requires app independently
require('./user');
require('./blog');

Exemplo de seu novo gerador:
Escrevendo a rota:
https://github.com/expressjs/generator/blob/master/templates/js/routes/index.js
Adicionando / namespacing ao aplicativo: https://github.com /expressjs/generator/blob/master/templates/js/app.js#L24

Ainda existem casos de uso para acessar o aplicativo de outros recursos, portanto, dependências circulares ainda são uma solução válida.

Como disse nos comentários, você pode usar uma função como module.exports. Uma função também é um objeto, então você não precisa alterar sua sintaxe.

app.js

var controllers = require('./controllers')({app: app});

controllers.js

module.exports = function(params)
{
    return require('controllers/index')(params);
}

controladores / index.js

function controllers(params)
{
  var app = params.app;

  controllers.posts = require('./posts');

  controllers.index = function(req, res) {
    // code
  };
}

module.exports = controllers;

Ou apenas faça isso:

var app = req.app

dentro do Middleware que você está usando para essas rotas. Curtiu isso:

router.use( (req,res,next) => {
    app = req.app;
    next();
});

Digamos que você tenha uma pasta chamada "contollers".

Em seu app.js, você pode colocar este código:

console.log("Loading controllers....");
var controllers = {};

var controllers_path = process.cwd() + '/controllers'

fs.readdirSync(controllers_path).forEach(function (file) {
    if (file.indexOf('.js') != -1) {
        controllers[file.split('.')[0]] = require(controllers_path + '/' + file)
    }
});

console.log("Controllers loaded..............[ok]");

... e ...

router.get('/ping', controllers.ping.pinging);

em seus controladores anteriores você terá o arquivo "ping.js" com este código:

exports.pinging = function(req, res, next){
    console.log("ping ...");
}

E é isso ....

1. Para tornar seu objeto db acessível a todos os controladores sem passá-lo para todos os lugares: faça um middleware de nível de aplicativo que anexa o objeto db a cada objeto req, então você pode acessá-lo em cada controlador.

// app.js
let db = ...;  // your db object initialized
const contextMiddleware = (req, res, next) => {
  req.db=db;
  next();
};
app.use(contextMiddleware);

2. para evitar passar instância de aplicativo em todos os lugares, em vez disso, passar rotas para onde o aplicativo está

// routes.js  It's just a mapping.
exports.routes = [
  ['/', controllers.index],
  ['/posts', controllers.posts.index],
  ['/posts/:post', controllers.posts.show]
];

// app.js
var { routes }    = require('./routes');
routes.forEach(route => app.get(...route));
// You can customize this according to your own needs, like adding post request

O app.js final:

// app.js
var express   = require('express');
var app       = express.createServer();

let db = ...;  // your db object initialized
const contextMiddleware = (req, res, next) => {
  req.db=db;
  next();
};
app.use(contextMiddleware);

var { routes }    = require('./routes');
routes.forEach(route => app.get(...route));

app.listen(3000, function() {
  console.log('Application is listening on port 3000');
});

Outra versão: você pode personalizar de acordo com suas necessidades, como adicionar solicitação de postagem

// routes.js  It's just a mapping.
let get = ({path, callback}) => ({app})=>{
  app.get(path, callback);
}
let post = ({path, callback}) => ({app})=>{
  app.post(path, callback);
}
let someFn = ({path, callback}) => ({app})=>{
  // ...custom logic
  app.get(path, callback);
}
exports.routes = [
  get({path: '/', callback: controllers.index}),
  post({path: '/posts', callback: controllers.posts.index}),
  someFn({path: '/posts/:post', callback: controllers.posts.show}),
];

// app.js
var { routes }    = require('./routes');
routes.forEach(route => route({app}));

Para banco de dados separado, serviço de acesso a dados que fará todo o trabalho do banco de dados com API simples e evitará estado compartilhado.

Separar routes.setup parece uma sobrecarga. Eu preferiria colocar um roteamento baseado em configuração em vez disso. E configure rotas em .json ou com anotações.