1

Почему при написании этого кода в консоль у меня падает браузер?

function duplicate(a)
{
if(Array.isArray(a)){
    for(var i=0;i<a.length;i++)
        {
            a.push(a[i]);
        }
    }
return a;
};
duplicate([1,2,3,4,5])
2
  • 2
    цикл никогда не закончится т.к. каждую итерацию a.length становится больше на 1 6 июн 2019 в 15:21
  • 1
    Потому что, увеличивая размер массива в каждой итерации цикла, цикл никогда не сможет дойти до конца массива.
    – andreymal
    6 июн 2019 в 15:21

1 ответ 1

0

Потому что длина массива увеличивается с каждым выполнением push, а значит цикл бесконечный. Ну и рано или поздно доступная память закончится, и вкладка упадёт. Чтобы работало правильно, надо просто сохранить начальную длину в переменную:

function duplicate(a) {
  if (Array.isArray(a)) {
    for (var i = 0, n = a.length; i < n; i++) {
      a.push(a[i]);
    }
  }

  return a;
}

console.log(duplicate([1, 2, 3]));

1
  • @andreymal, бывает)))
    – Qwertiy
    6 июн 2019 в 15:23

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.