-
Notifications
You must be signed in to change notification settings - Fork 0
Comments
Практическая работа No1 в курсе Яндекс Практикума#1
Практическая работа No1 в курсе Яндекс Практикума #1shishinmaks wants to merge 1 commit intomain from
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.
Поля лучше пометить final, тем самым исключив возможность их модификации извне
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.
От этой переменной можно избавиться, если перенести это значение в константу - private static final int AUTO_COUNT = 3
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.
Минимальную и максимальную скорости лучше вынести в константы для повышения читабельности кода
No description provided.