Почему не работает forEach?

Question

[Иван Гришов]

Мне нужно по клику на кнопку все элементы массива сделать в нижнем регистре и вывести массив в консоль.

Не могу понять, почему элементы в массиве не изменяются. Спасибо за ответ.

Answer 1

[Александр]

Если хотите их сохранить в новом массиве, тогда так:

- a9.forEach(function(elem) {
-   elem.toLowerCase();
- });
- 
- return a9;
+ return a9.map(function (elem) {
+   return elem.toLowerCase();
+ });

Если хотите сохранить в существующий массив, тогда так (правда смысла в этом не много):

- a9.forEach(function(elem) {
-   elem.toLowerCase();
- });
+ a9.forEach(function(elem, i, arr) {
+   arr[i] = elem.toLowerCase();
+ });

или

- a9.forEach(function(elem) {
-   elem.toLowerCase();
- });
+ for (let index = 0; index < a9.length; index++) {
+   a9[index] = a9[index].toLowerCase();
+ }

Comments

[Иван Гришов]

Да, мне нужно изменить в существующем массиве. Ваше решение сработало. Можете объяснить зачем здесь arr[i], ведь forEach изменяет элементы в текущем массиве. elem - это и есть этот элемент, почему нельзя к нему применить изменения?

[V Sh.]

Иван Гришов, если кратко, то потому что elem внутри forEach - это примитив, а примитивы копируются полностью. Вот если бы у вас был массив объектов, то в переменную elem ложился бы объект (по ссылке) и с ним нормально такое работало бы, да и то только вот так (т.к. toLowerCase - не мутирующая функция):

let b9 = [{id: 1, b: 'Quantum Break'}, {id: 2, b: 'Mass Effect: Andromeda'}];

function t9() {
	b9.forEach(function(elem) {
		elem.b = elem.b.toLowerCase();
	});
 }
  
  t9();
 console.log(b9);

Можно еще проще:

let a9 = ['Quantum Break', 'Gears of War 4', 'Mass Effect: Andromeda', 'Far Cry Primal'];

let b9 = a9.map((item) => item.toLowerCase());

 console.log(b9); //["quantum break", "gears of war 4", "mass effect: andromeda", "far cry primal"]

[Иван Гришов]

Александр, V Sh. спасибо за ответы.

[Александр]

Иван Гришов, forEach как раз ничего не изменяет. Он просто перебирает массив.