Express: come passare l'istanza dell'app alle route da un file diverso?

Voglio suddividere i miei percorsi in diversi file, dove un file contiene tutti i percorsi e l'altro le azioni corrispondenti. Al momento ho una soluzione per raggiungere questo obiettivo, tuttavia devo rendere globale l'istanza dell'app per potervi accedere nelle azioni. La mia configurazione attuale è simile a questa:

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.

};

controller / index.js:

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

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

controller / posts.js:

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

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

Tuttavia, questa configurazione ha un grosso problema: ho un database e un'istanza di app che devo passare alle azioni (controller / *. Js). L'unica opzione a cui potrei pensare è rendere globali entrambe le variabili, il che non è davvero una soluzione. Voglio separare le rotte dalle azioni perché ne ho molte e le voglio in una posizione centrale.

Qual è il modo migliore per passare le variabili alle azioni ma separare le azioni dai percorsi?

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

La variabile dell'applicazione creata chiamando express()viene impostata sugli oggetti richiesta e risposta.

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

Node.js supporta le dipendenze circolari.
Facendo uso di dipendenze circolari invece di require ('./ routes') (app) pulisce molto codice e rende ogni modulo meno interdipendente dal suo file di caricamento:

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');

Esempio dal loro nuovo generatore:
Scrittura del percorso:
https://github.com/expressjs/generator/blob/master/templates/js/routes/index.js
Aggiunta / spaziatura dei nomi all'app: https://github.com /expressjs/generator/blob/master/templates/js/app.js#L24

Ci sono ancora casi d'uso per accedere all'app da altre risorse, quindi le dipendenze circolari sono ancora una soluzione valida.

Come ho detto nei commenti, puoi usare una funzione come module.exports. Una funzione è anche un oggetto, quindi non è necessario modificare la sintassi.

app.js

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

controllers.js

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

controllori / index.js

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

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

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

module.exports = controllers;

O fallo semplicemente:

var app = req.app

all'interno del middleware che stai utilizzando per queste rotte. Come quello:

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

Diciamo che hai una cartella chiamata "contollers".

Nel tuo app.js puoi inserire questo codice:

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);

nei tuoi controller più vecchi avrai il file "ping.js" con questo codice:

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

E questo è tutto ...

1. Per rendere il tuo oggetto db accessibile a tutti i controller senza passarlo ovunque: crea un middleware a livello di applicazione che colleghi l'oggetto db a ogni oggetto req, quindi puoi accedervi all'interno di ogni controller.

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

2. per evitare di passare l'istanza dell'app ovunque, invece, passando i percorsi dove si trova l'app

// 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

L'ultima app.js:

// 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');
});

Un'altra versione: puoi personalizzarla in base alle tue esigenze, come l'aggiunta di una richiesta di post

// 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}));

Per il database separare il servizio di accesso ai dati che farà funzionare tutto il DB con una semplice API ed eviterà lo stato condiviso.

Separare route.setup sembra sovraccarico. Preferirei invece inserire un routing basato sulla configurazione. E configura le rotte in .json o con annotazioni.