Почему не работает функция 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,

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

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

Answer 3

[historydev]

const sumInput = (input = +prompt('Введите значение', ''), sum = 0) => (!!input && typeof(input) !== 'string') ? (sum += input, sumInput(+prompt('Введите значение', ''), sum)) : sum; 

console.log(sumInput());

Comments

[justaoneuser]

Прикольно, но без массива.
Вот так ещё можно:

const sumInput = (arr = []) => ((val, numVal) => val !== null && !isNaN(numVal) && val.trim().length ? sumInput([...arr, numVal]) : arr.reduce((a, b) => a + b))(val = prompt('', ''), Number(val))

Можно убрать вложенную анонимную функцию, но вообще смысл этих заданий чтобы разобраться с конкретной темой, на сколько понимаю, в данном случае с массивами. А вот такой однострочник со стрелочными функциями как у вас (или у меня) не способствует этой цели. А только запутывает в JS-синтаксисе, который не сказать что простой слишком.

[historydev]

justaoneuser,

Моё решение логически верное, нет смысла заполнять массив и считать сумму элементов, достаточно посчитать сумму и вернуть её.

Додумывать что-то за непонятно кем, что он там подразумевал, чему хотел научить - да мне плевать, честно.
Есть задача - есть решение, если задача некорректная, корректируется задача, а не адаптируется решение к ней.

Мне несколько лет назад отвечали на мой вопрос так, чтобы я увидел (или разобрался) как нужно, а не как я хочу. - благодаря этому и не только я знаю столько, сколько я знаю сейчас.

const sumInput = (input = +prompt('', ''), sum = 0, arr = []) => (!!input && typeof(input) !== 'string') ? (arr.push(input), sumInput(+prompt('', ''), 0, arr)) : arr.reduce((a, c) => a + c);

[justaoneuser]

historydev, логически верно и на SQL посчитать, например. Но это не соответствует условиям задачи.
В том то и дело, что "как нужно" - это хранить числа в массиве. Остальное - уже фантазии на тему.

Понимаете, решить экзотическим способом можно, но вопрос у автора был не про code golf, а ну немного другой был вопрос.

А так, вот:

const sumInput = (arr = [], fn = () => prompt('', ''), val = fn()) => val && !val.match(/[^\d]/g) ? sumInput([...arr, +val], fn, fn()) : arr.reduce((a, b) => a + b);