Удалить массив индексов из массива

Question

Резига, создатель JQuery создал a very handy Array.remove метод, который я всегда использовать его в своих проектах:

// Array Remove - By John Resig (MIT Licensed) 
Array.prototype.remove = function(from, to) { 
    var rest = this.slice((to || from) + 1 || this.length); 
    this.length = from < 0 ? this.length + from : from; 
    return this.push.apply(this, rest); 
}; 

// Remove the second item from the array 
array.remove(1); 
// Remove the second-to-last item from the array 
array.remove(-2); 
// Remove the second and third items from the array 
array.remove(1,2); 
// Remove the last and second-to-last items from the array 
array.remove(-2,-1); 

Он прекрасно работает. Но я хотел бы знать, может ли он расширяться, чтобы он мог принимать массив индексов в качестве первого аргумента?

В противном случае, я, вероятно, сделать еще один метод, который использует его:

if (!Array.prototype.removeIndexes) { 

    Array.prototype.removeIndexes = function (indexes) { 

     var arr = this; 

     if (!jQuery) 
      throw new ReferenceError('jQuery not loaded'); 

     $.each(indexes, function (k, v) { 

      var index = $.inArray(v, indexes); 

      if (index !== -1) 
       arr.remove(index); 

     }); 
    }; 
} 

Если Array.remove() не является расширяемым, чтобы соответствовать моим потребностям, что вы думаете о моем другом решении выше?

Answer 1

Эта версия должна работать. Он изменяет исходный массив. Если вы предпочитаете возвращать новый массив без изменения оригинала, используйте инициализатор с комментариями result и добавьте return result в конце функции.

Array.prototype.removeIndexes = function(indices) { 
    // make sure to remove the largest index first 
    indices = indices.sort(function(l, r) { return r - l; }); 

    // copy the original so it is not changed 
    // var result = Array.prototype.slice.call(this); 

    // modify the original array 
    var result = this; 

    $.each(indices, function(k, ix) { 
    result.splice(ix, 1); 
    }); 
} 

> [0, 1, 2, 3, 4, 5, 6, 7, 8].removeIndexes([4, 5, 1]); 
> [0, 2, 3, 6, 7, 8] 

Answer 2

Как насчет

Array.prototype.remove = function (indexes) { 
    if(indexes.prototype.constructor.name == "Array") { 
     // your code to support indexes 
    } else { 
     // the regular code to remove single or multiple indexes 
    } 

}; 

Answer 3

Я думаю, что это то, что вы ищете (Он работает с отрицательным индексом тоже):

if (!Array.prototype.removeIndexes) { 

    Array.prototype.removeIndexes = function (indexes) { 

     var arr = this; 

     if (!jQuery) throw new ReferenceError('jQuery not loaded'); 

     var offset = 0; 

     for (var i = 0; i < indexes.length - 1; i++) { 
      if (indexes[i] < 0) 
       indexes[i] = arr.length + indexes[i]; 
      if (indexes[i] < 0 || indexes[i] >= arr.length) 
       throw new Error('Index out of range'); 
     } 

     indexes = indexes.sort(); 

     for (var i = 0; i < indexes.length - 1; i++) { 
      if (indexes[i + 1] == indexes[i]) 
       throw new Error('Duplicated indexes'); 
     } 


     $.each(indexes, function (k, index) { 
      arr.splice(index - offset, 1); 
      offset++; 
     }); 
     return arr; 
    }; 
} 

var a = ['a', 'b', 'c', 'd', 'e', 'f']; 
var ind = [3, 2, 4]; 

a.removeIndexes(ind); 

console.log(a.join(', ')); 
// returns : a, b, f 

See fiddle