我应该在 Laravel MVC 中的哪里保存模型？

Question

我试图更具体地了解 MVC 并尽可能保持控制器层的薄。

我一直问自己的一件事是“我应该在哪里调用 modelname->save()？”

查看 Laravel 文档，他们将数据设置到模型并在控制器中调用 save 似乎不正确...

<?php

namespace App\Http\Controllers;

use App\Flight;
use Illuminate\Http\Request;
use App\Http\Controllers\Controller;

class FlightController extends Controller
{
    public function store(Request $request)
    {
        // Validate the request...

        $flight = new Flight;

        $flight->name = $request->name;

        $flight->save();
    }
}

这是一个非常简单的示例，这可能就是他们在控制器中执行所有操作的原因。

根据我的理解和我一直在阅读的所有内容，所有业务逻辑都应该位于模型内部，控制器负责视图和模型之间的“流量控制”。

那么我会在模型本身内部调用 save 吗？还是我应该使用服务层？

---

这是我当前的示例数据问题。

我正在更新模型的状态。该行已存在于数据库中。我使用 PATCH /route/ 来访问控制器方法。从那里我得到模型。

class TimecardController extends Controller {
...
    public function markAsPass(Request $request, $id) {
        $test = Test::findOrFail($id);

        //I don't think this is the corect way  
        //$test->status = "passed";
        //$test->markedBy = "Teacher123";
        //$test->save();


        $test->passed();
        ...
        return redirect($redirect_url);
    }
}

class Test extends Model {
...
    public function passed() {
        $this->status = "passed";

        //would I call save here? 
        //$this->save();
    }
}

我是否采用上述方法？还是创建一个服务层，在其中使用模型实例调用模型函数，然后在模型上调用 save？

//in service class
public function makeTestAsPassed($test){
    $test->passed();
    $test->save();

}

如果需要任何说明，请告诉我。

Answer 1

您说得对，业务逻辑属于模型。如果您对应用程序采取“资源丰富”的方法（即围绕实体创建控制器），那么您会发现您的控制器操作很少调用多个模型方法。

您可以在模型上调用create() 和update() 方法，而不是调用save()。在您的 store() 控制器操作中，您可以使用这样的一行创建一个新实体：

public function store(CreateRequest $request)
{
    $model = Model::create($request->all());
}

并在 update() 操作中更新现有模型，如下所示：

public function update(UpdateRequest $request, Model $model)
{
    $model->update($request->all());
}

当涉及到业务逻辑时，您也可以在模型上调用 other 方法。要使用资源丰富的控制器，您不必必须拥有与数据库表相关的模型。

接受运送订单。大多数人会很想将ship() 方法放在OrderController 中，但是当您发送订单时会发生什么？运送订单会导致什么实体？好吧，您将创建一个 shipment，因此它可以是 OrderShipmentController 上的 store() 方法。这个store() 方法可以在你的Order 模型上调用ship() 方法：

class OrderShipmentController extends Controller
{
    public function store(ShipOrderRequest $request, Order $order)
    {
        $order->ship();
    }
}

如您所见，使用资源丰富的控制器和route–model binding，您可以拥有“瘦控制器”，并将应用程序的业务逻辑置于模型中。

Answer 2

MVC 旨在便于维护。 您不认为“好”的方法是正确的方法。所有与业务逻辑相关的数据处理都在控制器中。否则，其他编码员会因为在控制器代码中找不到数据操作逻辑而感到困惑。

您的瘦控制器目标打败了 MVC。

另请注意，模型代码旨在精简，因为它是将数据库架构定义为数据库表的镜像的地方。

MVC 不是面向对象的抽象。 MVC 是一种代码维护一致性的结构。