在Express中正确使用app.set（）？

Question

在访问app.set（）和app.get（）之前，我无法访问我的快速中间件中的变量。但是，我想知道这是否适合使用app.set（）以及是否有更好的方法来访问中间件内的变量。这是一个例子：

var express = require("express");
var http = require("http");
var https = require("https");
var s3Client = require("./lib/s3_wrapper");
var drillClient = require('./lib/drill_wrapper');
var fs = require("fs");

var app = express();
app.set("s3Client", s3Client);
app.set("drillClient", drillClient);

var adminParams = JSON.parse(fs.readFileSync("./conf/config.json"), "utf-8");
var adminBucket = adminParams.bucket;
var adminS3Client = new s3Client(adminParams);

app.use(function(req, res, next) {
    var accessKey = req.header("Authorization");
    adminS3Client.getObject({Bucket: adminBucket, Key: accessKey})
                 .then(function(results) {
                     req.config = JSON.parse(results.Body);
                     next();
                 })
                 .catch(function(err) {
                     console.log(err);
                     res.status(err.status).send(err.message);
                 });
});

app.get("/getObject/:bucket/:objectKey", function(req, res) {
    var params = {
        Bucket: req.params.bucket,
        Key: req.params.objectKey
    };
    var s3 = req.app.get("s3Client");
    var s3Client = new s3(req.config);
    s3Client.getObject(params)
            .then(function(results) {
                res.status(200).send(results.Body);
            })
            .catch(function(err) {
                res.status(500).send(err.message);
            });
});

正如你在app.get（“/ getObject”）路由中看到的那样，我正在使用req.app.get（“s3Client”），因为没有它我无法创建我的s3Client的新实例，因为s3Client始终是未定义的在此刻。感谢您的时间。

Answer 1

我会冒险说这不是一个好用的。你把整个模块放在那个变量里面？为什么？如果您需要在其他地方使用该模块，您只需再次使用它。如果您将所有内容都放在一个文件中，那么将其添加到顶部就足够了。

const s3Client = require("./lib/s3_wrapper");

app.get("/getObject/:bucket/:objectKey", function(req, res) {
  //s3Client was already defined.
  s3Client.something()
});

然后在某个其他文件中，某处。

//you import it again
const s3Client = require("./lib/s3_wrapper");

s3Client.something()