0

在 JavaScript 中,我想覆盖对象上的函数,但仍然调用原始函数并返回它的值。所以我通常会做这样的事情:

var render = res.render;

res.render = function() {
    doSomethingNew();
    return render.apply(this, arguments);
};

但是,如果该覆盖包含需要在调用原始函数之前首先触发的异步回调,例如:

var render = res.render;

res.render = function() {
    var self = this;
    var args = arguments;

    var middlewares = getSomeMiddleware();

    return callNextMiddleware(middlewares, function() {
        return render.apply(self, args);
    });
};

function callNextMiddleware(middlewares, callback) {
    var middlewareFunc = middlewares.shift();

    if (middlewareFunc) {
        return middlewareFunc.call(function() {
            return callNextMiddleware(middlewares, callback);
        });
    }
    else {
        return callback();
    }
}

请注意,我在需要时使用了“return”语句。我有一个问题,“中间件”变量是一个函数数组,每个中间件函数如下所示:

function myMiddleware(next) {
    performLongRunningAsyncDataAccess(function() {
        next();
    });
}

因为它不使用'return next()',所以原始 res.render 方法的返回值永远不会传回。如果我让所有中间件函数使用'return next()',我可以让它工作,但它们来自外部源,所以我无法控制它们,我只能保证它们会调用'next()' .

一点背景知识,这是一个 Node.js 应用程序。中间件基本上是 Connect 中间件,我正在尝试覆盖 Express.js res.render 方法。

4

2 回答 2

2

通常,将异步函数与return语句混合是一个坏主意。您想要返回的所有内容都可以作为参数传递给回调函数。所以我仍然希望我能正确理解您的代码,但我会假设您调用该render函数,然后该函数会获取一组middleware函数。然后你想执行该数组中的所有函数,使用下一个作为前一个的回调。执行完所有函数后,render应该再次调用该函数,从而创建一种无限循环。假设所有这些,让我们看一下您的一些return陈述:

return middlewareFunc.call(function() {
    return callNextMiddleware(middlewares, callback);
});

这个块中的第一个return是无用的,因为middlewareFunc它是异步的,因此很可能会返回undefined。第二条return语句也是无用的,因为它从函数返回,您将其用作回调。但是由于您的回调只是通过 using 调用next();,因此永远不会使用返回值。

else {
    return callback();
}

在这个块callback中是render函数。那么让我们来看看这个函数:

res.render = function() {
    var self = this;
    var args = arguments;

    var middlewares = getSomeMiddleware();

    return callNextMiddleware(middlewares, function() {
        return render.apply(self, args);
    });
};

所以最后三个return语句基本上都在那里,因为你想从你的render函数中返回一些东西。但为了保持一致,您还应该考虑为该函数使用回调:

res.render = function(callback) {
    var self = this;
    var args = arguments;

    var middlewares = getSomeMiddleware();

    callNextMiddleware(middlewares, function() {
        //this will be called after all the middleware function have been executed
        callback();
        render.apply(self, args);
    });
};

所以基本上你摆脱了所有的return语句并使用纯异步设计模式。

于 2013-07-29T09:07:16.367 回答
1

callNextMiddleware应该返回其递归调用的返回值,而不是middlewareFunc's.

if (middlewareFunc) {
    var result;
    middlewareFunc.call(this, function() {
        result = callNextMiddleware(middlewares, callback);
    });
    return result;
}

小提琴:http: //jsfiddle.net/mWGXs

于 2013-07-29T08:27:24.820 回答