Пых – Telegram
Пых
8.28K subscribers
260 photos
14 videos
6 files
566 links
Блог Валентина Удальцова о разработке на PHP.

Хобот @phpyhobot
https://youtube.com/@phpyh
https://vkvideo.ru/@phpyh
https://news.1rj.ru/str/isPHPdying

Статистика: https://news.1rj.ru/str/INOTAROBOT?start=st1219340804

Для связи используйте личные сообщения канала.
Download Telegram
Как сделать тестируемым класс, использующий нечистую встроенную функцию PHP?

Очень просто — объявить функцию необязательным параметром конструктора.

final class Service
{
/**
* @psalm-var callable(): int
*/
private $time;

/**
* @psalm-param ?callable(): int $time
*/
public function __construct(?callable $time = null)
{
$this->time = $time ?? 'time';
}

public function do(): void
{
$time = ($this->time)();
}
}

$service = new Service();

$serviceForTesting = new Service(static fn (): int => 1234567890);


Не советую использовать либы-костыли типа php-mock/php-mock.
Через несколько минут я окажусь в гостях у Антона Морева на канале MoreView!

Буду ждать ваши каверзные вопросы 😅
По словам Антона, автор самого крутого вопроса получит 🍕

https://www.youtube.com/watch?v=LOrb3m0FQdU
Вчера в интервью 🎙 я умудрился в одно предложение засунуть сразу 4 баззворда:
функциональный язык, агрегат, Event Sourcing и чистые функции.

Спешу поделиться докладом, который придал мне столько смелости 🤣
У него довольно неочевидное название, но очень крутое содержание.

https://www.youtube.com/watch?v=USSkidmaS6w
Media is too big
VIEW IN TELEGRAM
После интервью ребята в нашем обсуждении предложили выяснить, как много среди нас музыкантов 🎼

За барабанами ваш покорный слуга ☺️
На прошлой неделе я провёл стрим про утечки памяти в PHP

По ссылке запись трансляции с таймкодами и разбивкой на эпизоды.

https://www.youtube.com/watch?v=NNMp-97rk9c

По результатам я предложил два изменения в Symfony, оба уже приняты:

#37705 добавить тег kernel.reset для сервиса mailer.logger_message_listener,

#37712 перенести MessageLoggerListener в конфиг mailer_debug.
time_diff_ms(DateTimeImmutable, DateTimeImmutable): int

Вчера игрался с задержками (delay) в брокерах сообщений.
Потребовалась функция для нахождения разницы между двумя метками времени в миллисекундах.

Ловите Gist с функцией и юнит-тестом 🍭

https://gist.github.com/vudaltsov/0bb623b9e2817d6ce359eb88cfbf229d
В обсуждении ребята подняли вопрос об учёте часовых поясов в функции из поста выше.

time_diff_ms() корректно учитывает временные зоны, так как сравнивает абсолютные метки времени, а не время настенных часов (wall-clock time).

Попутно мы выяснили, что метод DateTime(Immutable)::setTimezone() не изменяет абсолютное время, просто адаптирует время настенных часов под часовой пояс. Метка времени до и после setTimezone() остаётся такой же. Смотрите также комментарий к документации метода.

Добавил в Gist тесты для всех сценариев создания объекта времени.

https://gist.github.com/vudaltsov/0bb623b9e2817d6ce359eb88cfbf229d#file-time_diff_ms_test-php
Как одним выражением прочитать и изменить свойство

[$currentValue, $this->property] = [$this->property, $newValue];

Этот приём можно использовать, например, в базовом классе корня агрегата.

abstract class AggregateRoot
{
/**
* @psalm-var list<object>
*/
private array $events = [];

/**
* @psalm-return list<object>
*/
final public function releaseEvents(): array
{
[$events, $this->events] = [$this->events, []];

return $events;
}

final protected function raise(object $event, object ...$events): void
{
$this->events = [...$this->events, $event, ...$events];
}
}


