用 try catch 包装现有 javascript 对象的所有方法

Question

是否有一种下降方式可以使用 try catch 将所有方法包装在现有的 javascript 对象（例如第三方库）中，以便我可以处理异常错误？当然，我将在新接口下公开这些方法。

这是我现在想到的：（将其视为伪代码）

var MyInterface = {};
for (var property in thirdPartyLib) {
  if ((typeof thirdPartyLib[property]) === 'function'){
    MyInterfac[property] = function(){
      try {
        thirdPartyLib[property](arguments)
      }
      catch(err){
         //do my custom processing
      }
    }
  }   
}

//developer use MyInterface object

只是想知道，我的上述想法有什么更好的方法或任何缺陷吗？谢谢。

Answer 1

几乎！唯一的问题是您没有正确传递参数。 arguments 是一个伪数组对象，其中包含 所有 传递给函数调用的参数。你的代码最终会发生什么，真正的函数将被一个对象调用，而不是你在包装函数中收到的单个参数。

您还可以使用闭包进行就地包装，因此您无需担心使用不同的对象。

function safeWrap(service, fn) {
    var ogFn = service[fn];
    service[fn] = function () {
       try { return ogFn.apply(service, arguments); }
       catch (err) {
           // doh.
       }
    };  
}

function wrapObj(service) {    
    for (var fn in thirdPartyLib) {
        var type = typeof(thirdPartyLib[fn]);
        if (type === 'function') {
            safeWrap(thirdPartyLib, fn);
        } else if (type === 'object') {
            wrapObj(thirdPartyLib[fn]);
        }
    }
}

wrapObj(thirdPartyLib);

根据@RobG 的建议进行了编辑以使包装递归

如果性能有问题，您可以手动传递参数以避免使用Function.apply 带来的性能损失：

function safeWrap(service, fn) {
    var ogFn = service[fn];
    service[fn] = function (a, b, c, d, e, f, g, h, i, j, k, l) {
       if (arguments.length > 12) {
           console.error('Time to add %s more args!', arguments.length - 12);
       }
       try { return ogFn.call(service, a, b, c, d, e, f, g, h, i, j, k, l); }
       catch (err) {
           // doh.
       }
    };  
}