Laravel中按数量计算总价：逻辑应放控制器还是模型？

这个问题看似基础，但其实是理解MVC职责划分的好机会！答案是：把总价计算逻辑放在模型里，原因和具体实现如下：

为什么选模型而不是控制器？ #

MVC的核心思想是职责分离：

• 控制器只负责接收请求参数、调用业务逻辑、返回响应，它不应该处理和数据实体相关的业务计算；
• 模型是对应数据库表的实体类，负责处理自身的属性逻辑、数据验证和业务规则，总价（total）是Item实体的衍生属性，自然应该由模型来维护。

这样做的好处很明显：

• 复用性强：以后不管在哪个控制器（甚至命令行、队列任务）里创建或更新Item，都能自动计算总价，不用重复写计算代码；
• 维护方便：如果以后总价计算规则变了（比如要加税费、折扣），只需要修改模型里的代码，不用改动所有相关的控制器；
• 避免人为失误：不用每次都手动在控制器里计算total，减少遗漏或计算错误的可能。

具体实现方式 #

推荐用模型的事件监听来自动计算总价，这样在创建或更新Item时会自动触发计算：

1. 修改Item模型

在模型的boot方法里添加creating和updating事件监听：

class Item extends Model
{
    protected static function boot()
    {
        parent::boot();

        // 创建Item时自动计算总价
        static::creating(function ($item) {
            $item->total = $item->quantity * $item->price;
        });

        // 更新Item时，如果quantity或price有变化，重新计算总价
        static::updating(function ($item) {
            if ($item->isDirty('quantity') || $item->isDirty('price')) {
                $item->total = $item->quantity * $item->price;
            }
        });
    }
}

2. 控制器代码保持简洁

你的控制器代码不需要做任何修改（甚至可以更简洁），只需要传递请求参数即可：

Item::create([
    'quantity' => $request->input('quantity'),
    'price' => $request->input('price'),
]);

补充：另一种可选方案（访问器） #

如果你不需要把total持久化到数据库（也就是可以动态计算），可以用Laravel的访问器来实现，这样连数据库里的total字段都可以删掉：

class Item extends Model
{
    // 定义访问器，动态计算total
    public function getTotalAttribute()
    {
        return $this->quantity * $this->price;
    }
}

使用的时候直接调用$item->total就能拿到计算结果，不需要存储到数据库里。不过根据你的数据库结构，你已经设置了total字段，所以第一种事件监听的方案更适合你。

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