Помогите найти недостатки у кода

Question

Помогите найти недостатки у кода

Аня Кондратенкова Ученик (131), на голосовании 1 год назад

Напишите, какие недостатки у следующего кода, и исправьте их:

function sum(arr) {
let res = 0;

for (let elem of arr) {
res += elem;
}

return res / arr.length;
}

Answer 1

Судя по названию функции, она должна вовзращать сумму элементов массива, но в конце почему-то делит сумму на количество элементов. Я бы убрал деление.

 return res; //вместо return res / arr.length

Answer 2

Главный недостаток в том что это старая школа и явно взято из учебников 15-летней давности.
Современный Javascript сильно отличается от этого примера, сейчас для нахождения среднего арифметическое массива напишут скорее вот так

 let arr = new Array(1,2,3,4) 
const mean = x => (x.reduce((acc, i) => acc + parseInt(i), 0)) / x.length 
alert(mean(arr))

Answer 3

Dlazder Мудрец (16875) 1 год назад

Если не говорить о методах всяких новых как в ответе выше, то код норм, рабочий, логических недостатков нет.

ПавелПросветленный (25618) 1 год назад

Они не новые. Старше чем for..of цикл.

ПавелПросветленный (25618) 1 год назад

И там есть ошибка то ли в названии функции, то ли в решении. Функция называется "sum" а высчитывает она среднее значение.

Dlazder Мудрец (16875) Рукажоп, Я цикл for of не воспринимаю как новый), ну просто обертка синтаксическая, а суть та же остается что у обычных циклов, если бы там был for in или for я бы не заметил никакой разницы, ибо ничего сверхнового они не превносят. Ну название да, неверное.