Как правильно вызывать функции внутри цикла forEach?

Question

[Мигель Де Сервантес]

Привет, извиняюсь за небольшой говнокодец!
Не могу понять, почему код ниже возвращает "a,b2,a,b2,aa,b", вместо "a,b1,a,b2,aa,b"? Если смотреть под отладчиком, сначала в new_array[0] записывается ["a","b1"], а потом, почему-то перезаписывается на значение, которое идёт следующим в цикле forEach.

function splitMultipleRooms(array, j, char) {  
  var new_array=[];
  for (var i=0; i<array.length; i++) {
    var row = array[i];
    row[j].split(char).forEach(function(el) {
      var new_row = row;
      new_row[j] = el;
      new_array.push(new_row);
    });
  };
  return new_array;
};

var array = [['a','b1;b2'], ['aa', 'b']];
var test1 = splitMultipleRooms(array, 1, ';');
alert(test1)

А цель проста: во втором столбце попадаются значения, разделённые знаком ";". Надо делать split по этому символу, а все остальные элементы в строке дублировать.

Comments

[Сергей Горностаев]

Нельзя изменять массив, по которому идёт итерация.

[Sayonji]

Именно. Исправьте на var new_row = row.slice();

[Мигель Де Сервантес]

Sayonji: то есть "var new_array = array;" - это не копирование массива, а ссылка на него? Это особенность работы цикла forEach?
Почему такое происходит только в случае двумерного массива? Если я делаю:
var new_array=[];
var row = ['a','b1;b2'];
row[j].split(char).forEach(function(el) {
var new_row = row;
new_row[j] = el;
new_array.push(new_row);
});

То, new_array будет равен [['a','b1']['a','b2']]

[Sayonji]

Мигель Де Сервантес: Нет, так работает js, forEach тут ни при чем. Насчет второго вашего примера, вы не правы: https://jsfiddle.net/pvg3d8hp/ Всё будет точно так же.

Answer 1

[Мигель Де Сервантес]

Спасибо за ответы, код Sayonji помог, хотя я пока так и не понял, почему.
var new_row = row.slice();