Появилась дилемма, делать толстый контроллер или оставит все как есть.
Есть небольшой класс RandomService. Он получает список граждан и фильтрует их по критериям
Код: Выделить всё
$randomService = new RandomService($row->getFullWallId());
$randomService->applyTowns($row->towns);
$randomService->applyGroups($row->groups);
$randomService->applyCheckFixed($row->pinned_down);
$winners = $randomService->getRandomUsers($row->count_winners);
https://db.tt/qHxexAnIbU
Но т.к это начало использоваться уже в нескольких местах, есть подозрения, что эти методы надо убрать в сам класс RandomService
Тогда получается, должно быть что то вроде
Код: Выделить всё
$randomService = new RandomService($row->getFullWallId(), $row->towns, $row->groups, $row->pinned_down);
$winners = $randomService->getRandomUsers($row->count_winners);
Код: Выделить всё
$randomService = new RandomService(CompRand $row);
$winners = $randomService->getRandomUsers($row->count_winners);