使用大量路由时 res.render() 上的中间件

Question

我创建了一个网页以在本地使用它。我有很多如下所示的路线——31 个.ejs 文件和3 个.html 文件。 （它们都在同一个“views”文件夹中）。

//app.js - using node and express
    app.get('/page1', function(req, res){
        res.render('page1');
    });
    app.get('/page2', function(req, res){
        res.sendFile('views/page2.html', { root: __dirname });
    });

我对这些文件中的每一个都使用app.get。我有一种感觉，它不是 DRY 代码，所以现在我正试图找出一种更优雅、更优化的方法来实现相同的结果。

• 我知道许多 res.sendFile(); 可以替换为单个 express.static() 中间件语句。我通常在一个“公共”文件夹上使用express.static()，我用它来保存我所有的css文件——比如app.use(express.static(path.join(__dirname, 'public')));。但我仍然不明白如何使用它来简化我所有的res.sendFile()。
  

• 至于许多 res.render(); 路由，我知道如果我不传递任何自定义数据，我可能会用一个处理整个模板文件目录（及其相应路由）的中间件来替换它们) 或文件列表。我只是不知道该怎么做。

非常感谢任何帮助，谢谢！

---

[更新]

• 里奇
• node_modules
• public
• css 文件、图片等
• 观看次数
• 部分
• 所有部分文件
• programmingPublic
• 来自同一主题的所有 ejs 文件
• 其他文件（html 和其他 ejs）
• appjs
• packagejson
• package-lockjson

const express = require('express');
const bodyParser = require('body-parser');
const path = require('path');

const app = express(); 

// Body Parser Middleware
app.use(bodyParser.json());
app.use(bodyParser.urlencoded({extended: false}));

// engine
app.set("view engine", "ejs");

// Set static path
app.use(express.static(path.join(__dirname, 'public')));

const fs = require('fs');

function renderStatic(dir) {
    return function(req, res, next) {
        let target = path.join(dir, req.path);
        fs.access(target, fs.constants.R_OK, function(err) {
            if (err) {
                // file not found, just move on
                next();
            } else {
                res.render(target);
            }
        });
    }
}

app.use(renderStatic(path.join(__dirname, "views/programmingPublic")));

以下是我的侧边菜单的格式：（所有这些文件都在“programmingPublic”文件夹中）

<a href="/programming" class="title">Programming</a>
<li><a href="/c">C</a></li>
<li><a href="/cpp">C++</a></li>
<li><a href="/python">Python</a></li>
<li><a href="/javascript">JavaScript</a></li>
<li><a href="/php">PHP</a></li>

Answer 1

如果您有一堆页面需要调用 res.render()，但没有将自定义选项传递给每个渲染，那么您可以将所有这些模板隔离在它们自己的目录中，然后使用如下中间件：

const path = require('path');
const fs = require('fs');

function renderStatic(dir, options) {
    const regex = /^\.|\.\.|\/\.|\\\./;
    options = options || {};

    return function(req, res, next) {
        let target = path.join(dir, req.path);
        if (options.ext && !path.extname(target)) {
           target = target + options.ext;
        }
        // don't allow leading dot or double dot anywhere in the path
        if (regex.test(target)) {
           next();
           return;
        }
        fs.access(target, fs.constants.R_OK, function(err) {
            if (err) {
                // file not found, just move on
                next();
            } else {
                res.render(target);
            }
        });
    }
}

app.use(renderStatic(path.join(__dirname, "renderPublic"), {ext: ".ejs"}));

注意，您必须将这些模板文件隔离在它们自己的目录中，以免在其中找到其他文件。

为了安全完整性，此代码还需要过滤掉路径中的 . 和 .. 项，就像 express.static() 所做的那样，以防止攻击者进入您的目录层次结构以访问除目录中的文件之外的其他文件渲染静态目录。

---

然后，对于您使用res.sendFile() 而没有其他逻辑的路由，只需将这些HTML 文件隔离在它们自己的目录中并将express.static() 指向该目录即可。然后，express.static() 中间件将在该目录中找到匹配的 HTML 文件并自动为您执行res.sendFile()，与处理您的 CSS 文件完全相同。