Березин Максим: проектная работа №1 - первый pull request#126
Березин Максим: проектная работа №1 - первый pull request#126MBerezin wants to merge 18 commits intoYandex-Practicum:masterfrom
Conversation
This reverts commit 0a10f99.
add new methode to show division result
src/main/java/Main.java
Outdated
| Locale.setDefault(Locale.US); | ||
|
|
||
| Scanner scanner = new Scanner(System.in); | ||
| int peopleCounter; |
There was a problem hiding this comment.
Можно объединить инициализацию переменной с ее объявлением
int peopleCounter = countPeople(scanner);
Calculator calculator = new Calculator(peopleCounter);
| double productPrice; | ||
| String productName; | ||
| Product product; | ||
|
|
src/main/java/Main.java
Outdated
|
|
||
| productName = scanner.next(); | ||
|
|
||
| if (productName.equalsIgnoreCase("ЗАВЕРШИТЬ")){ |
There was a problem hiding this comment.
"ЗАВЕРШИТЬ" лучше вынести в статическую константу
src/main/java/Calculator.java
Outdated
| } | ||
|
|
||
| private String getCurrencyWriting(double price){ | ||
| int roundedPrice = (int)Math.floor(price); |
There was a problem hiding this comment.
Math.floor округляет до целого в меньшую сторону, а приведение к int отбрасывает дробную часть. То есть это по сути эквивалентные операции, но с разными типами в результате. Поэтому нет смысла делать и то и другое. Лучше сделать только Math.floor потому что это более явно.
| return String.format("%,.2f", price); | ||
| } | ||
|
|
||
| private String getCurrencyWriting(double price){ |
There was a problem hiding this comment.
Немного некорректно рассчитывается окончание. Получается если цена будет 121 рубль, метод вернет "рублей". Правильный алгоритм звучит так:
Проверить лежит ли остаток от деления на 100 в интервале от 11 до 14 включительно, если да, то возвращаем "рублей", если нет, то берем остаток от деления на 10 и прогоняем с твоими проверками.
То есть нужно во-первых проверять именно остаток от деления, во-вторых учесть, что числа заканчивающиеся на 11-12-13-14 и заканчивающиеся на 1-2-3-4, будут иметь разные окончания
- delete redundant indents - unite declaration and initialisation some variables
No description provided.