Оправданно ли использование doctrine в хелпер классе symfony 3?

Question

[redguard]

Хочу писать написать функцию которая достает записи из бд по id. И потом использовать ее во всех контроллерах. Собираюсь разместить эту функцию в хелпер классе который будет находиться тут AppBundle\Utils; Но моя функция в хелпере (Вернее даже весь хелпер класс) не будет иметь сразу доступа к doctrine.
Вот что пишут на stackoverflow:

The problem here is that your Helper class is not container-aware; that is, it has no idea about all the services Symfony has loaded

that you're injecting the entire service container into the Helper class. That's not a best practice -- you should only inject what you need

Оправданно ли использование doctrine в хелпер классе symfony 3?
Как правильно разметить такую функцию? В контроллере?
И как понимать вот это "container-aware"?:) Перевести то перевел но понять не могу

Answer 1

[Александр Евгеньевич]

Используйте свой класс Repository, в конструктор которого пихайте EntityManager, далее уже ваши сервисы смогут иметь доступ к этому репозиторию. Так вы изолируете доктрину.

Comments

[redguard]

Я новичок в symfony. И немного запутался. Хочу уточнить правильно ли я вас понял. Сделать я должен примерно это?

namespace AppBundle\Entity;
use Doctrine\ORM\EntityManager;

class UserRepositiory extends \Doctrine\ORM\EntityRepository
{

private $em;

public function __construct(EntityManager $em)
{
  $this->em = $em;
}

//далее написать функции для работы с бд 

}

[Александр Евгеньевич]

redguard: не наследуйтесь от доктриновского репозитория

[redguard]

Александр Евгеньевич: Оке понял

Answer 2

[Сергей Протько]

container-aware - знает про контейнер и может работать с ним, чего по хорошему оно не должно уметь делать.

Хочу писать написать функцию которая достает записи из бд по id.

Записи из БД или сущности? Это вопервых весьма большая разница, а во вторых уже же есть старый добрый $em->find(Entity::class, $id), парам конвертеры дефолтные и т.д

функцию в хелпер классе который будет находиться тут AppBundle\Utils

Вообще это весьма плохая идея. И про то что будет глобальная функция которая этим будет заниматься, и то что вы ее как статику в класс запихаете а не сервис сделаете, и то что Utils... это называется "мне было лень придумывать адекватное название".

Comments

[redguard]

Записи из БД или сущности?

К вопросу отношения не имеет. Я хочу просто иметь возможность использовать доктрину.

Utils... это называется "мне было лень придумывать адекватное название".

Действительно лень. И название как в документации оставил

сервис сделаете

Вот так?

services:
    app.myservice:
        class: AppBundle\Utils\MyService

Вообщем я так и сделал. Но оправданно ли

injecting the entire service container into the Helper class.

?

[Сергей Протько]

redguard: не оправдано. И вам НЕ НУЖНО инджектить весь сервис контейнер, только entity manager.

[redguard]

Сергей Протько: Понял. Спасибо.