-
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.
🍏 Для всего взаимодействия с пользователем достаточно открыть один сканнер
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 принято переменные писать в camelCase, а названия классов CamelCase.
SNAKE_CASE используется для именования констант, а snake_case - для именования xml файлов.
https://www.oracle.com/java/technologies/javase/codeconventions-namingconventions.html
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.
👏 Круто, что делаешь проверку на наличие числа
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.
🍏 При печати в памяти будут создаваться промежуточные строки из-за множественного использования оператора "+". Можно использовать StringBuilder, который сформирует только результирующую строку, или String.format()
kgornostaeva
commented
Oct 25, 2023
🍏 Классно было бы подсказать пользователю, как выйти из сценария
Изменение ревьюера Co-authored-by: kgornostaeva <48408134+kgornostaeva@users.noreply.github.com>
Изменения ревьюера Co-authored-by: kgornostaeva <48408134+kgornostaeva@users.noreply.github.com>
Hi, this is my first PR project