1

我已经解决了这个问题。但是,我正在寻找更快的解决方案,因为我的变量有数千个对象。

我有两个这样的数组:

var full = [{a:'aa1',b:'bb1'},{a:'aa3',b:'bb2'},{a:'aa3',b:'bb3'},{a:'aa2',b:'bb3'}],
some = [{a:'aa1',b:'bb1'},{a:'aa3',b:'bb3'}]; 

如果对象存在于某些对象上,我正在尝试标记一个称为cin的新属性。full预期结果:

 [{a:'aa1',b:'bb1',c:true},{a:'aa3',b:'bb2'},{a:'aa3',b:'bb3',c:true},{a:'aa2',b:'bb3'}]

一些重要提示:

  • 有些元素的元素总是少于完整的
  • 两个数组排序相等

我目前的做法是:

var getIndexByAB = function(arr, a,b){
     var initialIndex =  getIndexByAB.initialIndex || 0,
     len = arr.length;
     for(initialIndex; initialIndex < len ;initialIndex++ ){
         var el = arr[initialIndex];
         if( el.b === b && el.a === a ){
             getIndexByAB.initialIndex = initialIndex;
             return initialIndex;
         }
     }
     return -1;
}

var len = some.length;
for(var i = 0; i < len ; i++){
 var el=some[i],
 index = getIndexByAB(full,el.a,el.b);
 if(index > -1) full[index].c = true;
}

UPDADE:使用 Juan 评论改进了原始解决方案。

4

3 回答 3

1

由于它们是排序的,你可以只传递一个索引来开始搜索,这将避免 O(n^2)。您已经这样做了,但是通过将索引存储在全局变量中。相反,您应该将其作为参数传递给getIndexByAB.

function getIndexByAB(arr, a,b , initialIndex){
    // Was tracking last index by storing it in a global 'this.initialIndex'. 
    // 'this' points to 'window' in global functions. That's bad, it 
    // means this function can't be called on different arrays without
    // resetting the global 

    // var initialIndex =  this.initialIndex || 0,

    initialIndex = initialIndex || 0;
    var len = arr.length;
    for(initialIndex; initialIndex < len ; initialIndex++ ){
        var el = arr[initialIndex];
        if( el.b === b && el.a === a ){
            // Bad globals
            // this.initialIndex = initialIndex;
            return initialIndex;
        }
    }
    return -1;
}

var len = some.length;
var lastValidIndex = 0;
for(var i = 0; i < len ; i++){
    var el = some[i];
    // Pass the index here, so it doesn't start from scratch
    var index = getIndexByAB(full, el.a, el.b, lastValidIndex);
    if(index > -1) {
        full[index].c = true;
        lastValidIndex = index;
    }
}

顺便说一句,如果你确实想要一个函数来缓存一些值,这里是如何避免全局变量的方法。(不是说你应该在这种情况下使用它)

var getIndexByAB = (function(){
     // This will only be executed once, and is private
     // to getIndexByAB (all invocations)
     var lastGoodIndex = 0;

     return function(arr, a,b, resetIndex){
         if (resetIndex) {
            lastGoodIndex = 0;
         }

         var len = arr.length;
         for(var index = lastGoodIndex; index < len ; index++ ){
             var el = arr[index];
             if( el.b === b && el.a === a ){                 
                 lastGoodIndex = index;
                 return index;
             }
         }
         return -1;
    };
})();

或者,您可以通过缓存它来实现以下功能,getIndexByAB.initialIndex但它不是很优雅。避免这种情况的主要原因是getIndexByAB.initialIndex任何人都可以修改

于 2013-02-21T19:47:34.690 回答
0

由于数组都已排序并且some严格小于full,因此您可以通过使用不同索引同时遍历两个数组来节省一些时间。实际上,您full每次都在遍历以获取匹配元素的索引,因此您有O(N^2)运行时间,但您只需要从匹配的最后一个元素继续搜索即可。

于 2013-02-21T19:42:12.400 回答
0

不如@Juan 的回答高效(它利用了排序的性质等),但我认为我仍然会提出我的解决方案,因为它偶然迫使我想出一个克隆和比较 Javacript 对象的解决方案。

实用程序

// Create a copy of x without reference back to x
function clone(x){
  return JSON.parse(JSON.stringify(x));
}

// Pass any number of arguments of any type. Returns true if they are all identical.
function areEqual(){
  for(var i = 1, l = arguments.length, x = JSON.stringify(arguments[0]); i < arguments.length; ++i){
    if(x !== JSON.stringify(arguments[i])){
      return false;
    }
  }

  return true;
}

标记功能

// Your flagLabel being 'c'
function matchAndFlagWith(flagLabel,aFull,aSome){
  var aFlagged = clone(aFull);

  for(var i1 = 0, l1 = aSome.length, oSome; oSome = aSome[i1], i1 < l1; ++i1){
    for(var i2 = 0, l2 = aFlagged.length, oFlagged; oFlagged = aFlagged[i2], i2 < l2; ++i2){
      if(areEqual(oFlagged,oSome)){
        oFlagged[flagLabel] = true;
      }
    }
  }

  return aFlagged;
}

演示

http://jsfiddle.net/barney/p2qsG/

于 2013-02-22T11:18:43.930 回答