-
Notifications
You must be signed in to change notification settings - Fork 0
Comments
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Рекомендация: Форматирование кода - в Java принят немного другой стиль форматирования кода. Если не вдаваться в детали, то легко и быстро отформатировать код в Android Studio можно следующей комбинацией клавиш: в Windows Ctrl + Alt + L , в MacOs ⌘ + ⌥ + L.
src/main/java/Formatter.java
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Рекомендация: cost % 100 считается несколько раз, имеет смысл вынести в переменную, также имеет смысл оптимизировать условие: cost % 10>= 12 && cost % 10<= 14
src/main/java/Formatter.java
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Рекомендация: поыторяющиеся строки имеет смысл вынести в константы
src/main/java/Main.java
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Рекомендация: calculator.calculateScoreForEachPerson() рассчитывается несколько раз, имеет смысл вынести это значение в переменную
marugish
commented
Jan 14, 2024
Добрый день! Это вторая сдача работы. Просьба проверить.
0) Исправила ошибку в обработке названия товара
- Отформатировала код с помощью комбинации Ctrl+Alt+L , согласно правилам Java
- Вынесла cost % 100 в отдельную переменную, оптимизировала условие
3 и 4) повторяющиеся строки и выражения вынесла в отдельные переменные
Спасибо!
Первая сдача. Просьба принять на проверку. Описание представлено в readme.
Спасибо!