👿 Антийода-условия и RTFM

Некоторое время назад я задумался, насколько реально нужны условия Йоды, которые мы тогда по инерции практиковали на проекте.

Рассуждение следующее. Условия Йоды — это способ защитить ногу от пули при использовании присваивания в условиях. Решая надуманную проблему, мы в итоге получаем двукратное ухудшением читабельности кода: и спрятанное в if присваивание, и неестественное условие. Куда правильнее просто запретить присваивания в условиях, а сами условия оформлять как душа бизнес-логика подсказывает.

В итоге мы на проекте отказались от Звёздных войн, и я добавил строчку 'yoda_style' => false в .php_cs.dist, то есть полностью выключил правило.

И только сегодня, спустя 7 месяцев, @EtherLord выяснил, что это правило можно включить в режиме "антийоды": 'yoda_style' => ['equal' => false, 'identical' => false, 'less_and_greater' => false]. Тогда фиксер null !== $value будет преобразовывать в $value !== null и т.д.

RTFM, Валентин 😅

https://cs.symfony.com/doc/rules/control_structure/yoda_style.html

Уже через 5 минут к нам в гости на PHP Point заглянет Виктор Большаков, и мы из первых уст узнаем, кто такие CTO!

Присоединяйтесь к эфиру и задавате вопросы!

https://youtu.be/sGbi8A899Lg

Полезный сниппет

• Nowdoc с именем SQL сразу включает соответствующий Language injection в PhpStorm. После этого строка подсвечивается и автокомплитится (тонко настраивается в SQL Resolution Scope).

• Nowdoc запроса оформлен с комфортным для чтения отступом слева. Отступ не попадёт в результирующую строку, а будет обрезан в соответствии с закрывающим тегом. Доступно с PHP 7.3.

• PDOStatement с рождения является Traversable, а с 8.0 имплементирует IteratorAggregate.

• Строки резалт сета разобраны на переменные прямо в цикле foreach. Доступно с PHP 7.1.

Чего-то не хватало во вчерашнем сниппете, как-то не по-пыховски было, да? 😅

Я имею в виду, конечно, статанализ. Очевидно, Psalm не знает, что в резалт сете имеются ключи 'question_id' и 'title', и потому генерирует ошибку. Наша задача максимально деликатно ему помочь и явно прописать тип как можно ближе к "источнику".

Ещё вчера стаб для PDOStatement был неполным, но сегодня Мэттью смерджил мой PR, и теперь можно дополнить сниппет соответствующим phpdoc. Напоминаю, что необязательно указывать имя переменной в @var, если аннотируется ближайшее выражение справа.

Вуаля: https://psalm.dev/r/8c9ab50ba2.

🎙 22 апреля на первой Dats.Conf я расскажу, как структурировать код, чтобы не получить «большой ком грязи»

А пока приглашаю вас посмотреть интервью, которое в преддверии митапа у меня взял уже знакомый вам Виктор Большаков.

https://youtu.be/1ckezPnndNM

🎤 Сегодня я выступаю в Нижнем на митапе про организацию кода!

В 11:10 Денис Юрьев (Skysmart от Skyeng) расскажет, как переделать Symfony-проект в целое направление.

В 12:05 я расскажу, как структурировать код, чтобы не получить «большой ком грязи». Обсудим разные подходы к структурированию, принципы проектирования пакетов и инструменты для контроля.

Если вы в Нижнем, то приходите в Лекторий Правда, если нет — подключайтесь к трансляции на канале PHP Point!

https://youtu.be/SycSx0Qp3eg?t=3876

Continuous Integration

CI — must have для проекта любого размера. CI повышает качество кодовой базы, дисциплинирует команду и сокращает количество и продолжительность ревью.

Идеи проверок на базе нашего пайплайна в Happy Inc.:
• кодстайл (PHP CS Fixer, PHP_CodeSniffer),
• статический анализ (Psalm, PHPStan, PHPMD),
• валидность composer.json/lock (composer validate),
• наличие лишних пакетов (composer-unused),
• отсутствие пакетов в списке явных зависимостей (ComposerRequireChecker),
• уязвимости в пакетах (Roave Security Advisories, Local PHP Security Checker),
• синтаксис Yaml-файлов (Symfony Yaml),
• синтаксис Twig-шаблонов в проектах на Symfony (bin/console lint:twig),
• соответствие типов инъекций контейнера Symfony (bin/console lint:container),
• депрекации сервисов и конфигов Symfony (bin/console debug:container --deprecations),
• маппинг Doctrine и соответствие ему схемы БД (bin/console doctrine:schema:validate),
• конфигурация Doctrine для production (bin/console doctrine:ensure-production-settings --env=prod),
• связность/зацепление и направление зависимостей (Deptrac, dePHPend),
• ну и конечно же, тесты!

