我已经使用 Laravel 一段时间了，并且阅读了很多关于依赖注入(inject)和可测试代码的文章。在谈论 Facades 和 Mocked Objects 时，我感到很困惑。我看到两种模式:

class Post extends Eloquent {

    protected $guarded = array();

    public static $rules = array();

}

这是我的帖子模型。我可以运行 Post::all(); 来获取我博客中的所有帖子。现在我想将它合并到我的 Controller 中。

选项 #1:依赖注入(inject)

我的第一直觉是将 Post 模型作为依赖注入(inject):

class HomeController extends BaseController {

    public function __construct(Post $post)
    {
    $this->post = $post;
    }

    public function index()
    {
        $posts = $this->posts->all();
        return View::make( 'posts' , compact( $posts );
    }

}

我的单元测试看起来像这样:

<?php 

use \Mockery;

class HomeControllerTest extends TestCase {

    public function tearDown()
    {
        Mockery::close();

        parent::tearDown();
    }
    public function testIndex()
    {
        $post_collection = new StdClass();

        $post = Mockery::mock('Eloquent', 'Post')
        ->shouldRecieve('all')
        ->once()
        ->andReturn($post_collection);

        $this->app->instance('Post',$post);

        $this->client->request('GET', 'posts');

        $this->assertViewHas('posts');
    }
}

选项 #2:外观模拟

class HomeController extends BaseController {


    public function index()
    {
        $posts = Post::all();
        return View::make( 'posts' , compact( $posts );            
    }

}

我的单元测试看起来像这样:

<?php 

use \Mockery;

class HomeControllerTest extends TestCase {


    public function testIndex()
    {
        $post_collection = new StdClass();

        Post::shouldRecieve('all')
        ->once()
        ->andReturn($post_collection);

        $this->client->request('GET', 'posts');

        $this->assertViewHas('posts');
    }
}

我理解这两种方法，但我不明白为什么我应该或何时应该使用一种方法而不是另一种方法。例如，我尝试将 DI 路由与 Auth 类一起使用，但它不起作用，所以我必须使用 Facade Mocks。在这个问题上的任何钙化将不胜感激。

最佳答案

虽然您在选项 #1 上使用依赖注入(inject)，但您的 Controller 仍然与 Eloquent ORM 耦合。 (请注意，我在这里避免使用术语模型，因为在 MVC 中，模型不仅仅是一个类或一个对象，而是一个层。它是您的业务逻辑。)。

依赖注入(inject)允许依赖倒置，但它们不是一回事。根据依赖倒置原则，高层和低层代码都应该依赖于抽象。在您的情况下，高级代码是您的 Controller ，低级代码是从 MySQL 获取数据的 Eloquent ORM，但正如您所见，它们都不依赖于抽象。

因此，您无法在不影响 Controller 的情况下更改数据访问层。例如，您将如何从 MySQL 更改为 MongoDB 或文件系统？为此，您必须使用存储库(或任何您想调用的名称)。

因此，创建一个存储库接口(interface)，您的所有具体存储库实现(MySQL、MongoDB、文件系统等)都应该实现。

interface PostRepositoriesInterface {

    public function getAll();
}

然后创建您的具体实现，例如对于 MySQL

class DbPostRepository implements PostRepositoriesInterface {

    public function getAll()
    {

        return Post::all()->toArray();

        /* Why toArray()? This is the L (Liskov Substitution) in SOLID. 
           Any implementation of an abstraction (interface) should be substitutable
           in any place that the abstraction is accepted. But if you just return 
           Post:all() how would you handle the situation where another concrete 
           implementation would return another data type? Probably you would use an if
           statement in the controller to determine the data type but that's far from 
           ideal. In PHP you cannot force the return data type so this is something
           that you have to keep in mind.*/
    }
}

现在你的 Controller 必须类型提示接口(interface)而不是具体实现。这就是“接口(interface)上的代码而不是实现上的代码”的全部内容。这就是依赖倒置。

class HomeController extends BaseController {

    public function __construct(PostRepositoriesInterface $repo)
    {
        $this->repo= $repo;
    }

    public function index()
    {
        $posts = $this->repo->getAll();

        return View::make( 'posts' , compact( $posts ) );
    }

}

这样你的 Controller 就与你的数据层分离了。它对扩展开放但对修改关闭。您可以通过创建 PostRepositoriesInterface(例如 MongoPostRepository)的新具体实现来切换到 MongoDB 或文件系统，并仅更改绑定(bind)(请注意，我在这里不使用任何 namespace ):

App:bind('PostRepositoriesInterface','DbPostRepository');

到

App:bind('PostRepositoriesInterface','MongoPostRepository');

在理想情况下，您的 Controller 应该只包含应用程序而不是业务逻辑。如果您发现自己想从另一个 Controller 调用一个 Controller ，则表明您做错了什么。在这种情况下，您的 Controller 包含太多逻辑。

这也使测试更容易。现在您可以在不实际访问数据库的情况下测试您的 Controller 。请注意， Controller 测试必须仅在 Controller 正常运行时进行测试，这意味着 Controller 调用正确的方法，获取结果并将其传递给 View 。此时您还没有测试结果的有效性。这不是控制者的责任。

public function testIndexActionBindsPostsFromRepository()
{ 

    $repository = Mockery::mock('PostRepositoriesInterface');

    $repository->shouldReceive('all')->once()->andReturn(array('foo'));

    App::instance('PostRepositoriesInterface', $repository);

    $response = $this->action('GET', 'HomeController@index'); 

    $this->assertResponseOk(); 

    $this->assertViewHas('posts', array('foo')); 
}

编辑

如果您选择使用选项 #1，您可以像这样测试它

class HomeControllerTest extends TestCase {

  public function __construct()
  {
      $this->mock = Mockery::mock('Eloquent', 'Post');
  }

  public function tearDown()
  {
      Mockery::close();
  }

  public function testIndex()
  {
      $this->mock
           ->shouldReceive('all')
           ->once()
           ->andReturn('foo');

      $this->app->instance('Post', $this->mock);

      $this->call('GET', 'posts');

      $this->assertViewHas('posts');
  }

}

关于php - Laravel 依赖注入(inject) : When do you have to? 你什么时候可以模拟门面？两种方法的优点？，我们在Stack Overflow上找到一个类似的问题： https://stackoverflow.com/questions/22031972/