在express JS中使用全局转换对象

时间:2016-02-15 20:21:34

标签: javascript json node.js express internationalization

我为Express JS构建了一个非常简单的翻译模块,它是应用程序范围中的一个全局对象,它在应用程序运行时实例化:

translator.configure({
    translations: 'translations.json'
});

我在Express JS中添加了一些简单的中间件,它们为每个请求更改了翻译模块中的语言环境:

app.use(function(req, res, next) {
    var locale = // Get locale from request host header

    // Setup the translator
    translator.setLocale(locale);

    // Attach translator to request parameters
    res.locals.__ = translator.translations;

    // Pass control to the next middleware function
    next();
});

然后我通过视图中的变量__访问我的翻译(这里我使用的是ejs):

...
Here is my translated text: <%= __['test'] %>
...

我的翻译模块如下所示:

var translations,
    locale;

// public exports
var translator = exports;

translator.configure = function(opt) {
    translations = require('./' + opt.translations);
};

translator.setLocale = function(locale) {
    translator.translations = translations[locale];
}

translations.json文件只是一个简单的JSON结构:

{
    "us":{
        "test": "Hello!"
    },
    "es":{
        "test": "Hola!"
    }
}

我的问题是,这个整体结构是个坏主意吗?我对快递JS没有广泛的了解。全球对象让我有点紧张,因为翻译是基于其当前状态,从请求变为请求,这里有任何问题吗?表达JS是否在处理下一个请求之前完全完成了一个请求,或者是否存在一定程度的并发性会导致我的翻译陷入困境?

1 个答案:

答案 0 :(得分:1)

全局对象对于保存请求期间使用的状态是个坏主意。在下一个请求开始运行之前,请求不一定完成。如果请求处理程序在任何时候进行异步调用(比如读取文件),那么另一个请求可以在那时开始运行。

通常,您应该在请求对象本身上存储与特定请求相关的状态。这样,它不是全局的,只是专门为该请求存储,你可以同时拥有尽可能多的请求而不会发生冲突。

因此,除非为每个请求创建一个新的翻译器对象,然后将该特定的翻译器对象存储在请求对象中,否则理想情况下,您根本不能在翻译器对象中存储任何特定于请求的状态。

我没有完全按照你的翻译代码,但这看起来很麻烦:

app.use(function(req, res, next) {
    var locale = // Get locale from request host header

    // Setup the translator
    translator.setLocale(locale);

    // Attach translator to request parameters
    res.locals.__ = translator.translations;

    // Pass control to the next middleware function
    next();
});

因为您看起来像是在配置共享的全局转换程序对象,所以希望以后再使用它并期望它不会被任何其他请求更改。这似乎是在寻找麻烦。

如果您的请求处理程序在任何时候进行任何异步调用,则可以运行另一个请求处理程序,这可能会产生冲突,因为它们都尝试使用相同的转换程序对象。