Laravel 4 - 在其他控制器中使用控制器动作？

Question

好吧，我有几个控制器在做我的工作，现在我创建了更多控制器；在新控制器中的某个操作中间，我只记得我已经在旧控制器中编写了正确的代码。

所以例如在 oldController@handyAction

some_good_codez();

在 newController@newAction 中

$myOldController = Load::controller('oldController'); // I wish Laravel had this?
$myOldController->handyAction();

Answer 1

这是错误的。控制器必须只负责一项职责：接收 http 请求，从模型中获取一些数据并将其传递给视图。如果您的控制器必须与另一个控制器“对话”，这可能是因为您正在处理它们上的数据，或者通过它们读取数据，而您不应该这样做，因为处理数据是模型而不是控制器的责任。

首先，看看SOLID 和Single responsibility principle。如果你能买到Taylor Otwell's book: Laravel: From Apprentice To Artisan，就去做吧，它会让你很好地了解如何在 Laravel 中实现 SOLID 原理。

然后，如果您真的需要在两个不同的控制器上使用一个方法，请创建一个新类并在两个控制器上使用这个类。只需重构您的代码以执行以下操作：

class MyController1 extends Controller {

    public function __construct(MyClass $class)   
    {
        $this->myClass = $class;
    }

    public function store()
    {
        $this->myClass->doWhatever();
    }

}

class MyController2 extends Controller {

    public function __construct(MyClass $class)   
    {
        $this->myClass = $class;
    }

    public function update()
    {
        $this->myClass->doWhatever();
    }

}

class MyClass {

    public function doWhatever()
    {
        return 'done';
    }

}

这仍然是错误的，但要好一些。

更好的是在数据存储库中处理您的数据，因此请查看存储库模式，this Taylor's video 会让您对如何操作有一个很好的了解。它谈论可测试性，但不仅如此，您可以创建域存储库并在其中混合两个或多个模型，并在一个类中处理来自这些模型的数据。使用 Repository 模式的代码看起来更像：

创建接口（合约）：

interface MyDataRepository {

    public function getInfoA();
    public function getInfoB($dataA);

}

创建一个实现该接口的类

class MyData implements MyDataRepository {

    public function __construct(MyModelA $dataA, MyModelB $dataB)   
    {
        $this->dataA = $dataA;
        $this->dataB = $dataB;
    }

    public function getInfoA()
    {
        return $this->dataA->processData();
    }

    public function getInfoB($dataA)
    {
        return $this->dataB->processData( $this->getInfoA() );
    }

}

[编辑]

将存储库想象成一个袋子，您可以在其中放置所需的所有内容并从中提取所需的内容。

在这个存储库中，我们使用了 2 个模型（MyModelA 和 MyModelB），由 Laravel 的 IoC 容器实例化，在方法 getInfoB 中，存储库使用这两个模型来生成数据并将其传递回控制器。

这一切都来自领域驱动开发。域是一个复杂的模型，例如订单，它是使用来自订单、用户、商品、运输和付款表的数据构建的。如果没有所有这些信息，您的付款页面上就不会真正显示订单，因此您创建了一个订单存储库并将所有 ORM 混合在一个类中，它具有从这些表中获取数据并将其发送回的单一责任你的控制器。

[/edit]

当它需要你的接口实例时，你必须告诉 Laravel 实例化该类：

App::bind('MyDataRepository', 'MyData ');

只有在你使用接口的时候才会这样做，这里我们使用一个具体的类来做同样的事情，我们不需要通知任何东西，Laravel 知道该怎么做：

public function __construct(MyClass $class)   
{
    $this->myClass = $class;
}

并使用存储库创建您的控制器：

    class MyController1 extends Controller {

    public function __construct(MyDataRepository $data)   
    {
        $this->data = $data;
    }

    public function store()
    {
        $this->data->getInfoA();
    }

}

class MyController2 extends Controller {

    public function __construct(MyDataRepository $class)   
    {
        $this->data = $data;
    }

    public function update()
    {
        $this->myClass->getInfoB();
    }

}

Laravel 会自动实例化你的类的一个实例，你不需要做任何其他事情来让它工作。