保存实体时捕获Doctrine各类异常是否属于过度设计？

先看你贴的这段代码：

// Save entity
try {
    $em->persist($someEntity);
    $em->flush();
    $this->logger->info("Saved someEntity");
} catch (DBALException $e) {
    $this->logger->addError("DBALException [{code}]: {message}", array('code' => $e->getCode(), $e->getMessage()));
} catch (PDOException $e) {
    $this->logger->addError("PDOException [{code}]: {message}", array('code' => $e->getCode(), $e->getMessage()));
} catch (ORMException $e) {
    $this->logger->addError("ORMException [{code}]: {message}", array('code' => $e->getCode(), $e->getMessage()));
} catch (Exception $e) {
    $this->logger->addError("Exception [{code}]: {message}", array('code' => $e->getCode(), $e->getMessage()));
}

你提到自己习惯只用最后的Exception捕获，疑惑这种多类型捕获是不是过度设计——其实这个问题没有绝对答案，核心要看你的实际需求和未来的维护计划：

什么时候这种写法是实用的？ #

• 需要差异化处理逻辑时：这是最核心的场景。比如：
  • 遇到DBALException（比如死锁、临时连接失败）时，你可能想加个重试机制；
  • 遇到ORMException（比如实体映射错误、关联关系配置问题）时，可能需要直接返回明确的错误提示给用户，甚至触发告警通知开发团队；
  • 而通用Exception可能对应其他不可预见的错误，需要记录更详细的上下文信息。
    这种情况下，分开捕获不同异常类型完全合理，能让代码逻辑更清晰，处理更精准。
• 日志排查需求：如果你的团队需要通过日志快速定位问题，区分异常类型会很有帮助。运维或开发看到DBALException就知道是数据库层面的问题，看到ORMException就去查实体映射，比只看模糊的Exception要高效得多。

什么时候属于过度设计？ #

如果你的处理逻辑完全一致（比如像代码里那样只是统一格式打日志），而且确定未来也不会有差异化处理的需求，那这种写法确实有点冗余。你可以简化成捕获更上层的异常类，比如Doctrine的DoctrineException（它是DBALException和ORMException的父类），再加上PDOException和通用Exception，既保留了类型区分，又减少重复代码：

try {
    $em->persist($someEntity);
    $em->flush();
    $this->logger->info("Saved someEntity");
} catch (\Doctrine\Common\Exception\DoctrineException $e) {
    $this->logger->addError("{exception_type} [{code}]: {message}", [
        'exception_type' => get_class($e),
        'code' => $e->getCode(),
        'message' => $e->getMessage()
    ]);
} catch (\PDOException $e) {
    $this->logger->addError("PDOException [{code}]: {message}", [
        'code' => $e->getCode(),
        'message' => $e->getMessage()
    ]);
} catch (\Exception $e) {
    $this->logger->addError("Exception [{code}]: {message}", [
        'code' => $e->getCode(),
        'message' => $e->getMessage()
    ]);
}

如果连日志区分类型都不需要，那直接捕获Exception打日志就足够了，代码更简洁。

总结 #

总的来说，多类型异常捕获本身不是过度设计，关键看你是否有对应的需求。如果只是为了“看起来专业”而盲目拆分，那就是过度设计；但如果是为了精准处理问题、提升排查效率，那就是很实用的写法。

内容的提问来源于stack exchange，提问作者Stephane Gosselin