Skip to content

Проектная работа 1.#98

Open
val-hero wants to merge 3 commits intoYandex-Practicum:masterfrom
val-hero:master
Open

Проектная работа 1.#98
val-hero wants to merge 3 commits intoYandex-Practicum:masterfrom
val-hero:master

Conversation

@val-hero
Copy link

Если возможно, то хотелось бы получить от ревьюера комментарий по поводу циклов while, считается ли нормальной практикой такое использование вложенных циклов для валидации данных и запроса ввода?

@faritowich
Copy link

Привет! Да, почему бы и нет. Это лишь инструмент, и в данном случае он правильно выполняет свою работу по валидации ввода, так что всё отлично. Можно было сделать валидацию по-другому через if, но вовсе необязательно. Можно для улучшения читаемости кода вынести логические блоки по валидации каждого названия и стоимости вынести в отдельные методы, например, enterName, enterPrice.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants