Conversation
| } | ||
| } else { | ||
| System.out.println("Вы ввели некорректное значение. Введите числовое значение больше 1"); | ||
| scanner.nextLine(); |
There was a problem hiding this comment.
Мы во всех условиях вызываем метод scanner.nextLine();, почему бы не вынести его наружу всех условий?
src/main/java/Main.java
Outdated
| product.productName = scanner.nextLine(); | ||
| System.out.println("Введите стоимость товара в формате \"рубли,копейки\""); | ||
|
|
||
| while(true) |
There was a problem hiding this comment.
Лучше не пренебрегать фигурными скобками - их отсутствие может сильно затруднить читаемость кода.
| } else { | ||
| System.out.println("Вы ввели некорректное значение. Введите числовое значение"); | ||
| scanner.nextLine(); | ||
| } |
There was a problem hiding this comment.
Можно вынести некоторые логические блоки, например, весь цикл while, отвечающий за обработку ввода товаров, в отдельный метод - это позволяет делать код более читабельным и аккуратным. То же касается других логических блоков - обработка ввода количества людей, конечный вывод.
Вообще на тему того, как лучше организовать написание кода, могу посоветовать книгу Роберта Мартина "Чистый код" .
src/main/java/Main.java
Outdated
| String ending = new String(); | ||
|
|
||
| public void add(Product product) { | ||
| total = total + product.productPrice; |
There was a problem hiding this comment.
Можно total += product.productPrice;
src/main/java/Main.java
Outdated
|
|
||
| public void add(Product product) { | ||
| total = total + product.productPrice; | ||
| shoppingList = shoppingList + product.productName + " " + product.productPrice + "\n"; |
There was a problem hiding this comment.
Можно shoppingList += product.productName + " " + product.productPrice + "\n";
src/main/java/Main.java
Outdated
| class Product { | ||
| String productName; | ||
| double productPrice; | ||
| } |
There was a problem hiding this comment.
Лучше классы выносить в отдельные файлы. 1 файл - 1 класс.
|
Во втором коммите я добавил логику обработки некорректного ввода отрицательной стоимости товара, и внес в код косметические изменения, на которые указали в ревью. В отдельный метод обработку некорректного ввода пока не выносил - хочу сдать скорее и бежать к третьему спринту, отставание в неделю нагонять. Но у меня остались несколько вопросов:
|
Это первая попытка сдачи ДЗ. Первый отправленный на ревью пулреквест вышел неудачным, поэтому исправился здесь