Node.js代码提示

Question

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

有一个模块来获取主页。

app.js：

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

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

frontend.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

如果远程配置和全局统计数据与路由无关，则应该将它们放在中间程序中而不是控制器中：

middlewares.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) 
    };
};

app.js：

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

或

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

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

注意:代码未经测试。