Приведу пример кода, для пояснения дилемы:
1 вариант:
ArticleController
{
public function createArticle(Request $request, ArticleService $articeService, UserService $userService, EntityManager $em){
$articeService->create(CreateArticleDTO::createFromRequest($request));
$userService->incrementArticleCount($this->getUser());
$em->flush();
}
}
ArticleService{
public function (CreateArticleDTO $dto)
{
/// create $entity
$this->entityManager->persist($entity);
}
}
UserService{
public function (User $user)
{
$user->setArticelCount($user->getArticelCount()+1);
}
}
2 вариант.
ArticleController
{
public function createArticle(Request $request, ArticleService $articeService, UserService $userService){
$articeService->create(CreateArticleDTO::createFromRequest($request));
$userService->incrementArticleCount($this->getUser());
}
}
ArticleService{
public function (CreateArticleDTO $dto)
{
/// create $entity
$this->entityManager->persist($entity);
$this->entityManager->flush();
}
}
UserService{
public function (User $user)
{
$user->setArticelCount($user->getArticelCount()+1);
$this->entityManager->flush();
}
}
1 Варинт.
За: Где то видел, что это было в лучших практиках. Но воможно это было в эпоху 2 симфони, а возможно я вообще ошибаюсь. Смысл - что все сущности сохранятся в однйо транзакции. Или всё или ничего.
Против: занижаем связность. Т.е. нельзя быть уверенным, что вызвав данный сервис - статья сохранится. Придет другой разработчик, увидит название, вызовет - а статья не сохраняется. В тестировании надо отдельнов вызывать flush, чтобы проверить что статья сохранится. Вроде тестируем сервис, а вроде нужно что-то поверх него вызывать, чтобы проверить работоспособность. Если будет пакетное сохранение, то все равно придется вызывать flush в сервисах - архитектура будет не единой.
2) Вараинт: ну единственный минус - это несколько транзакций вместо одной. Что вроде как против тех же лучших практик. Возможно что-то еще, чего я не знаю.