Итоги конкурса!
Как я проверял ваши решения 🤓
Если ответ был текстовый, то я оценивал правильность идеи. Если ответ был кодом, то я тестировал его при помощи InMemoryCachingReportFactoryTest.php, предварительно адаптировав тест, если были изменены названия классов или добавлены зависимости. Если тест крашился из-за банальной ошибки (например, неинициализированного типизированного свойства), я такой ответ не засчитывал. Если в коде визуально были неточности, но мой несложный тест он проходил, я засчитывал. Думаю, это справедливо, всё-таки совсем непротестированный код на прод заливать не стоит 😉
Победитель 🏆
Первый правильный ответ пришёл от Andrew (@Groonya):
• https://gist.github.com/aivchen/39a956afc47634c7f8bb2de4688aa7c1 (PHP 7),
• https://gist.github.com/aivchen/5c8d51bed4188910ed34660c3b92fea0 (PHP 8).
Интересно, что в первом сниппете Андрея тоже есть неинициализированное свойство
Пояснение от меня 💡
Эту задачу я рассмотрю сегодня в рамках своего доклада. Решение выложил отдельным видео на PHP Point. Присоединяйтесь к PHPFest через 2 часа по ссылке https://2020.phpfest.ru/showtime/, всех жду!
Как я проверял ваши решения 🤓
Если ответ был текстовый, то я оценивал правильность идеи. Если ответ был кодом, то я тестировал его при помощи InMemoryCachingReportFactoryTest.php, предварительно адаптировав тест, если были изменены названия классов или добавлены зависимости. Если тест крашился из-за банальной ошибки (например, неинициализированного типизированного свойства), я такой ответ не засчитывал. Если в коде визуально были неточности, но мой несложный тест он проходил, я засчитывал. Думаю, это справедливо, всё-таки совсем непротестированный код на прод заливать не стоит 😉
Победитель 🏆
Первый правильный ответ пришёл от Andrew (@Groonya):
• https://gist.github.com/aivchen/39a956afc47634c7f8bb2de4688aa7c1 (PHP 7),
• https://gist.github.com/aivchen/5c8d51bed4188910ed34660c3b92fea0 (PHP 8).
Интересно, что в первом сниппете Андрея тоже есть неинициализированное свойство
$reports
, однако оператор ??=
на это по понятным причинам не ругается, поэтому формально код корректный.Пояснение от меня 💡