Привет всем!
Хотелось бы показать как заставить других желать тебе сломать руки.
Увы слишком часто сталкиваюсь с подобным в любых проектах что нативный код, симфони или иной ферйм, ui или бек.
https://bitbucket.org/BonBonSlick/workspace/snippets/kxpMrg
Метод ответственный  за профили всех типов пользователей.
Кому понравилось могу кинуть еще индуский код лары.

господибожекакойэтопиздецяебал

Кидай лару

Зачем вы пишете, используя полустатичную надстройку над симфонией?

there is no need to check 5 times if a user is a user
🧐

Этот код конечно не идеал, но он довольно прост в понимании и это уже неплохо ;)

И такое бывает, там проверка на null и вроде instanceof UserInterface дальше. Я так, бегло пробежался, с этим проектом уже не работаю давно.

Только какой-то зашкаливающее количество === на строку кода

Вот код лары, писал индус
https://bitbucket.org/BonBonSlick/workspace/snippets/kxpMre
Чего только стоит

 $output .= $ind['speaker'] . "\r\n" . gmdate(
                    "H:i:s",
                    strtotime($ind['from'])
                ) . " - " . gmdate(
                           "H:i:s",
                           strtotime($ind['to'])
                       ) . "\r\n" . strip_tags($ind['sentence']) . "\r\n\r\n";

или

 $output .= $ind['speaker'] . PHP_EOL . gmdate(
                    "H:i:s",
                    $ind['from']
                ) . " - " . gmdate(
                           "H:i:s",
                           $ind['to']
                       ) . PHP_EOL . strip_tags($ind['sentence']) . PHP_EOL . PHP_EOL;

SOLID? не слышали...

Опубликовал кода лары.

 $output .= $ind['speaker'] . "\r\n" . gmdate(
                    "H:i:s",
                    strtotime($ind['from'])
                ) . " - " . gmdate(
                           "H:i:s",
                           strtotime($ind['to'])
                       ) . "\r\n" . strip_tags($ind['sentence']) . "\r\n\r\n";

 $output .= $ind['speaker'] . PHP_EOL . gmdate(
                    "H:i:s",
                    $ind['from']
                ) . " - " . gmdate(
                           "H:i:s",
                           $ind['to']
                       ) . PHP_EOL . strip_tags($ind['sentence']) . PHP_EOL . PHP_EOL;

Мне стало страшно

Это выглядит печальней даже ;)

А что не так? Везде комментарии, всё понятно

Так делать не стоит. Вносить правки затратно по времени, надо помнить каждый кейс который покрывает метод. Крайне легко сломать, при чем сразу много роутов. К примеру, надо показать флеш месседж не верифицированному  пользователю когда он заходит в профиль если у него указан телефон и оплачена подписка и заходит он первый раз в профиль.

Клиент то думает, всего то надо показать сообщение пользователю и винит разработчика в его некомпетентности, потому что ушло на это 2-4 часа, что может стоить от 20$ до 240$, 10$-60$/ h .

чаще всего такой код, из за нехватки времени. Типа давайте быстре быстре быстре, потом перепишем. А в итоге так и остается.

ну либо в вашем кейса типа фриланс на 2 - 4 часа, можно вообще не надеяться на качество кода.