-
Notifications
You must be signed in to change notification settings - Fork 0
Comments
Conversation
src/main/java/Calc.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.
Лучшей практикой будет выносить такие коды (1, -1, 0) в константы при помощи final. Так будет понятно из названия константы, что она обозначает, можно будет отслеживать, где данная константа ещё используется, просто кликнув ctrl+click по объявленной константе. А то сейчас сходу не понятно, что означают эти числовые коды
Плюс если кодовое число используется не в одном месте, как сейчас, то не будет риска поменять в одном месте и забыть в другом
src/main/java/Calc.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.
- Если планируем использовать только в данном классе, стоит сделать private
- Лучше давать аргументам более очевидные названия. Из названия f1, f2, f3 непонятно, что они означают
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.
Не совсем понятно, почему вся логика по обработка ввода, выводу результата, склонении окончаний находится в одном общем класса, а логика по обработке ввода людей - в Main. Некритично, но можно также вынести
Uh oh!
There was an error while loading. Please reload this page.
попытка 2