Вопрос №56342 от пользователя Dmitry Morozov в уроке «Конфигурация», курс «PHP: Объектно-ориентированный дизайн»

Dmitry Morozov

Я реализовал опции как отдельные переменные, в отличии от решения учителя. Является ли это ошибкой в данном случае, или это просто один из вариантов решения?

3 0

Роман Ашиков

Если ваше решение проходит тесты и линтер не выдаёт ошибок, то оно, как минимум, не плохое и подходит в качестве одного из вариантов решения. В целом, тут нет необходимости выносить опции в отдельные переменные, если я конечно правильно понял ваш вопрос не видя код. Но как один из вариантов, почему нет?

0

Dmitry Morozov

Старался описать вопрос без спойлеров. В решении учителя опции находятся в массиве, в конструкторе происходит мердж переданных опций и далее по коду обращение по ключу массива. Я же сделал опции отдельными переменными, конструктор получается более громоздким, из-за проверки наличия ключа в передаваемых опциях, но далее по коду, как по мне, их использование нагляднее. Собственно вопрос, какой из вариантов будет предпочтительнее на практике?

0

Роман Ашиков

Старался описать вопрос без спойлеров.

Для этого на Хекслете есть код-ревью. Вы можете ссылаться на номера конкретных строк в нём и задавать вопросы. О том как работает код-ревью можно почитать тут — https://help.hexlet.io/ru/articles/111135-kod-revyu

Сложно дать однозначный ответ на ваш вопрос по нескольким причинам: я не вижу ваш код, а также в реальности всё зависит от ситуации. Я хочу напомнить вам, что поддержка Хекслета помогает студентам в процессе поиска решения, также я могу ответить на вопросы по решению учителя. Если ваш код успешно проходит тесты и линтер не выдаёт ошибок, то решение как минимум не плохое и подходит в качестве одного из вариантов.

0

Есть вопрос или хотите участвовать в обсуждении?

Зарегистрируйтесь или войдите в свой аккаунт

Отправляя форму, вы соглашаетесь c «Политикой конфиденциальности» и «Условиями оказания услуг»

Рекомендуемые программы

С нуля до разработчика. Возвращаем деньги, если не удалось найти работу.

Иконка программы Фронтенд-разработчик
Профессия
Разработка фронтенд-компонентов веб-приложений
22 сентября 8 месяцев
Иконка программы Python-разработчик
Профессия
Разработка веб-приложений на Django
22 сентября 8 месяцев
Иконка программы PHP-разработчик
Профессия
Разработка веб-приложений на Laravel
22 сентября 8 месяцев
Иконка программы Node.js-разработчик
Профессия
Разработка бэкенд-компонентов веб-приложений
22 сентября 8 месяцев
Иконка программы Верстальщик
Профессия
Вёрстка с использованием последних стандартов CSS
в любое время 5 месяцев
Иконка программы Java-разработчик
Профессия
Разработка приложений на языке Java
22 сентября 10 месяцев
Иконка программы Разработчик на Ruby on Rails
Профессия
Новый
Создает веб-приложения со скоростью света
22 сентября 5 месяцев

Похожие вопросы

Константин Ситников 25 июня 2021 →

Ревью (https://ru.hexlet.io/code_reviews/418878) Не проходит тест на проверку вот этого значения "another-p...

Ильдар 02 февраля 2021 →

Добрый день. Уже второй день как завис на этом задании. Со временем уже сам путаюсь что написал в итоге. По...

Pavel Mironovich 19 октября 2020 →

Добрый день. Не могли бы Вы объяснить, почему function construct($options) вызывает ошибку Too few argumen...

Ди Анжело 11 сентября 2020 →

Добрый день. Возможно тупой вопрос, но почему то не меняется параметр на 'containNumbers' => true. В чем мо...

Антон Шведов 04 августа 2020 →

Хотя теорию и читал (и не один раз), но пока не посмотрел код учителя в голове не уложилась теория... Может...