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

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（用户数组）（用于批量更新）

大致类似于：

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查询编写将快数百倍。