[英]constructor injection vs method injection
Laravel 鼓励依赖注入。 由于我在我的项目中使用 laravel,我想我会尝试使用这种方法。
我通过键入提示我的依赖项并让它解决它们来利用 Laravel 的服务容器。 我有四个控制器。 它们都扩展了一个名为 GlobalController 的基类。 我也有两个模型。 它们都扩展了一个名为 GlobalModel 的基类。
我的第一次尝试是使用方法注入。 全局控制器看起来像这样:
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。 它的一些功能是:
编辑和更新使用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 的使用让代码更加恶心。
我感到很困惑。 我明白我可以从依赖注入中获得的好处,但我确信我的方法注入和构造函数注入的实现是非常错误的。 我应该选择什么实现? 或者我应该从这两个中选择一个吗?
如果在大多数方法中都使用了模型,请使用构造函数注入。 并且$model 和$this 的使用没有任何问题。 但是,如果您仍想清理代码,请考虑存储库模式 (SRP)。 您可以在那里管理很长的代码行。 请参阅此stackoverflow 答案- 如何在 Laravel 5 项目中组织类? . 我希望这会帮助你的困惑。
我绝对更喜欢 Laravel 控制器的第一种方法。 起初,您不需要在每种方法中都使用注入模型。 (为什么要在索引函数中注入用户模型?)。
其次,您不能再使用 RouteModelBinding 的好处,必须手动检查具有给定 $id 的模型是否确实存在并采取相应措施。 您也不能使用像 CreateUserRequest 这样可以处理验证和授权的特定 FormRequest。 (虽然这是一个可选功能)
另请注意,在构造函数中注入的模型绝不是具有用户数据的“真实”模型。 所以这只会让你访问 eleoquent 函数。 所以你也可以在你的代码中使用 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()
);
声明:本站的技术帖子网页,遵循CC BY-SA 4.0协议,如果您需要转载,请注明本站网址或者原文地址。任何问题请咨询:yoyou2525@163.com.