-
Notifications
You must be signed in to change notification settings - Fork 0
Comments
Conversation
@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.
Лучше добавить проверку параметра people, чтобы случайно не поделить на 0, если в будушем эту функцию будет кто-то переиспользовать
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.
productListPrice / people можно посчитать 1 раз, записать результат в переменную и ей пользоваться
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.
Я бы назвал класс PeopleAmountReader, так будет понятнее, за что он отвечает
No description provided.