構造函數注入 vs 方法注入

Question

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。 它的一些功能是：

• index - 顯示所有數據
• 編輯 - 顯示編輯表單
• 更新 - 更新數據庫

編輯和更新使用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 的使用讓代碼更加惡心。

我感到很困惑。 我明白我可以從依賴注入中獲得的好處，但我確信我的方法注入和構造函數注入的實現是非常錯誤的。 我應該選擇什么實現？ 或者我應該從這兩個中選擇一個嗎？

Answer 1

如果在大多數方法中都使用了模型，請使用構造函數注入。 並且$model 和$this 的使用沒有任何問題。 但是，如果您仍想清理代碼，請考慮存儲庫模式 (SRP)。 您可以在那里管理很長的代碼行。 請參閱此stackoverflow 答案- 如何在 Laravel 5 項目中組織類？ . 我希望這會幫助你的困惑。

Answer 2

我絕對更喜歡 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()
);