Правильно ли передавть объект Entity в метод репозитория или лучше передать конкретные элементарые значения?

Question

[gitdev]

В метод нужно передать Entity или конкретные типы как в данном случае. Что более правильно?

public function getSimilar(int $specializationId, int $currentId)
    {
        $v = $this
            ->createQueryBuilder('v')
            ->where('specialization_id = : specializationId')
            ->andWhere('id <> ' . $currentId)
            ->setParameters([
                'specializationId' => $specializationId,
                'id' => $currentId,
            ])
            ->orderBy('expirationDate DESC', 'bonus DESC')
            ->setMaxResults(2)
            ->getQuery()
            ->getResult();
    }

Answer 1

[Alex P.]

Вызывайте метод репозитория с значениями полей сущности в качестве аргументов..

$SomeEntytyREpo->getSimilar($Entity->getId(), $Entity->getOneMoreFieldValue)

Если Вы планируете проводить еще какие - то манипуляции с сущностью перед тем, как делать запрос в БД, то лучше вынесите эту логику в отдельный сервис или метод...

Comments

[gitdev]

почему именно так?

[Максим Федоров]

gitdev, потому что данные скаляры могут быть не обязательно из сущности, а из реквести или из очереди...
То есть сущности может не быть, а по этим айдишникам сущности выбрать нужно

[gitdev]

Максим Федоров, Макс, но в сервис лучше передать объект Entity?

public function getSimilar(Vacancy $vacancy): array
    {
        return $this->vacancyRepository->getSimilar($vacancy->getSpecialization()->getId(), $vacancy->getId());
    }

[tukreb]

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

Только приведённый пример кода, не имеет смысла, зачем получать то, что у вас уже есть.

[Alex P.]

Да