Также обратите внимание на репозиторий Static analysis tools for PHP и доклад 25+ инструментов для аудита кода.

API Payload Naming Convention

Всё очень просто: чем меньше трансформаций, тем лучше.

Например, вы проектируете API для сервиса на PHP. Ваши потенциальные клиенты — фронт на TypeScript, приложение для Android на Kotlin и приложение для iPhone на Swift. Беглым гуглением выясняем, что все четверо для свойств объектов используют camelCase. Стоит ли выбирать snake_case? Нет.

Только в случае легаси API имеет смысл дальше строгать эндпойнты в том же стиле, чтобы избежать винегрета.

PHP Дайджест Стрим #202-204

Сегодня вечером на канале PHP Point пробегусь по трём весенним PHP Дайджестам. Начало, как обычно, в 20:00, всех жду!

https://youtu.be/DYNbjSTjwPk

Продвинутая интерполяция

Иногда в длинную строку нужно подставить константы / статические свойства или результаты функций / статических методов.

Первой на ум приходит интерполяция, но в PHP она невозможна без переменной, даже при использовании фигурных скобок. То есть вот так нельзя: echo "Число Эйлера: {M_E}". Можно для всех выражений объявить переменные и уже их интерполировать, но получится слишком громоздко. Можно оформить через sprintf, но в многочисленных безымянных %s и %20$d будет легко запутаться.

Костыль Выход есть! Присвоим переменной имя функции strval. Мы получили "интерполятор" callable(mixed): string, при помощи которого в литерал можно подставить любое приводимое к строке выражение:

$intrp = 'strval';
echo "Число Эйлера: {$intrp(M_E)}";

Если пойти чуть дальше и обернуть sprintf в класс с __invoke, зафиксировав первый аргумент (partial application!), получится мощный универсальный интерполятор.

https://3v4l.org/DogHB

🎙 PHP Russia 2021

Два года назад я дебютировал на PHP Russia с митапом Как контрибьютить в Symfony и зачем это делать.

В этом году у меня полноценный доклад Thesis: как забыть про ORM и перейти на нативные SQL-запросы с очень интересной повесткой! Всех жду, в первую очередь оффлайн в Москве.

По промокоду PHPforFriends можно сэкономить 7% на билетах.

https://phprussia.ru/moscow/2021/abstracts/7654

❤️ PHP Russia 2021

В понедельник прошла конференция PHP Russia. В том, что она состоялась сейчас, на гребне второй волны, огромная заслуга организаторов. Подход был настолько серьёзным, насколько это можно себе представить.

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

Огромное спасибо @samdark, @Halfnomad и @pronskiy за глубокое понимание и поддержку! С таким программным комитетом можно успешно выступить прямо в аду.

А ещё я был рад видеть вас, подписчиков, в зале! Легко и приятно на сцене, когда с половиной аудитории ты давно в диалоге.

Я счастлив как слон, до встречи на следующей конференции! 🤗

#[Route(name: self::class)]

Лайфхак для симфонистов. Если вы размещаете каждый экшн в отдельном invokable классе, то его имя можно использовать в качестве имени роута.

При таком подходе не нужно соблюдать никакие конвенции именования и хардкодить строки.

Работает и с атрибутами, и с аннотациями, однако в аннотациях нельзя использовать self, только само имя класса. На скрине оба примера.

Кодишь 2.0

Мама, я в телевизоре!

Большое спасибо организаторам конференции Кодишь 2.0 за приглашение и отличные выходные в Брянске!

https://youtu.be/tBgpNqkoN_k

P.S. Thesis ну прям очень скоро будет, в поезде многое доделал.

Тесты должны быть независимыми

Юнит-тесты должны быть независимыми по определению. Всё необходимое инициализируется в блоке Arrange и утилизируется из памяти после Assert. Использовать в юнитах setUp / tearDown я не рекомендую.

Тесты, работающие с общим состоянием, должны после выполнения возвращать его в исходную точку. Для этого можно использовать in-memory хранилища или откатывать транзакции.

Чтобы защитить себя от неявных зависимостей между тестами, следует запускать их в случайном порядке. Для этого прописываем <phpunit executionOrder="random"> в phpunit.xml или выполняем phpunit с флагом --order-by=random.