Repository navigation
Консольное приложение. Проектная работа №1. Проверка 1 - #1
AlexOtkidach wants to merge 2 commits into
Conversation
| //После добавления товара в калькулятор нужно показать пользователю сообщение об успешном добавлении товара. | ||
| System.out.println("Товар успешно добавлен!"); | ||
| //После добавления товара нужно спрашивать у пользователя, хочет ли он добавить ещё один товар. | ||
| System.out.println("Может еще по одной;)"); |
There was a problem hiding this comment.
Такие подряд идущие println лучше объединить в один
|
|
||
| list += p.productName + "\n"; | ||
| } | ||
| public String getResult() { |
There was a problem hiding this comment.
Поплыло форматирование, выглядит не очень читабельно. Есть хороший хоткей, можно нажать Alt + Ctrl + L и студия красиво отформатирует весь код на экране
There was a problem hiding this comment.
Круто, правда очень удобно)
| if (sumGuests <= 0) { | ||
| System.out.println("Ой, кажется вы ошиблись;)! Введите от 2х человек и мы попробуем снова!"); | ||
| } | ||
| System.out.println("Давайте попробуем снова: "); |
There was a problem hiding this comment.
Такие отдельные логические блоки, как этот (ввод и обработка количества людей), а также логические блоки с вводом товаров и выводом результатов лучше выносить в отдельные методы и затем вызывать их там, где необходимо. Это позволяет улучшить читаемость кода, т.к. разнородная логика не смешивается в один сплошной код, а также упрощает поддержку кода.
There was a problem hiding this comment.
Понял, поправлю и буду обращать внимание.
2 participants
No description provided.