Node.js 代码提示

Question

你好，我想知道我对这种代码是否正确，是否有更好的方法。

有一个获取主页的模块。

应用程序.js:

[...]
// Get homepage
app.get('/', frontend.index);
[...]

frontend.index 函数必须从 redis 获取几个数据对象(util.getRemoteConfig 和 util.getGlobalStats)并使用它们来呈现 Jade 模板，所以我这样做了。

前端.js:

[...]
// [GET] Homepage
exports.index = function(req, res){   
    var remoteConfig = {};
    var globalStats = {};

    util.getRemoteConfig(function(remoteConfig) { 

        var version = 'n.a.';
        if (remoteConfig.version)
            version = remoteConfig.version;

        util.getGlobalStats(function(globalStats) {

            res.render('index.jade', 
                { title: config.items.default_title, version: version, global_stats: JSON.stringify(globalStats) }
            );
        });
    });     
};
[...]

util模块从redis中获取数据并通过回调传递。

util.js:

[...]
exports.getRemoteConfig = function(cb) {
    client.hget('remote_config', function(err, obj) {
        if (err) return cb(err);
        // do something with obj and return it
        cb(obj);
    });
};

exports.getGlobalStats = function(cb) {
    client.hgetall("global_stats", function (err, obj) {
        if (err) return cb(err);
        // do something with obj and return it
        cb(obj);
    });
};
[...]

这工作正常，但它真的正确吗？我可以做比这更好的事情吗？

谢谢，任何提示都会有用。

Answer 1

如果远程配置和全局统计信息是独立于路由的，您应该将它们放入中间件而不是 Controller :

中间件.js:

exports.remoteConfig = function(req,res,next){
    util.getRemoteConfig(function(err,config){
        if(err){
            return next(err);
        }
        req._remote_config = config;
        return next(null);
    });
}

controllers.js:

exports.index = function(req, res){   
    var remoteConfig = req._remote_config;
    var globalStats = req._global_stats;

    res.render('index.jade', { 
        title: config.items.default_title, 
        version: version, 
        global_stats: JSON.stringify(globalStats) 
    };
};

应用程序.js:

app.get('/', middleware.remoteConfig, middleware.globalStats,controllers.index);

或

app.use(middleware.remoteConfig);
app.use(middleware.globalStats);

如果中间件应该用于所有路由。

注意:代码未经测试。