Advertisement
Not a member of Pastebin yet?
Sign Up,
it unlocks many cool features!
- // i t
- let n = 10;
- let cards = [];
- let cardNumber = -1;
- let free = -1001; // из имени не понятно для чего эта переменная предназначена
- while (cardNumber < 0 || cardNumber >= n) {
- cardNumber = +prompt('vvedi nomer kartochki');
- if (cardNumber >= 0 && cardNumber < n ) { // это же условие (инвертированное) в вайле, если приходится копипастить код - что-то не так.
- // в цикл отвечающий за валидный ввод номера карты вы вложили всю прочую логику, что усложняет восприятие
- // кода, желательно стремиться к простоте кода, к его понятности
- // я бы убрал ифы оставив лишь вайлы на обоих вводах от пользователя,
- // и не вкладывал в них ничего кроме ввода данных.
- // как я вижу эту задачу:
- // 1 принять валидный номер карты, если не валидный повторить пункт 1 (не вкладывать ничего в ввайл)
- // 2 принять валидную сумму вложения, если сумма не валидная повторить пункт 2 (не вкладывать ничего в ввайл)
- // 3 вывести весь массив карт и сумму всех вложений.
- // 4 (желательно) повторить все с первого пункта
- while (free < -1000 || free > 1000){
- free = +prompt('skolko polozhit?');
- if (free >= -1000 && free <= 1000){ // это же условие (инвертированное) в вайле, если приходится копипастить код - что-то не так.
- cards[cardNumber] = free; // возможно стоит тоже завести промежуточную переменную, чтобы не валидные данные не попадали в массив.
- console.log('v summe na vseh kartah' + ' ' + cards[cardNumber]) // весь массив так не вывести
- }
- }
- }
- }
- // это действие подразумевалось под "промежуточной переменной" ? - да
- // Вопрос по выводу всего массива: по условию задачи берем одну карту, которая первая разрывает цикл или я ошибаюсь? - не понял вопрос.
- // пункт 3 я бы реализовал циклом по всем картам с выводом значений каждой и суммой всех в конце.
- // пункт 4 можно реализовать как цикл на заданное количество итераций, или с дополнительным вопросом
- // пользователю - "желаете повторить ввод карты и суммы?"
- // ваш вариант кода рабочий, старайтесь не повторять участки кода и стремиться к его упрощению.)
Advertisement
Add Comment
Please, Sign In to add comment
Advertisement