构造函数注入与方法注入

病态码

Laravel鼓励依赖注入。由于我在项目中使用laravel,因此我认为我将尝试使用这种方法。

我通过类型化提示我的依赖关系并让其解决它们来利用Laravel的服务容器我有四个控制器。它们都扩展了一个名为GlobalController的基类。我也有两个模型。它们都扩展了一个名为GlobalModel的基类。

我的第一次尝试是使用方法注入。GlobalController看起来像这样:

namespace App\Http\Controllers;

use Illuminate\Http\Request;
use App\Models\GlobalModel;

class GlobalController extends Controller
{

    public function __construct()
    {
        $this->middleware(['authenticate', 'token']);
    }

    // functions that handle normal http requests and ajax requests

    }

从GlobalController扩展的控制器之一称为UserController。它的一些功能是:

  • 索引-显示所有数据
  • 编辑-显示编辑表单
  • 更新-数据库更新

编辑和更新use route-model-binding

namespace App\Http\Controllers;

use Illuminate\Http\Request;
use App\Models\User;

class UserController extends GlobalController
{

    public function index(User $user)
    {
        $users = $user->all();
        return view('pages/view_users')->with('users', $users);
    }

    public function edit(User $user)
    {
        return view('pages/edit_user')->with('user', $user);
    }

    public function update(Request $request, User $user)
    {
        $data = $request->all();
        if ($user->validate($data))
        {
            $user->update($data);
            return $this->successResponse($request, 'users', 'Successfully edited user');
        }
        return $this->failedResponse($request, $user);
    }

    // other functions

    }

在此过程中,请求和用户多次被注入。例如,如果必须更改Request实现,则必须手动更改许多函数以键入提示该特定Request对象的信息。一点都不好。由于通常在大多数函数中调用它们,因此我尝试进行构造函数注入。

这是使用构造函数注入的GlobalController:

namespace App\Http\Controllers;

use Illuminate\Http\Request;
use App\Models\GlobalModel;

class GlobalController extends Controller
{
    protected $request;
    protected $model; // use polymorphism

    public function __construct(Request $request, GlobalModel $model)
    {
        $this->request = $request;
        $this->model = $model;
        $this->middleware(['authenticate', 'token']);
    }

    // functions that handle normal http requests and ajax requests

}

这是UserController,它使用包含相同功能的构造函数注入:

namespace App\Http\Controllers;

use Illuminate\Http\Request;;
use App\Models\User;

class UserController extends GlobalController
{

    public function __construct(Request $request, User $user) // use polymorphism
    {
        parent::__construct($request, $user);
    }

    public function index()
    {
        $users = $this->model->all();
        return view('pages/view_users')->with('users', $users);
    }

    public function edit(int $id)
    {
        $this->model = $this->model->find($id);
        return view('pages/edit_user')->with('user', $this->model);
    }

    public function update(int $id)
    {
        $this->model = $this->model->find($id);
        $data = $this->request->all();
        if ($this->model->validate($data))
        {
            $this->model->update($data);
            return $this->successResponse('users', 'Successfully edited user');
        }
        return $this->failedResponse();
    }

    // other functions

}

现在,我不能全力以赴,但我认为此实现似乎不正确。它变得不那么可读。$ model和$ this的用法使代码更加令人恶心。

我感到很困惑。我了解依赖注入可以带来的好处,但是我确信方法注入和构造函数注入的实现是极其错误的。我应该选择哪种实现?还是我应该从这两个中选择一个?

shock_gone_wild

我绝对更喜欢Laravel控制器的第一种方法。最初,您不需要在每种方法中都使用注入模型。(为什么要在索引函数中注入用户模型?)。

其次,您不能再使用RouteModelBinding的好处,而必须手动检查具有给定$ id的模型是否确实存在,并采取相应的措施。此外,您不能使用可以处理验证和授权的特定FormRequests(例如CreateUserRequest)。(尽管这是一个可选功能)

还要注意,注入构造函数的模型永远不会是带有用户数据的“真实”模型。因此,这仅使您可以访问常用功能。因此,您也可以在代码中使用User :: find($ id)。这总是会给你假的。

public function __construct(User $user)
{
    dd($user->exists);
}

如果要抽象事物,可以在构造函数中注入存储库。

public function __construct(UserRepository $userRepository)
{
   $this->userRepository = $userRepository;
   // then the Repository is responsible for retrieving users
   // and you are not coupled to Eloquent. If you later want, you can Read
   // users from an XML File if you need
}

附加信息(有点题外话):尽管这种情况很少见,而且我不需要更改请求类,但是可以通过创建一个这样的自定义请求类来做到这一点:

namespace App;


use Illuminate\Http\Request;

class MyRequest extends Request
{
   // override request methods or add your new own methods
}

然后在全局index.php中:

    $response = $kernel->handle(
    // instead of Illuminate\Http\Request::capture()
    $request = \App\MyRequest::capture()
);

本文收集自互联网,转载请注明来源。

如有侵权,请联系 [email protected] 删除。

编辑于
0

我来说两句

0 条评论
登录 后参与评论

相关文章