将对象附加到 Node.js 进程

Question

我正在为我的 Node.js Express Web 服务器使用名为

nconf

的环境变量和参数解析模块。

https://github.com/indexzero/nconf

我决定使 nconf 数据全局化的最佳方法是将其简单地附加到流程变量（如

process.env

中），这是一个好主意还是坏主意？会因为压低“流程”而减慢执行速度吗？

这是我的代码：

var nconf = require('nconf');

nconf.argv()
    .env()
    .file({ file: './config/config.json' });

nconf.defaults({
    'http': {
        'port': 3000
    }
});

process.nconf = nconf;

//now I can retrieve config settings anywhere like so process.nconf.get('key');

坦白说，我有点喜欢这个解决方案。现在我可以在任何地方检索配置数据，而无需需要模块。但这可能有缺点......而且这很可能是一个非常糟糕的主意。我不知道。

Answer 1

不会减慢执行速度，但感觉“臭”。如果您决定需要的话，它很难发现，也很难测试。

更好的解决方案是将设置附加到模块并使用 require() 在需要的地方导入它。

最好的解决方案是将您的设置对象传递给需要它的类或模块。要么直接，要么作为某种“全球背景”的一部分。

例如。

var global = {
  settings: {
    port: 8080
  }
}

//...

global.api = new Api(global);

//...

function Api(global) {
  var port = global.settings.port;
}

更新：有关原始模式为何不好的更多信息：

1) 可发现性

您将设置附加到

process.settings

并转到另一个项目。一年后，其他人接手，或者你需要更新一些东西。您还记得您将设置附加到

process.nconf

吗？还是

process.settings

？

现在想象一下您有 10 个不同的全局事物，以不同的名称附加在不同的位置。

它并不像直接附加到全局上下文那么糟糕，但清楚地看到您正在使用的东西来自哪里（构造函数或模块）当然更好。

2) 测试

您决定需要测试您的模块。因此，现在您需要调整每个测试的设置，而不是从文件或 argv 加载它们。你是怎么做到的？

对于全局

process.nconf

或

require("settings")

模式，您需要执行以下操作：

function canOpenAPIOnTheConfiguredPort(done) {
    var nconfSaveApiPort = process.nconf.api.port;
    process.nconf.api.port = '1234';
    var api = new Api();
    test.assertEqual(api.port, '1234');
    process.nconf.api.port = nconfSaveApiPort;
    done();
}

随着应用程序的增长，这很快就会变得烦人（例如，想象一下必须模拟 10 件事）。相比之下，以下是使用依赖注入（构造函数）模式的方法。

function canOpenAPIOnTheConfiguredPort(done) {
    var api = new Api({
        port: '1234'
    });
    test.assertEqual(api.port, '1234');
    done();
}

Answer 2

注意 nconf 是一个单例。

我习惯在程序一开始就进行配置，然后当我需要在另一个文件中进行设置时，我会这样做：

var nconf = require ('nconf');
nconf.get('x');