-
Notifications
You must be signed in to change notification settings - Fork 0
Comments
Conversation
калькулятор Сальников Р.М.
@doc-ODA
doc-ODA
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.
поля лучше делать приватные и добавлять геттеры и сеттеры к ним(это называется инкапсуляция, ты ограничиваешь доступ к переменным напрямую)
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.
Привет. Поля изменил, думал что речь об ошибках только здесь ведется, поторопился и отправил только с измененными полями, а ошибки описанные в YP не исправил(( Буду исправлять!
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.
Все еще жду исправлений)
калькулятор Сальников Р.М.
На проверку