Почему массив не модифицируется по ссылке?

Question

[MishaXXL]

Есть переменная a, которая ссылается на наш массив
На сколько я помню, если с ним проводить манипуляции, он должен изменяться, но в данном случае ничего не произошло, хотя похожая схема со сменой свойств объекта работает отлично

const a = [1,2,3,4,5]

function changeArray(arr) {
	arr.concat(arr)
}

changeArray(a)

// Почему a не равно [1, 2, 3, 4, 5, 1, 2, 3, 4, 5]?
console.log(a)

Comments

[0xD34F]

Почему a не равно [1, 2, 3, 4, 5, 1, 2, 3, 4, 5]?

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

[MishaXXL]

0xD34F, да что-то взял пример не проверив )
Но всё же, как мы в данном случае можем мутировать наш изначальный массив, чтобы у нас именно в него добавилось еще 5 значений?

[0xD34F]

Повторяю - откройте документацию. Смотрите список методов массива, ищите тот, что добавляет элементы.

[MishaXXL]

0xD34F, т.е. данная задача по оптимизации работы по памяти решается только через push в цикле?

[0xD34F]

Я не знаю, что у вас там за задача, и не хочу этого знать - об этом надо было сразу говорить, при оформлении вопроса.

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

Answer 1

[Алексей Ярков]

concat возвращает новый массив, а не мутирует существующий. Читайте документацию.

Comments

[MishaXXL]

Как в данном случае мутировать изначальный массив в такой вид?
Суть задачи было в том, чтобы у нас по памяти вышло всего 10 значений (изначальный массив в котороый добавили значений), а не 15, где создали еще один новый массив

[Rerurk]

MishaXXL, подсказка, в js всё по ссылке

[Евгений Мартынов]

MishaXXL, arr = arr.concat()

[Aetae]

Евгений Мартынов, внутри функции и без возврата это, очевидно, ничего не сделает.

MishaXXL,

function changeArray(arr) {
  arr.push(...arr)
}

Однако есть (больше, но конечное) ограничение на количество аргументов, там где ожидаются гигантские массивы так делать не рекомендуется.

[MishaXXL]

Aetae, спасибо!
В гигантских массивах нужно уже через цикл с push, верно?

Если интересно, откуда взялся вопрос
Там в промежутке 15 сек почему-то доказывалось, что должен был сработать concat
https://youtu.be/mORNlADs_P4?t=2965
Вроде тим лид сам же задал этот вопрос и странно, что тогда так на него сам ответил при разьяснении
Или я может чего не понял

[Aetae]

MishaXXL, ну что я могу сказать - чувак зафейлился по-жесткому.:)
Можешь переписать код 1 в 1 с экрана и увидишь результат.:)

А на гигантских - нужно concat с заменой оригинального массива на новый, он для этого предназначен и более оптимизирован. Или же вообще микрооптимозировать, заранее вычисляя длины и используя массивы массивов под капотом абстрагирующих классов. Всё зависит от задач и нагрузки.