Conversation
src/main/java/Main.java
Outdated
| break; | ||
| } else { | ||
| System.out.println("Стоимость товара в формате 'рубли.копейки' (XX.XX): "); | ||
| price = scanner.nextDouble(); |
There was a problem hiding this comment.
Пользователь может ввести на вопрос про стоимость товаров вместо числа строку. Для количества человек ты обрабатываешь такую ситуацию через hasNextInt, стоит добавить обработку и для стоимости
| public double summaryCoast; | ||
| public double moneyForPerson; | ||
|
|
||
| public double sum(double coast) { |
There was a problem hiding this comment.
Возвращаемое значение метода не используется. Можно сделать метод void или использовать значение
| printResult(); | ||
| } | ||
|
|
||
| public static void addGuests() { |
There was a problem hiding this comment.
Можно методы расчета вынести в отдельный класс Калькулятор, сделать не static, а обычными, и в main() создавать объект калькулятора и вызывать методы от него
src/main/java/Main.java
Outdated
| } else { | ||
| System.out.println("Стоимость товара в формате 'рубли.копейки' (XX.XX): "); | ||
| price = scanner.nextDouble(); | ||
| while (price <= 0) { |
There was a problem hiding this comment.
price не меняется внутри, поэтому в случае ввода отрицательного числа программа тут зациклится
| } | ||
| } | ||
|
|
||
| public static double runSum() { |
There was a problem hiding this comment.
Возвращаемое значение метода не используется. Можно сделать метод void или использовать значение
| return prodCalc.summaryCoast; | ||
| } | ||
|
|
||
| public static double runDivide() { |
There was a problem hiding this comment.
Возвращаемое значение метода не используется. Можно сделать метод void или использовать значение
| if (n > 10 && n < 20) return "рублей"; | ||
| if (n1 > 1 && n1 < 5) return "рубля"; | ||
| if (n1 == 1) return "рубль"; | ||
| return "рублей"; |
There was a problem hiding this comment.
Хорошо написанный метод, маленькая вложенность, из-за этого легко читать
No description provided.