Symfony Doctrine循环事务问题：单个命令失败致全部未提交

核心问题 #

1. 缩进错误导致事务逻辑脱离循环
   原代码foreach后未添加大括号，仅第一行变量赋值在循环范围内，所有命令共用同一个事务，完全违背了“每个命令独立事务”的设计意图。

2. 非成功场景未处理事务回滚
   当checkCommande返回空或结果不为OK时，事务始终处于开启状态，未提交也未回滚，导致后续事务无法正常执行，出现阻塞。

3. 直接操作Connection未适配Doctrine机制
   Doctrine的EntityManager与数据库Connection状态强关联，直接操作Connection的事务可能导致ORM状态与数据库不一致；且循环中未清除EntityManager的实体缓存，易引发内存溢出或实体状态混乱。

修正后的代码 #

foreach($commandes as $commande) {
    $em = $this->em;
    $em->beginTransaction(); // 用EntityManager管理事务，贴合Doctrine设计

    try {
        $tmp = $this->checkCommande($commande);

        if ($tmp && 'OK' === $tmp['result']) {
            $commande->setCompleted(true);
            $em->flush();
            $em->commit();
        } else {
            // 不满足条件时主动回滚，避免事务阻塞
            $em->rollback();
        }
        
        // 清除实体缓存，避免循环中堆积实体导致内存问题或状态混乱
        $em->clear();
    } catch (\Exception $e) {
        $em->rollback();
        throw $e;
    }
}

关键修改说明 #

• 给foreach添加大括号，确保每个命令的事务逻辑独立运行在循环内。
• 改用EntityManager的事务方法（beginTransaction/commit/rollback），保证ORM与数据库连接状态一致。
• 新增非成功场景的事务回滚逻辑，避免事务长时间开启阻塞后续操作。
• 循环结束后调用$em->clear()，清除EntityManager中的实体缓存，防止内存溢出和实体状态异常。

内容的提问来源于stack exchange，提问作者Marwen ben alaya