Daten aus einem Array entfernen, es zu einem anderen Array zu vergleichen

Question

Ich versuche, die Artikel in „item“ Array und dem copyofOpList Array zu vergleichen, die Daten Vorkommen in copyofOpList

das ist mein Versuch abzurufen:

var _deleteUsedElement1 = function(item) { 

        for (var i = 0; i < item.length-1; i++){  
          for (var j = 0; j< $scope.copyofOpList.length-1; j++){ 
         if (item[i].operationCode == $scope.copyofOpList[j].code) { 
         $scope.copyofOpList.splice(j, 1); 

         } } } }; 

$scope.compareArrays = function() { 
    ...Get data from web Service 
    _deleteUsedElement1(item); 
    } 

das copyofOpList Array hat 14 Elemente, und das Elementarray hat 2 Array aber mein Code löscht nur ein Vorkommen (das erste), also wie kann ich meinen Code korrigieren, um alle Vorkommen im copyofOpList Array im Vergleich zum Artikel-Array Dank für Hilfe

Answer 1

Ich würde versuchen, in einer Schleife zu vermeiden Looping - das ist weder eine sehr elegante noch eine sehr effiziente Art und Weise das gewünschte Ergebnis zu erhalten.

Hier ist etwas eleganter und höchstwahrscheinlich effizienter:

var item = [1,2], copyofOpList = [1,2,3,4,5,6,7]; 

var _deleteUsedElement1 = function(item, copyofOpList) { 
    return copyofOpList.filter(function(listItem) { 
    return item.indexOf(listItem) === -1; 
    }); 
    }; 

    copyofOpList = _deleteUsedElement1(item, copyofOpList); 
    console.log(copyofOpList); 
    //prints [3,4,5,6,7] 
} 

Und da ich gerade bemerkt, dass Sie Objekteigenschaften sind zu vergleichen, hier ist eine Version, die auf passende Objekteigenschaften-Filter:

var item = [{opCode:1},{opCode:2}], 
    copyofOpList = [{opCode:1},{opCode:2},{opCode:3},{opCode:4},{opCode:5},{opCode:6},{opCode:7}]; 

var _deleteUsedElement1 = function(item, copyofOpList) { 
     var iOpCodes = item.map(function (i) {return i.opCode;}); 
     return copyofOpList.filter(function(listItem) { 
     return iOpCodes.indexOf(listItem.opCode) === -1; 
     }); 
    }; 

copyofOpList = _deleteUsedElement1(item, copyofOpList); 
console.log(copyofOpList); 
//prints [{opCode:3},{opCode:4},{opCode:5},{opCode:6},{opCode:7}] 

Ein weiterer Vorteil dieser Vorgehensweise besteht darin, dass Sie Ihre Arrays nicht modifizieren, während Sie noch daran arbeiten, ein positiver Effekt, den sowohl JonSG als auch Furhan S. in ihren Antworten erwähnten.

Answer 2

Spleißen ändert Ihr Array. Verwenden Sie einen temporären Puffer-Array für neue Werte wie folgt aus:

var _deleteUsedElement1 = function(item) { 
    var _temp = []; 
    for (var i = 0; i < $scope.copyofOpList.length-1; i++){  
     for (var j = 0; j< item.length-1; j++){ 
      if ($scope.copyofOpList[i].code != item[j].operationCode) { 
       _temp.push($scope.copyofOpList[j]); 
      } 
     } 
    } 
    $scope.copyofOpList = _temp; 
};