将./routes中的所有路由添加到中间件堆栈

Question

现在，我对我的路由目录中的每条路由使用app.use()和require()将它们添加到中间件堆栈中（我正在使用Express）。

app.use('/', require('./routes/index'));
app.use('/users', require('./routes/users'));
app.use('/post', require('./routes/post'));
app.use('/submitPost', require('./routes/submitPost'));
...

与其对每个文件手动执行操作，不如使用for循环遍历./routes中的路由文件，并将每个文件添加到中间件堆栈中。 这是我所拥有的，但不起作用：

require('fs').readdir('/routes', function (err, files) {
    if (!err) {
        for (var i = 0; i < files.length; i++) {
            var file = files[i].substr(files[i].lastIndexOf('.'));
            app.use('/' + file, require('./routes/' + file));
        }
    }
});

有人可以帮助我更正这段代码。 另外，将./routes中的所有路由自动添加到中间件堆栈是否有任何缺点？

提前致谢。

Answer 1

这里的主要问题可能是在添加中间件时。 您正在使用readdir异步方法。 您可能在代码后声明了一个全部捕获的404处理程序，并且由于异步添加了所需的路由，因此可能会在全部捕获之后添加它们。 当请求通过中间件传播时，它将在到达路由之前终止它。

另一个问题是您使用的路径： /routes将尝试查找文件系统的路径。 ./routes或__dirname + '/routes'是您想要的。

以下代码示例对我有用：

var files = require('fs').readdirSync('./routes')  
for (var i = 0; i < files.length; i++) {
    var file = files[i].substr(0, files[i].lastIndexOf('.'));
    app.use('/' + file, require('./routes/' + file));
}

Answer 2

顺便说一下，您可以为此使用文件清单 。 它实际上是专门为该用例创建的，尽管它仍然希望您自己调用app.use ，因为顺序对于快速路线很重要。

因此，您可以执行以下操作：

var fm = require('file-manifest');
var routes = fm.generate('./routes');
app.use('/', routes.home);
app.use('/foo', routes.foo);
// etc.

如果您确实希望所有事情都神奇地发生，则可以使用自定义的reduce函数来实现，但这要更加明确，并确保以正确的顺序设置路由（因此，您不会以/foo掉落而告终/foo/bar之前，并阻止其到达）。

我相信我有资格写这个库。

Answer 3

有几种方法可以做到这一点。 这是使用基本fs和path模块的简洁方法。

var fs = require("fs"),
    path = require("path");

var root = "./routes/"

fs.readdir(root, function (err, files) {
    if (err) {
        throw err;
    }

    files.forEach(function (file) {
        var filename = file.slice(0, -3);
        var routePath = '/' + ((filename === 'index') ? '' : filename);  //filter index to use just '/'
        app.use(routepath, require(root + filename));
    });
});