Почему массив перезаписывается (Javascript)?

Question

[DarthJS]

Понимаю, что вопрос касается основ, но я видимо что-то упустил :)

Подскажите, почему так происходит?
Есть функция, принимает несколько аргументов, хочу чтоб при пуше нового элемента, он добавлялся, естественно, в конец массива, а получается, что массив заполняется полностью одним элементов и перезаписывается последним поступившим. Должно быть что-то не так с циклом.

this.rule = function(tr, td, size){

var rez = [];
for(var i = 0; i < size; i++){
rez.push(tr[i]);
}
console.log(tr, td, size);
console.log(rez);
}

Comments

[Станислав Фатеев]

А что выдает console.log(tr, td, size) в вашем случае?

[DarthJS]

Станислав Фатеев: циферки :) При клике на функцию она получает три значения, допустим 1,1,5 и я хочу, чтоб при каждом клике массив rez рос и в него поступали новые значения td, а получается так, что этот массив перезаписывается

[Станислав Фатеев]

А можно участок когда побольше, вместе с вызовом?

[DarthJS]

Станислав Фатеев: да, вот: jsfiddle.net/d3hszefb/2

[Владислав]

Не совсем понятно, в чем проблема? Пример рабочий. Вот массив rez который я получаю при выполнении функции rule([1,2,3], [1,2,3], 3):
[1, 2, 3]

Answer 1

[Антон Мудренок]

ваш rez находится в локальной области видимости функции, если хотите, что бы можно было добавлять, надо писать в таком стиле

var rule = {};
rule.rez = [];
rule.addElements = function (array){
  var l = array.length;
  var i = 0;
  for (i = 0; i < l; i++) { this.rez.push(array[i]) };
};

Comments

[DarthJS]

да, сглупил. Спасибо за ответ

[Владислав]

rez находится в области видимости функции, так же как и цикл. В чем здесь проблема?

[Антон Мудренок]

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

[Владислав]

Антон Мудренок: Я понял теперь о чем речь. Не совсем понятно из условия было что функция не массивы принимает в аргументы, а числа(строки), но работает многократно

Answer 2

[Станислав Фатеев]

Кодом,
rez.push(tr[i]);
вы добавляете i-e значение из tr, а оно там всегда одно, что можно увидеть с помощью console.log(tr);
Возможно там должно быть: rez.push(tr);или rez.push(td);