我的服务应该实现事务和提交方法吗？

Question

假设我想在我的UsersController 中进行批量用户更新。

在我的UsersController 我愿意：

foreach ($users as $user) {
    $userService = new UserService();
    $user->updateUser($data);
}

如果有很多用户，它可能会变慢，因为 UserService::updateUser 方法只是执行persist()/flush()

所以我想知道这样做是否是个好主意：

class UserService {
  public function setUseTransaction($flag)
  {
      $this->useTransaction = $flag;
      return $this;
  }

  public function updateUser($data)
  {
     // some data mapping

     $entityManager->persist($user);

     if ($this->useTransaction) {
       $entityManager->flush();
     }
   }

   public function commit()
   {
      $entityManager->flush();
   }
}

然后在我的UsersController 我可以做到：

$userService = new UserService();
$userService->setUseTransaction(true);

foreach ($users as $user) {
    $userService = new UserService();
    $user->updateUser($data);
}

$userService->commit();

你有什么想法？

Answer 1

我不想在我的服务层之上公开任何事务管理的东西。我可能会将所有这些东西都推到我的服务中，并公开两个公共方法 updateUser(userEntity) （用于一次性使用隐式刷新）和 updateUsers(array-of-users) （用于批量更新）

大概是这样的：

class UserService {

    public function updateUser(User $user){
        $this->_updateUser();
        $this->em()->flush();        
    }

    public function updateUsers(array $users){
        foreach($users as $u) $this->_updateUser($u);
        $this->em()->flush();     
    }

    private function _updateUser(User $user){
        //do stuff to $user
        $this->em()->persist($user);
    }
}

然后，如果您稍后决定要将更新分块为 100 个或其他组，那么您的所有批量更新逻辑都很好地考虑在服务中，而不是在控制器中可能存在多个位置。

Answer 2

将其全部包装在事务中肯定会加快速度。

不过，将整个批量更新编写为单个 SQL 查询会快数百倍。