Conversation
src/main/java/Calc.java
Outdated
| } | ||
| this.addItem(item); | ||
| return 1; | ||
| } |
There was a problem hiding this comment.
Лучшей практикой будет выносить такие коды (1, -1, 0) в константы при помощи final. Так будет понятно из названия константы, что она обозначает, можно будет отслеживать, где данная константа ещё используется, просто кликнув ctrl+click по объявленной константе. А то сейчас сходу не понятно, что означают эти числовые коды
Плюс если кодовое число используется не в одном месте, как сейчас, то не будет риска поменять в одном месте и забыть в другом
src/main/java/Calc.java
Outdated
| } | ||
| } | ||
|
|
||
| String getWordForm(double sum, String f1, String f2, String f5) { |
There was a problem hiding this comment.
- Если планируем использовать только в данном классе, стоит сделать private
- Лучше давать аргументам более очевидные названия. Из названия f1, f2, f3 непонятно, что они означают
src/main/java/Main.java
Outdated
| } else { | ||
| people = peopleIn; | ||
| return true; | ||
| } |
There was a problem hiding this comment.
Не совсем понятно, почему вся логика по обработка ввода, выводу результата, склонении окончаний находится в одном общем класса, а логика по обработке ввода людей - в Main. Некритично, но можно также вынести
попытка 2