内部から要素を削除する配列プロトタイプ

Question

グローバルサービス内でjsネイティブ配列を拡張して、グローバルオブジェクトのプロトタイプを作成せずに余分な機能を追加しようとしています。

app.factory('Collection', function($http, $q) { 
    var Collection = function(arr) { 
     this.key = 'id'; 
     this._last = 0; 
     this._first = 77777777; //just big number. 
     this.append(arr); 
    } 
    Collection.prototype = new Array; 
    Collection.prototype.orderBy = function(n, reverse) { 
     if (reverse) { 
      this.sort(function(a, b) { 
       return b[n] - a[n]; 
      }) 
     } else { 
      this.sort(function(a, b) { 
       return a[n] - b[n]; 
      }) 
     } 
    } 
    Collection.prototype.spliceBy = function(key, val) { 
     for (var i = 0; i < this.length; i++) { 
      if (this[i][key] !== val) { 
       this.splice(i, 1); ///THIS NEVER HAPPENS !! 
       console.log('removed ' + i + ' from ', this); 
      } 
     } 
    } 
    Collection.prototype.subset = function(key, val) { 
     return this.filter(function(v) { 
      return (v[key] === val); 
     }); 
    } 
    Collection.prototype.add = function(obj) { 
     for (var i = 0; i < this.length; i++) { 
      if (this[i][this.key] > this._last) { 
       this._last = this[i][this.key]; 
      } 
      if (this[i][this.key] < this._first) { 
       this._first = this[i][this.key]; 
      } 
      if (this[i][this.key] === data[this.key]) { 
       if (override) { 
        this[i] = data; 
        console.log('updated uniquePush'); 
       } 
       return i; 
       break; 
      } 
     } 
     var id = this.push(data) - 1; 
     data._index = id; 
     return id; 
    } 
    return collection 
}); 

これは、spliceBy機能以外は正常に動作しています。 値= xを持たない要素をフィルタリングする必要があります。私のコントローラ

.controller(function($scope,Collection){ 

$scope.posts = new Collection; 

$scope.posts.add({id:1,type:'post'}); 
$scope.posts.add({id:2,type:'comment'}); 

//Collection is now [{id:1,type:post},{id:2,type:comment}]; 

//i want to remove all comments from array 
$scope.posts.spliceBy('type','comment'); 

}); 

で例えば spliceByを呼び出すときしかし、何も起こりません：*（

Answer 1

スプライスからインデックスを更新しているので、あなたは、行に削除するには2つの要素を持っている場合spliceBy機能が動作しません。私Array.lengthとする代わりに、これを試してみてください：。。。

Collection.prototype.spliceBy = function(key, val) { 
    var i = this.length; 
    while (i--) { 
     if (this[i][key] !== val) { 
      this.splice(i, 1); ///THIS NEVER HAPPENS !! 
      console.log('removed ' + i + ' from ', this); 
     } 
    } 
}