Conversation
src/main/java/Main.java
Outdated
| { | ||
| System.out.println("Ошибка. Попробуйте ещё раз. Введите число > 1"); | ||
| } | ||
| } |
There was a problem hiding this comment.
Такие отдельные логические блоки, как этот (ввод и обработка количества людей), а также логические блоки с вводом товаров и выводом результатов лучше выносить в отдельные методы и затем вызывать их там, где необходимо. Это позволяет улучшить читаемость кода, т.к. разнородная логика не смешивается в один сплошной код, а также упрощает поддержку кода.
src/main/java/Main.java
Outdated
| int countPeople; | ||
|
|
||
| while(true) | ||
| { |
There was a problem hiding this comment.
while(true)
{
Не совсем джава-стиль. Вообще есть хороший хоткей, можно нажать Alt + Ctrl + L и студия красиво отформатирует весь код на экране
src/main/java/Main.java
Outdated
| } | ||
| } | ||
|
|
||
| public static class Calculator { |
There was a problem hiding this comment.
В данном случае лучше не делать такую структуру программы со вложенным классом Calculator в классе Main. Лучше вынести Calculator в отдельный файл
src/main/java/Main.java
Outdated
|
|
||
| void out(){ | ||
| System.out.println("Добавленные товары:" + allNames); | ||
| System.out.println("Общая стоимость: " + allPrices + declension(allPrices)); |
There was a problem hiding this comment.
Такие подряд идущие println можно объединить в один
src/main/java/Main.java
Outdated
|
|
||
| void add(String name, double price) { | ||
| allNames = allNames + "\n" + name + " " + String.format("%.2f", price) + declension(price); | ||
| allPrices = allPrices + price; |
There was a problem hiding this comment.
Можно также написать allPrices += price;
| public class CheckInput { | ||
| Scanner scanner = new Scanner(System.in); | ||
|
|
||
| int CheckCountPeople() { |
There was a problem hiding this comment.
в Java принято называть методы camelCase'ом, т.е. с маленькой буквы
No description provided.