JavaScript - 使用包含异步回调的函数覆盖函数并仍返回原始值

Question

在 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方法。

Answer 1

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

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

此块中的第一个return 是无用的，因为middlewareFunc 是异步的，因此很可能会返回undefined。第二个return 语句也没有用，因为它从函数返回，您将其用作回调。但是由于您的回调只是使用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 语句并使用纯异步设计模式。

Answer 2

callNextMiddleware 应该返回其递归调用的返回值，而不是 middlewareFunc 的。

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

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