Почему не работает функция js?

Question

[ramil456]

У меня есть такая задача:

Напишите функцию sumInput(), которая:

Просит пользователя ввести значения, используя prompt и сохраняет их в массив.
Заканчивает запрашивать значения, когда пользователь введёт не числовое значение, пустую строку или нажмёт «Отмена».
Подсчитывает и возвращает сумму элементов массива.
P.S. Ноль 0 – считается числом, не останавливайте ввод значений при вводе «0».

Вот код который я написал:

function sumInput() {
	let arr = []
	let sum = 0
	function checkNum() {
		let numVal = +prompt('Введите значение', '')
		if (!isNaN(numVal)) {
			checkNum()
			arr.push(numVal)
			console.log(arr)
		} else {
			for (let i = 0; i < arr.length; i++) {
				sum += arr[i]
			}
			console.log(sum)
		}
	}
	checkNum()
}

В ответ выводит undefined, вместо суммы элементов в массиве

Comments

[WbICHA]

В ответ выводит undefined

В каком месте? У тебя в обоих логах андефайт в принципе быть не может.

Answer 1

[justaoneuser]

Можно сделать так (оставил весь ваш код, только заменил порядок одной строчки, чтобы заработало):

function sumInput() {
	let arr = []
	let sum = 0
	function checkNum() {
		let numVal = +prompt('Введите значение', '')
		if (!isNaN(numVal)) {
//			checkNum()           // было так
			arr.push(numVal)  // добавляем элемент в массив
			console.log(arr)
			checkNum()         // только после этого вызываем функцию checkNum
		} else {
			for (let i = 0; i < arr.length; i++) {
				sum += arr[i]
			}
			console.log(sum)
		}
	}
	checkNum()
}

Код не работал потому что "вся полезная работа" откладывалась из за преждевременного вызова checkNum. А поскольку вызов рекурсивный, то сложно заметить такую ошибку и вообще понять что происходит в коде. Но в принципе ваш подход рабочий.

То что в консоли бывает undefined - это потому что основная функция sumInput ничего не возвращает явно (нет return). Но такие функции в JS всё же возвращают значение, просто оно не определено - undefined.

P.S.: Чтобы выполнить остальные условия из задания (пуcтая строка, кнопка Отмена), можно сделать так:

let val = prompt('Введите значение', '');  // сохраняем оригинальное значение
let numVal = Number(val);   // приводим тип к числу, можно и через +val, как у вас
/*
расширяем условие проверки полученного значения
1. Если приведённая к типу число переменная не является числом
2. И если переменная не null (так будет в случае нажатия Отмена)
3. И если тип оригинального значения - строка и после удаления пробелов сохраняет длину (не пустая строка)
*/
if (!isNaN(numVal) && (val !== null) && (typeof val == 'string' && val.trim().length)) {

Всё это можно красивее написать, но потребуется сильнее изменить структуру кода.

Answer 2

[Rsa97]

1. Зачем нужен рекурсивный вызов checkNum?
2. При нажатии кнопки "Отмена" возвращается null, а +null === 0.
3. Массив тут не нужен. Совсем не нужен.

Comments

[ramil456]

это задача из книги learnJS на тему массивы

[justaoneuser]

1. Зачем нужен рекурсивный вызов checkNum?

Чтобы повторно вызывать prompt и всю остальную логику для условия задачи.
Есть идеи как это сделать без рекурсии?
Можно while(true) с break по условию или что то ещё.
Но чем это лучше рекурсии? Стек не переполнится? Нет лишних вызовов функции? - Всё так, но для этой задачи не принципиально.

По номеру 2 вы правы.
А вот массив нужен по условию задачи.

[Rsa97]

justaoneuser, цикл гораздо проще и нагляднее рекурсии. В данном случае вы уходите в рекурсию не изменив массива и, когда получаете завершающий элемент, массив у вас пустой. В результате получаем:

0 // вывод суммы пустого массива
Array [ 3 ]    // вывод массива в обратном порядке
Array [ 3, 2 ] // при развёртывании рекурсии
Array(3) [ 3, 2, 1 ]
undefined // результат работы функции sumInput(),
          //поскольку она ничего не возвращает

[justaoneuser]

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

[WbICHA]

Rsa97,

цикл гораздо проще и нагляднее рекурсии.

Я бы поспорил с этим утверждением.) Вот вообще не всегда именно проще и лучше. Но в данном случае да, поддерживаю.