异常处理的最佳实践：PDO + 控制器和存储库

Question

我搜索了有关使用 PDO 处理异常的最佳实践，但大多数示例仅从简单的一类方法来看。

如果我在 ORM 类型模型中使用控制器和 repo，那么各种 try/catch 和 throw 块应该发生在哪里？

一个简化的例子：

CONTROLLER：（创建 repo 对象，触发 loadProduct 方法，并加载模板）

class ProductController {
    public function viewProduct($product_id) {
        $ProductRepository = new ProductRepository($this->Pdo);
        $Product = $ProductRepository->loadProduct($product_id);
            
        include(__DIR__.'/../templates/product_template.php');
    }
}

模型/存储库：

class ProductRepository
{
    private $Pdo;
        
    public function __construct(PDO $Pdo)
    {
        $this->Pdo = $Pdo;
    }
            
    public function loadProduct($product_id,$withimages=0)
    {
        $Stm = $this->Pdo->prepare('
            SELECT p.product_id,p.model,p.price,p.prodinfo,pi.image_path
            FROM products p LEFT JOIN product_images pi
            ON p.product_id = pi.product_id
            WHERE p.product_id = :product_id
            AND pi.is_primary = 1
        ');
        $Stm->bindParam(':product_id',$product_id,PDO::PARAM_INT);
        $Stm->execute();
    
        return $this->arrayToObject($Stm->fetch(PDO::FETCH_ASSOC));    
    }
}

是否应该将 try/catch 块放入控制器中，如果 execute() 没有返回任何内容，则抛出异常？如果$Pdo->prepare() 方法没有触发，则抛出一个单独的异常？

Answer 1

一般来说，异常处理策略非常简单。特别是对于 PDO。因为 PDO 仅在出现严重故障的情况下才会抛出一个，并且继续执行几乎没有意义 - 所以，只需默认停止即可。

因此，对于代码的平均部分，无论是模型、存储库还是其他任何东西，都不需要专门的处理。

只有在您有查询失败场景的某些地方，才必须使用 try-catch。最常用的场景是事务回滚。因此，如果您有事务，您可能希望将其包装在 try 中，然后在 catch. 中回滚

回答 cmets 中的澄清：

这是两个本质上不同的场景：

• 如果 SELECT 找不到产品，没有什么异常。
• 虽然 INSERT 以某种方式创建了无效的 SQL 查询，但这确实是一场灾难。

他们需要不同的处理方式。

对于第一种情况，您根本不需要例外，这是常规行为。只需在您的模板中添加一个显示“未找到任何内容”的分支

对于第二个，创建一个自定义异常处理程序，记录错误、发送 503 并显示通用 503 错误页面。