Проект готов 1. Спринт 2. #42
Conversation
src/main/java/Main.java
Outdated
| } else { | ||
| System.out.println("Ошибка. Введите число."); | ||
| scanner.next(); | ||
| } |
There was a problem hiding this comment.
Лучше выносить большие блоки в отдельные методы или классы. Например, для обработки ввода количества людей создать метод void calcuLatePersons (или даже вынести логику в отдельный класс), в котором будет цикл и вся логика обработки ввода. то же касается метода по обработке ввода товаров и их стоимости. Это позволяет делать код более читабельным и аккуратным.
Вообще на тему того, как лучше организовать написание кода, могу посоветовать книгу Роберта Мартина "Чистый Код" .
src/main/java/Main.java
Outdated
| while (true) { | ||
| if (scanner.hasNextInt()) { | ||
| numberOfPeople = scanner.nextInt(); | ||
| if (numberOfPeople > 1) { |
There was a problem hiding this comment.
Лучше по возможности избегать большой вложенности if'ов, это затрудняет понимание логики программы. Впрочем, иногда бывают случаи, когда с вложенными ифами лучше, чем без них
src/main/java/Main.java
Outdated
| calculator.setProducts(product); | ||
| calculator.setTotalPrice(price); | ||
| System.out.println("Товар успешно добавлен."); | ||
| System.out.println("Добавить ещё один товар?"); |
There was a problem hiding this comment.
Чтобы не писать лишний код можно объединять два println в один
| @@ -0,0 +1,78 @@ | |||
| import java.util.Scanner; | |||
|
|
|||
| public class Assistant { | |||
There was a problem hiding this comment.
В таком маленьком проекте это не столь принципиально, но в целом лучше давать сущностям более говорящие названия - чтобы сразу было понятно, для чего они нужны
2 participants
Версия 1. "Консольное приложение"