Javascript Array.prototype печатается почти везде

Question

Так как я должен был удалить некоторые элементы из моих массивов, я последовал за несколько кусков коды, найденных в StackOverflow и придумал это:

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); 
}; 

По некоторым причинам, однако этот фрагмент кода печатается ВСЕГДА, когда что-то имеет что-то с массивом.

Пример:

этот кусок кода:

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); 
}; 

Scadenza.prototype.init = function() { 
    this.promemoria = (this.promemoria == "")?("NO"):(this.promemoria); 
    var gruppo = this.group; // convert array to string. 
    this.group = ""; 
    for (var i in gruppo) { 
     if (i != (gruppo.length - 1)) { 
      this.group += gruppo[i] + ", "; 
     } 
     else { 
      this.group += gruppo[i]; 
     } 
    } 
    alert(this.group); 
}; 

Этот фрагмент кода должен преобразовать this.group массива (временно хранятся в переменной «Gruppo») в строку (это совершенно очевидно, я думаю).

Это, конечно, делает его работу значительно, если он не будет, что его предупреждение является:

[НЕОБХОДИМЫЕ ДАННЫЕ] функция (от, до) {вар остальное = this.slice ((to || from) + 1 || this.length); this.length = от < 0? this.length + from: from; return this.push.apply (это, отдых); }

Этот фрагмент код в настоящее время также отправляется в базу данных через запрос AJAX и результат запроса, при желаемой coloumn, это одно:

функции (от, до) {var rest = this.slice ((to || from) + 1 || this.length); this.length = от < 0? this.length + from: from; вернуть это.push.apply (это, отдых); },

Я очень удивлен, что это происходит, но у меня нет абсолютно никакой идеи о том, как это исправить.

При загрузке страницы или нажатии кнопки, которая выбрасывает это событие, при загрузке страницы не было обнаружено никаких ошибок.

Любая идея?

ps: Не уверен, что помогает, но я использую jQuery.

@comments:

Нормальный цикл на самом деле не исправить это:

Scadenza.prototype.init = function() { 
    this.promemoria = (this.promemoria == "")?("NO"):(this.promemoria); 
    var gruppo = this.group; // convert array to string. 
    this.group = ""; 
    for (var i = 0; i < gruppo.length; i++) { 
     if (i != (gruppo.length - 1)) { 
      this.group += gruppo[i] + ", "; 
     } 
     else { 
      this.group += gruppo[i]; 
     } 
    } 
    alert(this.group); 
}; 

оповещения по-прежнему то же самое.

Answer 1

Использовать надлежащие for (var i=0; i<arr.length; i++) петли для итерационных массивов. for in enumerations will enumerate prototype properties as well, не используйте их на массивах. Вы делаете это в своем методе init, например.

Btw, для выполнения этой задачи вы хотите использовать .join все равно:

Scadenza.prototype.init = function() { 
    if (this.promemoria == "") 
     this.promemoria = "NO"; 
    this.group = this.group.join(", "); // convert array to string 
    alert(this.group); 
}; 

Answer 2

При изменении Array прототипа, метод добавляется к объекту Array, как новый атрибут. Когда вы выполняете итерацию с for (var attr in object), вы повторяете объект и все его атрибуты. Поэтому ваш новый метод включен в этот цикл.

Вам необходимо использовать петлю for (var i=0; i<a.length; i++). Это будет включать только элементы в массиве.