Repository navigation
Conversation
| private static Scanner s = new Scanner(System.in); | ||
| private static int customers; | ||
| private static String product; | ||
| private static double price; |
There was a problem hiding this comment.
если переменная используется только в одном методе, то лучше и сделать её локальной - объявить в методе
| customers = s.nextInt(); | ||
| while (customers <= 1) { | ||
| System.out.println("Это некорректное значение для подсчёта"); | ||
| System.out.println("Введите корректное количество гостей. Если гостей меньше двух, то расчет не требуется"); |
There was a problem hiding this comment.
Можно объединить два println в один, а перенос сделать при помощи \n
| System.out.println("Введите корректное количество гостей. Если гостей меньше двух, то расчет не требуется"); | ||
| customers = s.nextInt(); | ||
| } | ||
| c = new Calculator(customers, 0); |
There was a problem hiding this comment.
Можно сделать методы и поля Calculator static, чтобы не приходилось создавать экземпляр, а обращаться к его методам напрямую
| } else if (enough.equalsIgnoreCase("да")){ | ||
| break; | ||
| } | ||
| } while (!enough.equalsIgnoreCase("нет") || !enough.equalsIgnoreCase("да")); |
There was a problem hiding this comment.
Это условие всегда true, enough всегда будет или не "нет", или не "да". В таком случае лучше использовать цикл while(true)
| int friendsCount; | ||
| int numberOfCustomers; | ||
| double bill; | ||
| Product p; |
There was a problem hiding this comment.
Неиспользуемая переменная, лучше удалить
| case 2: | ||
| case 3: | ||
| case 4: | ||
| corretFormat = value + " " + "рубля"; |
There was a problem hiding this comment.
Есть принцип, который называется DRY - Don't repeat yourself. Можно вынести повторяющиеся
corretFormat = value + " " в отдельную строку, а в case обрабатывать и добавлять к ней рубли с окончаниями
2 participants
No description provided.