Как оптимизировать сложные проверки в Entity?

Question

[tukreb]

Есть Entity с методом add()
В этом методе, есть сложная проверка, которая через Doctrine тянет объекты из разных Entity. На чистом SQL разумеется можно сделать более быстрые запросы, но в Entity этого делать нельзя.
Если вынести это в service, то тогда метод add() в Entity остаётся без проверок, и кто-то или я может случайно вызывать его в обход service.
Как поступают в таких ситуациях? Инжектят через приватных конструктор в Enity требуемые сервисы и потом в нужном методе вызывают? Или пишут статические сервис методы? Но если верить этому https://stackoverflow.com/questions/10330704/symfo... то так делать нельзя

В общем какой здесь best practice?

Comments

[Модератор]

tukreb дорогой пользователь, настоятельно рекомендуем еще раз обратить самое пристальное внимание на п. 3.1 регламента работы сервиса (и, в особенности, на его последний абзац).
В противном случае ваши вопросы будут удаляться по причине тег-спама, а систематические нарушения приведут к блокировке учетной записи.

Answer 1

[Dmitry]

Как вариант, можно вынести только проверку в отдельный сервис в доменном слое (Domain Layer), и этот сервис передавать как аргумент в метод add() вашей сущности. Если этот сервис бросит исключение, то метод add прекращает выполнение. В сервис передавать в конструкторе репозиторий или другой объект, который подгрузит необходимые для проверки данные.

Далее вынести вызов метода add() в сервис на уровне приложения (Application Layer), как вы написали, в котором и передавать ранее созданный сервис проверки в метод add().

В такой реализации вам всегда будет необходим этот доменный сервис, и вызвать метод add() без него не получится.

Пример:

class Entity
{
	public function add(ComplexEntityValidator $validator) {
		$validator->validate();
	}
}

class ComplexEntityValidator
{
	public function __construct(EntityRepository $repository) {

	}

	public function validate() {
		// проверка
	}
}

class EntityApplicationService
{
	public function __construct(ComplexEntityValidator $validator) {

	}

	public function __invoke() {
		$entity = new Entity();
		$entity->add($validator);
		// сохранение сущности
	}
}

Comments

[tukreb]

После чтения stackoverflow у меня появилось такое сомнение (возможно глупое):
Если я буду принимать в Entity в методе add() аргумент сервиса, не значит ли это, что теперь Entity знает об сервисе, хотя по best practice не должен этого знать?

[Dmitry]

tukreb, Eric Evans в своей книге "Domain-Driven Design" (стр. 105) пишет, что это нормально:

When a significant process or transformation in the domain is not a natural responsibility of an ENTITY or VALUE OBJECT, add an operation to the model as standalone interface declared as a SERVICE. Define the interface in terms of the language of the model and make sure the operation name is part of the UBIQUITOUS LANGUAGE. Make the SERVICE stateless.

В моем понимании Entity может знать о сервисе, но только о сервисе доменного слоя, и этот сервис не должен иметь состояния. В вашем случае это выполняется, но передавать в сущность, к примеру, сервис из уровня приложения или инфраструктуры (тот же EntityManager или QueryBuilder) - это да, уже неправильно.

[tukreb]

Дмитрий, спасибо, я так же нашёл что-то похоже используя элементы Doctrine https://stackoverflow.com/a/14685002/10142018

Но ваш вариант мне кажется более чистым :) спасибо.