Node.js代码提示

时间:2014-08-04 12:30:45

标签: node.js express module redis export

您好我想知道我是否对这种代码采用了这种代码,并且有更好的方法。

有一个模块可以获取主页。

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

这工作正常,但它是否真的正确?我可以做比这更好的事吗?

谢谢任何提示都很有用。

1 个答案:

答案 0 :(得分: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);

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

注意:代码未经测试。