Помогите с переопределением массива

[NuclleaR]

Доброго времени суток, уважаемыке знатоки! Дали задание переопределить метод splice

функция не дописана, ещё не реализовывал добавление элементов.
проблема собственно такого характера: как переопределить исходный массив? this = Arr не прмогает

Array.prototype.splice = function (start, deleteCount) {
    var removedArr = [];
    var Arr = [];

    for ( var i = start; i <= deleteCount; i++ ) {
        removedArr.push( this[i] );
    }

    for ( var j = 0; j < start; j++) {
        Arr.push(this[j]);
    }

    for ( var n = deleteCount+1; n < this.length; n++ ) {
        Arr.push(this[n])
    }

    return removedArr;
};

[Octane]

this.length = 0;
this.push.apply(this, Arr);

[NuclleaR]

this.length = 0;
this.push.apply(this, Arr);

Спасибо помогло!!!!!!

[NuclleaR]

В результате получил такой код

Array.prototype.splice = function (start, deleteCount) {
    var removedArr = [];
    var Arr = [];

    if (start < 0){
        start = this.length + start;
    }

    for ( var i = start; i <= start + deleteCount-1; i++ ) {
        removedArr.push( this[i] );
    }

    for ( var j = 0; j < start; j++) {
        Arr.push(this[j]);
    }

    for (var a = 2; a <= arguments.length; a++){

        if (typeof arguments[a] == 'undefined'){
            break;
        } else {
            Arr.push(arguments[a]);
        }
    }

    for ( var n = start+deleteCount; n < this.length; n++ ) {
        Arr.push(this[n])
    }

    this.length = 0;
    this.push.apply(this, Arr);

    return removedArr;
};

[nerv_]

Дали задание переопределить метод splice

Array.prototype.splice = (function(method) {
    return function() {
        return method.apply(this, arguments);
    };
}(Array.prototype.splice));

var arr = [1,2,3];
arr.splice(1, 1);

alert(arr);

[Sweet]

NuclleaR, в твоём варианте есть несоответствие нативному методу:

var array = [,,];
array.splice(0, 1);
0 in array; // должно быть false, у тебя - true

А используя метод .slice, можно избавиться от циклов и значительно сократить код. Я бы сделал так:

Array.prototype.splice = function (start, deleteCount) {
    if (start < 0) start += this.length;
    if (start > this.length) start = this.length;
  
    var removed = this.slice(start, start + deleteCount);
    var tail = this.slice.call(arguments, 2).concat(this.slice(start + deleteCount));
    
    this.length = start + tail.length;
    
    for(var i = 0; i < tail.length; i++) {
      if (i in tail) this[start + i] = tail[i];
      else delete this[start + i];
    }
    
    return removed;
};

Вроде такая реализация идентична нативной.

[nerv_]

Я бы сделал так

А я бы никак не делал Где сказано, что надо написать свою реализацию? Я вижу только переопределить. Т.е. этого достаточно

Array.prototype.splice = function() {
    alert('splice');
};

var arr = [1,2,3];
arr.splice(1, 1);

[Sweet]

Где сказано, что надо написать свою реализацию?

Я всё таки думаю, что нужна реализация, а не просто переопределись на что угодно. Иначе почему выбрали именно .splice? Видимо, потому что у этого метода достаточно сложный алгоритм.

[NuclleaR]

nerv_, Sweet,

Заданием было написать метод идентичный нативному slice. я пока учусь и таких тонкостей
var array = [,,];
array.splice(0, 1);
0 in array; // должно быть false, у тебя - true
не знаю

[NuclleaR]

Спасибо за ответы