Question

我正在为我的node.js Express Web服务器使用名为nconf的环境变量和参数解析模块。

我认为使nconf数据全局化的最好方法是简单地将它附加到流程变量（如在process.env中），这是一个好主意还是坏主意？它是否会减慢执行速度＆＃34;流程＆＃34;？

这是我的代码：

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

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

Answer 1

它不会减慢执行速度，但感觉不舒服＆＃34;。它很难发现，如果你决定需要，它很难测试。

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

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

例如

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

将对象附加到Node.js进程

2 个答案: