посоветуйте паттерн рефакторинга по преобразованию метода возвращающего объект композит (шапка - список) в yield

шапка остается шапкой

элементы списка должны быть теперь возвращаться по yield

проблема именно в шапке

шапку тоже надо в yield возвращать?

в шапке агрегированные данные есть?

шапку тоже надо в yield возвращать?

нет

хотя стоп

проблема уровня шарписта: нужно вернуть не yield - список, а "фабрику",производящую этот yield список

проблема уровня шарписта: нужно вернуть не yield - список, а "фабрику",производящую этот yield список

ты так и не научился формулировать вопросы

ТССЯВБ проблема решена преобразованием хелпера в сервис

Привет, сюда для ревьювинга нельзя кинуть ссылку на гитхаб?

Привет всем, есть тут кто хорошо разбирается в классах, PSR, SOLID на php? Хочу спросить посмотреть на репу, есть ли, на ваш взгляд, недочеты в плане реализации.

ссылка на гитхаб + alfredleo/shelter

там всего 5 классов голого пхп

Привет всем, есть тут кто хорошо разбирается в классах, PSR, SOLID на php? Хочу спросить посмотреть на репу, есть ли, на ваш взгляд, недочеты в плане реализации.

Вангую - там все плохо

Вангую - там все плохо

Хотябы 2-3 замечания, если не сложно.

За хороший код ревью могу крипту кинуть 10$ эфиром.

Вангую - там все плохо

лёгкие деньги

ERROR: S client not available

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

Пишите сюда или в личку кому интересно.

Вангую - там все плохо

Бинго ;)

ссылка на гитхаб + alfredleo/shelter

https://github.com/alfredleo/shelter

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

Судя по всему ты очень сильно ActiverRecord любишь ;)

Надо учиться. Тут думаю сидят умные люди). да я из йии

А подскажите пожалуйсто, что было бы отличной проверкой знаний и портфолио на уровень миддл по php? backend?

Пишите сюда или в личку кому интересно.

https://github.com/alfredleo/shelter/blob/master/local/shelter/Shelter.php#L40 не оч удачная затея кмк, лучше все же делать add(Pet $pet), а вот это решение принимать уровнем выше, вообще этот класс похож на обычную коллекцию с захардкоженными методами для конкретного типа, глянь на Doctrine\Collection или что-нибудь такое, потому что привязываться так вот к классу (https://github.com/alfredleo/shelter/blob/master/local/shelter/Shelter.php#L73) не оч удачная затея