这个mysql触发器有什么问题吗？

Question

我有一个产品、 sales_order 和 sales_order_product 表。

产品的性质决定了价格和重量的频繁变化。

客户通常会将订单保存几天，然后再提交并付款。

由于这种滞后和价格/重量波动，我们允许他们冻结原始订单时的价格/重量。

我们在 sales_order 表中保存订单金额（小计）、销售税、订单重量以及其他一些内容。

我知道不建议在数据库层内拥有任何业务逻辑，尽管我认为这更多是维护引用完整性的一种手段。

以下是我用来计算上述数据的触发器之一。我才开始测试它，到目前为止还不错。就性能而言，到目前为止我还没有看到任何问题，但我的测试还不是很广泛。

这个触发器有什么看起来不对劲的地方吗？我问这个问题是因为虽然我已经使用过诸如时间戳之类的触发器，但我从未真正以这种身份使用过它们（考虑到我们正在谈论金钱，我不想搞砸一些可能会让我丢掉工作的事情）。

我意识到对税率进行硬编码可能不是一个好主意，到时候我可能会改变它。

CREATE TRIGGER after_sales_order_product_update
AFTER UPDATE ON sales_order_product
FOR EACH ROW
BEGIN

    SET @product_amount = (SELECT SUM(quantity * unit_price) 
                           FROM sales_order_product 
                           WHERE sales_order_id = OLD.sales_order_id),
        @product_discount = (SELECT SUM(quantity * unit_discount) 
                             FROM sales_order_product 
                             WHERE sales_order_id = OLD.sales_order_id),
        @total_weight = (SELECT SUM(quantity * unit_weight) 
                         FROM sales_order_product 
                         WHERE sales_order_id = OLD.sales_order_id),
        @tax_amount = ROUND(@product_amount * 0.0975,2);

    UPDATE sales_order 
    SET product_amount = @product_amount,
        product_discount = @product_discount,
        tax_amount = @tax_amount,
        total_weight = @total_weight,
        product_total = product_amount - product_discount
    WHERE sales_order_id = OLD.sales_order_id;

END

Answer 1

你的设计看起来不错；只是一些想法：

1. 您的触发器仅在 UPDATE 之后触发。当一行首先被插入，或者稍后被删除时会怎么样？
2. 那么 NULL 又如何呢？如果任何核心字段（quantity、unit_price、unit_discount、unit_weight）可能为 null，则将给出您在派生字段中设置了您可能不期望的空值。
3. 没有行（又是NULL）。假设客户从购物车中删除了一行，sales_order 是否可以没有 sales_order_products？在这种情况下，您可能希望 sales_order 值为零，这可能需要特殊编码。
4. 订单状态。订单通过后会发生什么？销售完成后，无论发生什么情况（税率变化等），这些值都应该是固定的，因此您可能需要触发器在更新任何内容之前检查订单状态。

您可能可以通过进行一些故事板/基于场景的测试来解决所有这些问题。