如何以编程方式替换（包装）新方法中的方法？

Question

我有几个方法需要以基本相同的方式包装在新方法中。 我的第一个解决方案不起作用，我明白为什么，但我不知道是否有一个简单的解决方案可以解决这个问题，或者是否不能按照我想要的方式完成。

这是一个例子。 我有对象 ac 具有 onClick 方法。 我需要在 onClick 方法之前执行一些代码。 我尝试了以下方法：

// objects a-c
a = {onClick : function () { console.log('a'); }};
b = {onClick : function () { console.log('b'); }};
c = {onClick : function () { console.log('c'); }};

// first try
var arr = [a, b, c]
for (var i = 0; i < arr.length; i++) {
    var oldOnClick = arr[i].onClick;
    arr[i].onClick = function () {
        // new code here
        oldOnClick();
    }  
}

// outputs 'c'
// what i want is to maintain a reference to the original method
// so in this case, execute my new code and output 'a'
a.onClick();

这不起作用，因为当调用新方法时，oldOnClick将指向上次迭代中的方法，而不是分配时的to方法。

有没有一个我忽略的简单解决方案？

Answer 1

你需要的是关闭：

for(var i=0, l=arr.length; i<l; i++){
 (function(i){
   var oldOnclick = arr[i].onClick; 
   //etc.
 })(i);
}

Answer 2

您是否尝试过使用 Javascript 的 AOP 框架？

例如使用 jQuery AOP 插件：

jQuery.aop.before( {target: String, method: 'replace'}, 
  function(regex, newString) { 
    alert("About to replace string '" + this + "' with '" + newString + 
          "' using regEx '" + regex + "'");
  }
);

另请检查此处。

Answer 3

Javascript 绑定规则非常奇怪。 确实，JavaScript 很奇怪。

我不知道我会将此称为修复它的方法，但是通过引入子函数，您可以引入另一个绑定，从而解决这个特定问题。

您的（针对快速 Chrome 黑客攻击而修改的）代码变为：

a = {onClick : function () { alert('a'); }};
b = {onClick : function () { alert('b'); }};
c = {onClick : function () { alert('c'); }};

var arr = [a, b, c]
for (var i = 0; i < arr.length; i++) {
    var oldOnClick = arr[i].onClick;
    arr[i].onClick = bind(oldOnClick);
}

a.onClick();
b.onClick();
c.onClick();

function bind(oldFunc)
{
    return function () {
        //New Code
        oldFunc();
    }  
}

上面的代码抛出三个警报：a、b、c。 任何替换“//新代码”的内容都将在正确的时间运行。

Answer 4

var a = {onClick : function () { console.log('a'); }};
var b = {onClick : function () { console.log('b'); }};
var c = {onClick : function () { console.log('c'); }};

var arr = [a, b, c];
for (var i = 0; i < arr.length; i++) {
    var oldOnClick = arr[i].onClick;
    arr[i].onClick = wrapHandler(oldOnClick);  
}

function wrapHandler(handler) {
    return function() {
        console.log("New stuff");
        handler();
    }
}

a.onClick(); // outputs "New stuff" then "a"
b.onClick(); // outputs "New stuff" then "b"
b.onClick(); // outputs "New stuff" then "c"