JavaScript клонирование массива. Почему браузер виснет?

Question

[Ivan Sharapenkov]

function cloneArray(mas) {
	var masClone = new Array();
	for(i = 0; i < mas.length; i++) {
		masClone.push(mas[i]);
		for(j = 0; j < mas[i].length; j++) {
			masClone[i].push(mas[i][j]);
		}
	}
	return masClone;
}

var s1 = 0;
var s2 = 1;
var d1 = 1;
var d2 = 0;

var mass1 = [ 
s = [s1, s2],
d = [d1, d2]
];

var mass2 = cloneArray(mass1);

Вот такой код. Никак не могу понять почему браузер виснет, понятно что из-за цикла, но все же вроде бы правильно.

Answer 1

[Илья Шатохин]

Какая-то непонятная мне магия тут...

var mass1 = [ 
s = [s1, s2],
d = [d1, d2]
];

Вот правильный вариант глубокого копирования (пример и сравнение с другими решениями jsfiddle.net/0yseqsuy):

var isArray = Array.isArray || function (arr) {
  return Object.prototype.toString.call(arr) === "[object Array]";
}

var isObject = function (obj) {
  return Object.prototype.toString.call(obj) === "[object Object]";
}

function cloneDeep(mas) {
  var masClone = isArray(mas) ? new Array(mas.length) : {};   
  
  Object.keys(mas).forEach(function (key) {
    if (isArray(mas[key]) || isObject(mas[key]))
      masClone[key] = cloneDeep(mas[key]);
    else
      masClone[key] = mas[key];
  });

  return masClone;
}

Но для старых браузеров нужны полифилы Object.keys и forEach.

Comments

[Ivan Sharapenkov]

Двумерный массив, ну или матрица. В данном случае я ее использую для игрового поля.

[Илья Шатохин]

sM0kfyz: это я понял, я не понял зачем там такое странное присваивание внутри массива

[Евгений Петров]

Вот и я не понял — кто это?) Кручу, верчу, обмануть хочу:)

[Ivan Sharapenkov]

Да фиг его знает))) Мне так показалось удобнее будет

[Илья Шатохин]

sM0kfyz: я написал в ответе нормальную функцию копирования

[Ivan Sharapenkov]

Илья Шатохин: Спасибо

[Евгений Петров]

Илья, только не пойму, а чем это отличается от простого slice()?)
Куча операций, а получили ровно то же самое.

[Roman Zhak]

Илья Шатохин клонировать массив можно проще

var a = [1,2,3], b; 
b = a.slice();

+ думаю, будет существенно быстрее

[Ivan Sharapenkov]

Roman Zhak: Евгений Петров: Но в этом случае копируются не значения, а ссылки, то есть это зависимая копия от исходного массива

[Евгений Петров]

sM0kfyz: Вы почитайте, что я вам написал. Вы в любом случае копируете значения ссылок.
Если вы ещё не поняли — нет никакой разницы.

[Roman Zhak]

sM0kfyz jsfiddle.net/zhak55/gu2Lqkw5

[Евгений Петров]

sM0kfyz: Если вам и до сих пор непонятно jsfiddle.net/qj8qxjqp

[Ivan Sharapenkov]

Евгений Петров: Если я использую slice изменение значения одного массива влечет изменение в другом. А в вашем коде нет. Разве не так?

[Илья Шатохин]

Евгений Петров: Roman Zhak: Ваши решения лишь копируются ссылки вложенных массивов, а не их значение.

[Илья Шатохин]

Евгений Петров: Roman Zhak: jsfiddle.net/6urb8t1m

[Илья Шатохин]

sM0kfyz: Евгений Петров: Roman Zhak: чтобы развеять все сомнения, я расширил функцию функцию - теперь она клонирует в том числе и вложенные объекты.

[Keyten]

Держите специально под матрицы:

function cloneMatrix(matrix){
 return matrix.map(function(row){
  return row.slice();
 });
}

[Илья Шатохин]

Keyten: вложенные в матрицу объекты не скопируются, а перенесутся ссылки на них.

[Keyten]

Илья Шатохин: действительно, я рассчитывал на матрицу с числами)