-
Notifications
You must be signed in to change notification settings - Fork 0
пул реквест для первого проекта в яндекс практикуме#1
пул реквест для первого проекта в яндекс практикуме #1posleflamingo wants to merge 1 commit intomain from
Conversation
Komm mit Im Gleichschritt
@ArturNurtdinov
ArturNurtdinov
left a comment
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.
Данный класс предназначен только для хранения данных, которые все инициализируется в конструкторе, поэтому можно полям поставить модификатор final, убрать private и геттеры - получится то же самое, но будет прямой доступ к полям и меньше кода
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.
Для повышения читабельности кода число 24 лучше вынести в константу с говорящим названием
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.