Repository navigation
Pull Request - #1
SAndreyV1683 wants to merge 7 commits into
Conversation
| calculate(product); | ||
| } | ||
|
|
||
| public void calculate(Product product) { |
There was a problem hiding this comment.
Привет. Этот метод можно сделать приватным, потому что он используется только в классе Calculator. Не критично.
| @@ -0,0 +1,23 @@ | |||
| public class Formatter { | |||
|
|
|||
| float sum; | |||
There was a problem hiding this comment.
на будущее, лучше явно указывать модификаторы доступа в java. Переменные без модификатора доступа по умолчанию являются package-private. Это не всегда подходящий модификатор.
| System.out.println("Введите стоимость товара."); | ||
| float price = 0; | ||
| boolean isFloat = false; | ||
| do { |
There was a problem hiding this comment.
несколько вложенных циклов do-while плохо читаются. Хорошим подходом будет вынести вложенный цикл в отдельную функцию
| do { | ||
| if (scanner.hasNextFloat()){ | ||
| price = scanner.nextFloat(); | ||
| if (price < 1) { |
There was a problem hiding this comment.
Привет! Поскольку ты используешь дробные значения, этим условием ты отсекаешь цены от 0 до 1. Правильным условием является if(price <= 0).
There was a problem hiding this comment.
Добрый день, спасибо.
2 participants
Это мой первый Pull Request