-
Notifications
You must be signed in to change notification settings - Fork 0
Comments
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.
В комментарии написано "Модель хранения продуктов", а класс называется Calculator, это может запутать людей которые читают код
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.
В данном случае если пользователь введет строку "два", то он получит ошибку "Неправильный ввод! Количество человек должно быть > 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.
Тут повторяется код, который написан в 22 - 26 строках
Что бы не повторять одинаковые части кода их можно вынести в функции и вызывать их.
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.
Может быть, но насколько помню final позволяет сохранять 1 байт памяти. А присвоение нового инстанца функциональность класса не подразумевает, меняется только внутреннее состояние объекта . Или вопрос в том, что предпочитается помечать final только числовые и текстовые константы, которые явно для этого созданы, чтобы потом не наткнуться на штыки 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.
Понял, спасибо за пояснения!
1.Если возникнут проблемы с отображением кириллицы в консоли, то вот что мне помогло:
Что бы исправить Help menu -> Edit Custom VM Options
добавляем в конец
-Dconsole.encoding=UTF-8
-Dfile.encoding=UTF-8
Перезаходим в студию.
2.Так же реализовал несколько способ ввода для добавления товара т.к. не совсем точно составлено тз
1-й способ Ввод через 2 строки (имя отдельно, цену отдельно)
2-й способ Ввод через одну строку (имя и цена вводиться в одну строку, при этом имя должно быть обернуто в одинарные кавычки)