Conversation
| @@ -0,0 +1,31 @@ | |||
| public class Calculator { | |||
| int users; | |||
There was a problem hiding this comment.
Рекомендация: Форматирование кода - в Java принят немного другой стиль форматирования кода. Если не вдаваться в детали, то легко и быстро отформатировать код в Android Studio можно следующей комбинацией клавиш: в Windows Ctrl + Alt + L , в MacOs ⌘ + ⌥ + L.
src/main/java/Formatter.java
Outdated
| } | ||
| case 2, 3, 4 -> | ||
| { | ||
| if (cost % 100 == 12 || cost % 100 == 13 || cost % 100 == 14) |
There was a problem hiding this comment.
Рекомендация: cost % 100 считается несколько раз, имеет смысл вынести в переменную, также имеет смысл оптимизировать условие: cost % 10>= 12 && cost % 10<= 14
src/main/java/Formatter.java
Outdated
| int cost = (int) Math.floor(totalCostForEach); | ||
| switch (cost % 10) { | ||
| case 1 -> { | ||
| if (cost % 100 == 11) currency = " рублей."; |
There was a problem hiding this comment.
Рекомендация: поыторяющиеся строки имеет смысл вынести в константы
src/main/java/Main.java
Outdated
| // Вывод результатов пользователю | ||
| calculator.displayProducts(); | ||
| System.out.println("Итоговая стоимость: " | ||
| + formatter.roundingTheDouble(calculator.calculateScoreForEachPerson()) |
There was a problem hiding this comment.
Рекомендация: calculator.calculateScoreForEachPerson() рассчитывается несколько раз, имеет смысл вынести это значение в переменную
|
Добрый день! Это вторая сдача работы. Просьба проверить.
|
Первая сдача. Просьба принять на проверку. Описание представлено в readme.
Спасибо!