Conversation
src/main/java/Main.java
Outdated
| case 5: case 6: case 7: case 8: case 9: case 0: | ||
| rubName = " рублей"; | ||
| } | ||
| System.out.print(++namb + " " + name + " " + price + " " + rubName + "\n"); |
There was a problem hiding this comment.
++namb непонятно зачем так сделано. Если это нужно чтобы увеличить индекс на единицу, то лучше было написать i+1. Так бы лучше читалось
| do { | ||
| System.out.println("На скольких человек необходимо разделить счёт \n(Введите количество человек более 1):"); | ||
| while (!scanner.hasNextInt()) { | ||
| System.out.println("Это некорректное значение для подсчёта \nНа скольких человек необходимо разделить счёт \n(Введите количество человек более 1):"); | ||
| scanner.next(); | ||
| } | ||
| userInput = scanner.nextInt(); | ||
| } while (userInput <= 1); |
There was a problem hiding this comment.
нет сообщения об ошибке, если ввести число <=1
| if (userInput <=1) { | ||
| System.out.println("Ошибка"); | ||
| start(); | ||
| } |
There was a problem hiding this comment.
не очень хорошее решение - сложно читается. Тут получается рекурсия и ошибка ввода обрабатывается в двух местах. Если бы функция была сложнее можно было бы легко запутаться. Лучше было бы написать что то типа
while (true){ String userInputString = scanner.next(); if (isUserInputStringOk()) { userInput = Integer.getInteger(userInputString); break; } else { System.out.println("Ошибка"); } }
There was a problem hiding this comment.
Спасибо, что предлагаете более простые и правильные решения. Учимся на ошибках.
Проектная работа №1 на проверку. Android-разработчик расширенный
Студент курса по Android-разработке Когорта №17
Соболев Денис