-
Notifications
You must be signed in to change notification settings - Fork 0
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.
🍏 Лучше пометить эти поля как private и добавить для них геттеры/сеттеры. О них можно почитать здесь https://javarush.com/groups/posts/1928-getterih-i-setterih
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.
👍 Круто, что вынес в константы эти значения
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.
👍 Здорово, что обработал некорректный ввод
Это мой Pull Request, вы только посмотрите!