Некогда подглядел у @fes0r.
Недавно узнал, что есть подход, при котором юнит-тесты кладут рядом с тестируемыми классами 🤔

Примеры:
https://github.com/thephpleague/flysystem/tree/2.x/src,
https://github.com/EventSaucePHP/EventSauce/tree/master/src/Integration.

Выглядит заманчиво — не надо синхронизировать две файловые структуры и бегать туда-сюда. И на ревью сразу видно, написан тест для нового класса или нет. Также этот подход должен повысить мобильность модулей кода — модуль не надо собирать по папкам src и test, чтобы переименовать или вынести в Composer-пакет.

В библиотеках такие тесты легко удалить из архива правилом **/*Test.php export-ignore в .gitattributes. Проект можно аналогичным образом чистить перед деплойментом.

Что думаете? Приглашаю обсудить в Пыхтелку 🐯
Отличная статья про именованные агрументы от автора Psalm Мэтта Брауна!

В ней рассказывается про нюансы использования именованных аргументов в PHP 8 и про то, как новая версия Psalm 3.14.0 поможет их снивелировать.

https://psalm.dev/articles/getting-ready-for-named-arguments
Оказывается, для Symfony уже давно есть Psalm-плагин 😮

На радостях сделал 5 пулл-реквестов на базе стабов, которые мы скопили на рабочем проекте. Все 5 были приняты и вошли в релиз 1.4.3 ☺️

Кто до сих пор не использует статический анализ, советую просмотреть мои PR. Приёмочные тесты дадут вам некоторое представление о том, как статический анализ может вывести типизацию привычных инструментов на новый уровень.

https://github.com/psalm/psalm-plugin-symfony
Forwarded from PHP Digest
Открытое собеседование по PHP | Эпизод #2 — Прием заявок

Пилотный выпуск открытого собеседования зашел отлично, и мы получили много ценных отзывов. Спасибо всем, кто был с нами на стриме и задавал вопросы!

Если вдруг вы не видели первый выпуск, то вот запись https://www.youtube.com/watch?v=FQNd9W3nb3A

Тем временем начинаем подготовку второго выпуска!

Требования для участия не поменялись:
• уровень middle/senior;
• PHP 7.x, PSR;
• ООП, SOLID, coupling/cohesion, вот это все;
• тестирование, PHPUnit;
• желателен опыт с Symfony 4/5;
• SQL, желательно PostgreSQL;
• представление о современных трендах в архитектуре приложений.

Темы будут ± те же, а вот вопросы подготовим новые, без Psalm 🙂

Заявку на участие можно отправить до 27 августа через форму: https://forms.gle/ES3nXiwf4ycosGEy9.

Вопросы в личку: @vudaltsov, @pronskiy.
Раньше, чтобы создать nullable ValueObject из nullable примитива, приходилось писать колбасу вроде

null === $stringClientId ? null : ClientId::fromString($stringClientId)
.

Сегодня условные типы Psalm позволяют перенести if в статический конструктор:

/**
* @template T of ?string
* @psalm-param T $id
* @psalm-return (T is null ? null : self)
*/
public static function fromString(?string $id): ?self
{
if (null === $id) {
return null;
}

Assert::uuid($id);

return new self($id);
}

ClientId::fromString($stringClientId)


https://psalm.dev/r/ab0090be4c
Рекомендую доклад Temporal Modeling.

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

https://youtu.be/hh5lskgATCk

Список всех докладов Матиаса на эту тему
> Вам нужны условия Йоды только если ваши тесты 💩 и вы думаете, что перестановка аргументов сравнения имеет большее значение, чем корректная запись ваших бизнес-правил.

https://twitter.com/frankdejonge/status/1298602389281411072

Мы используем в проекте условия Йодыфиксер yoda_style), но этот твит заставил меня задуматься, зачем 🤣

Когда-то давно я бы ответил "потому что так в Symfony Coding Standards", позже "потому что так удобнее присваивать результат сравнения", сегодня "только потому что привык".
Используете ли вы условия Йоды?
Anonymous Poll
32%
да
68%
нет