Вопрос №49406 от пользователя Gleb Varganov в уроке «Null Object Pattern», курс «PHP: Полиморфизм»

Gleb Varganov

Добрый вечер! Хотел узнать Вашего мнения по поводу правильности конструкции в ревью. https://ru.hexlet.io/code_reviews/339861. В файле src/FakeSubscription.php мое решение и решение учителя делает одну и ту же логику. Но они реализованы по разному. Есть ли принципиальная разница с точки зрения правильного кода или два решения вполне рабочие?

1 0

Kirill Mokevnin

Они рабочие, но isAdmin уже логическое выражение возвращающее булево значение. Там не нужно дополнительно делать тернарник.

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 месяцев

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

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

Ревью (https://ru.hexlet.io/code_reviews/426571). Не понимаю что я делаю не так. Получается что у меня что-...

Алексей Швецов 29 марта 2021 →

Здравствуйте. При решения испытал некий... диссонанс. В моменте, где нужно было дописать конструктор User д...

Станислав Пефтев 26 августа 2020 →

Обратите внимание на проверку существования пользователя. Если её не сделать, то код упадёт с ошибкой, пото...

Вильдан Фазлыев 14 июля 2020 →

Как так получается, что мы создаем объект класса, передавая в него объект, который в данный момент еще не с...

Евгений Вафиев 09 мая 2020 →

Добрый вечер) Хотелось бы уточнить недостатки такой реализации (https://ru.hexlet.io/code_reviews/263055)))...