Question

寻找关于使以下Javascript更高效的另一种眼光。

以下JSON是从Resteasy服务生成的：

var testing = {
   "com:klistret:cmdb:ci:pojo:successful":true,
   "com:klistret:cmdb:ci:pojo:count":1,
   "com:klistret:cmdb:ci:pojo:elements":{
      "com:klistret:cmdb:ci:pojo:id":123,
      "com:klistret:cmdb:ci:pojo:name":"Mars",
      "com:klistret:cmdb:ci:pojo:type":{
        "com:klistret:cmdb:ci:pojo:id":1,
        "com:klistret:cmdb:ci:pojo:name":"Environment"
      },
      "com:klistret:cmdb:ci:pojo:configuration":{
        "@www:w3:org:2001:XMLSchemainstance:type":"Environment",
        "@Watermark":"past",
        "com:klistret:cmdb:ci:commons:Name":"Mars"
      }
    }
 };

扩展Extjs JSONReader以处理createAccessor方法中高于2的键深度。想知道是否有办法使代码更有效率？下面的函数将被调用为function(testing, "com:klistret:cmdb:ci:pojo:configuration.@Watermark")，其中com:klistret:cmdb:ci:pojo:elements属性是根。

createAccessor : function(){
    var re = /[\[\.]/;

    return function(expr) {
        if(Ext.isEmpty(expr)){
            return Ext.emptyFn;
        }

        if(Ext.isFunction(expr)){
            return expr;
        }

        # THIS FUNCTION I WANT TO BE EFFICIENT
        return function(obj){
         while (String(expr).search(re) !== -1) {
 var i = String(expr).search(re);
 var key = expr.substring(0, i);

 if (obj.hasOwnProperty(key)) {
  obj = obj[key];
 }

 expr = expr.substring(i+1, expr.length);
}

            return obj[expr];
        };
    };
}()

Answer 1

基本优化是避免使用search扫描字符串两次，这非常慢。

您可以做的最好的事情就是只需调用expr.split('.')来替换所有字符串扫描和子字符串提取，这将调用aaa.bbb.ccc.ddd形式的访问器并将它们转换为类似{{1}的数组}}。您似乎支持的其他两个字符（['aaa','bbb','ccc','ddd']和[）不起作用。

或者，您可以对整个字符串执行]的初始匹配，并保持匹配以获得类似的数组，但这可能比以前的解决方案慢。

Answer 2

这就是我使用的。我只允许点注释，记住：

Ext.override(Ext.data.JsonReader, {
  createAccessor: function() {
    return function(expr) {
      if (Ext.isEmpty(expr)) {
        return Ext.emptyFn;
      } else if (Ext.isFunction(expr)) {
        return expr;
      } else {
        return function(obj) {
          var parts = (expr || '').split('.'),
              result = obj,
              part,
              match;
          while (parts.length > 0 && result) {
            part = parts.shift();
            match = part.match(/^(.+?)(\[(\d+)\])?$/);
            result = result[match[1]];
            if (result && match[3]) {
              result = result[match[3]];
            }
          }
          return result;
        }
      }
    };
  }()
});

更高效的Javascript

2 个答案: