Можно ли сократить этот код, но чтобы он не лишался своих функций?

Question

[is-jabir]

Недавно начал изучать JavaScript. Вот написал простенький код, который спрашивает имя и возраст, а потом его выводит. Интересно, можно ли его максимально сократить, просто в программирование вошел также не давно, опыта не много.

'use strict';
let userName = prompt (`Здравствуйте, как вас зовут?:`,``);
let userAge = prompt (`Сколько вам лет?:`, ``);
while (userName == null) {
	let userName2 = prompt (`Я же спрашиваю, как вас зовут?:`, ``);
	userName = userName2; /* если пользователь не ввел
	                         значения или нажал "Отмена", при вопросе "Как вас зовут?", 
	                         цикл не завершиться, пока пользователь не введет значение.*/
}
while (userAge == null) {
	let userAge2 = prompt (`Почему молчите? Возраст укажите:`, ``);
	userAge = userAge2;/* если пользователь не ввел
	                      значения или нажал "Отмена", при вопросе "Сколько вам лет?", 
	                      цикл не завершиться, пока пользователь не введет значение.*/
}
alert (`Здравствуйте ${userName}, вам ${userAge} лет.`);

Answer 1

[Александр Черемхин]

У вас приличный код. Код должен быть не короткий, а понятный.
Как вариант, можно цикл чуть по другому определить.

while (!userName) {
...
}

P.S.
userName == null почитайте про разницу разницу между == и ===. В js это важно и обычно используют userName === null
Успехов!

Comments

[is-jabir]

Спасибо, приятно. Про строгое сравнение я знаю, но почему то забыл про него, пользуюсь не очень часто, но все равно спасибо!)

[Александр Черемхин]

alert (
`Здравствуйте ${userName}, вам ${userAge} лет. 
Совет дня: "Откройте бутылку вина прямо сейчас!!!"`
);

Answer 2

[Julia Ovod]

С помощью простой рекурсии будут вызываться функции sayHello и getAge до тех пор, пока наш пользователь не даст ответ:

const promt = (message = '') => window.prompt(message);

function sayHello() {
  let name = promt('Здравствуйте, как вас зовут?');
  return name || sayHello();
}

function getAge() {
  let age = promt('Сколько вам лет?');
  return age || getAge();
}

function greeting() {
	let name = sayHello();
  let age = getAge();

  alert(`Здравствуйте ${name}, вам ${age} лет.`);
}

greeting();

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

const promt = (message = '') => window.prompt(message);

function getAnswerAnyway(defaultQuestion, repeatQuestion) {
  const iter = (atempts = 2) => {
    const question = atempts === 2 ? defaultQuestion : repeatQuestion;
    let age = promt(question);
    return age || iter(atempts - 1);
  }

  return iter(2);
}

function greeting() {
	let name = getAnswerAnyway(
    'Здравствуйте, как вас зовут?',
    'Я же спрашиваю, как вас зовут?',
  );

  let age = getAnswerAnyway(
    'Сколько вам лет?',
    'Почему молчите? Возраст укажите',
  );

  alert(`Здравствуйте ${name}, вам ${age} лет.`);
}

greeting();

Рабочий код можно посмотреть здесь.

Comments

[Interface]

Зачем нужна первая строка в вашем примере?

const promt = (message = '') => window.prompt(message);

[Шохрух Шаймардонов]

Interface, это функция, принимающая параметр "сообщение" которое будет подставляться в prompt. Своего рода обертка для удобства использования

[Julia Ovod]

Interface, На самом деле можно опустить, если для message нет какого-то определенного значения по умолчанию, ну и promt может поменяться, например, на confirm.

[Роман]

...до тех пор, пока наш пользователь не даст ответ

или пока не произойдет переполнение стека вызовов.

[Julia Ovod]

Роман, Это правда) Ну тут какой вопрос, такой ответ.

[Хохiл Блокчейнович]

индусский код как он есть

[Александр Черемхин]

Просто сравните, что было вначале и во что-то все зарефакторили))).
Задайте себе на вопрос - какой код вы бы хотели поддерживать?
Ответ очевиден - конечно код автора...

[is-jabir]

Зачем так усложнять код? Я забыл, если пользователь ничего не ввел и нажал "Продолжить" все равно вылезал следующее окно с другим вопросом, в этом коде я это поправил:

'use strict';
let userName = prompt (`Здравствуйте, как вас зовут?:`,``);
let userAge = prompt (`Сколько вам лет?:`, ``);
while (userName == null || userName == false) {
	let userName2 = prompt (`Я же спрашиваю, как вас зовут?:`, ``);
	userName = userName2;  /* если пользователь не ввел
                           значения или нажал "Отмена", при вопросе "Как вас зовут?", 
                           цикл не завершиться, пока пользователь не введет значение.*/
}
while (userAge == null || userAge == false) {
	let userAge2 = prompt (`Почему молчите? Возраст укажите:`, ``);
	userAge = userAge2;  /* если пользователь не ввел
                        значения или нажал "Отмена", при вопросе "Сколько вам лет?", 
                        цикл не завершиться, пока пользователь не введет значение.*/
}
alert (`Здравствуйте ${userName}, вам ${userAge} лет.`);

Также можете посмотреть, сколько весит мой код (test.js), а сколько ваш (toster.js):
(из своего кода я убрал комментарии).

[Julia Ovod]

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

[Julia Ovod]

is-jabir, Сравнивать байты — не самый лучший вариант определения качества кода.

У вас код был чисто в императором стиле, я лишь предложила вариант в декларативном.

[Александр Черемхин]

is-jabir, зря убрали, код от этого только проиграл...
Все что надо убрать уберет сборщик (webpack, ...)

[Александр Черемхин]

Julia Ovod,

не согласна, потому что цикл это такая же рекурсия.

я не про рекурсию а про чтение кода. На мой вкус первый вариант читабельнее

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

согласен.

[is-jabir]

Julia Ovod, Лично я если честно не вижу разницы какой код. Если он короче, читабельнее, меньше весит и соответственно быстрее работает, то этот вариант по лучше, как по мне. Также остальные программисты легко смогут им пользоваться и его редактивировать.

[is-jabir]

Александр Черемхин, убрал когда взвешивал

[Julia Ovod]

is-jabir, Почитайте, что такое императивный и декларативные стили программирования.

[is-jabir]

Julia Ovod , обязательно прочту, я просто высказал свое мнение