Laravel コンストラクタインジェクションについて

Question

以下のようなことを実現したいのですが可能でしょうか？
そもそもコンストラクタインジェクションの使い方がおかしい？

親クラスのコンストラクタ

protected $hoge;
public function __construct(Hoge $hoge)
{
    $this->hoge = $hoge;
    $this->hoge->fuga = 'fuga'; // デフォルト値
}

上記の親クラスを継承した子クラスのコンストラクタ。

public function __construct()
{
    parent::__construct();
    $this->hoge->fuga = 'piyo'; // 子クラスでデフォルト値を上書きする。
}

上記子クラスのコードは、エラーになります。
parent::__construct();の引数にHogeのインスタンスを渡していないからです。

そのため、子クラスのコンストラクタを以下のようにする必要があります。

public function __construct(Hoge $hoge)
{
    parent::__construct($hoge);
    $this->hoge->fuga = 'piyo'; // 子クラスでfugaを上書きする。
}

もしくは、親クラスのコンストラクタを以下にするのも解決法かとは思いますが。。

protected $hoge;
public function __construct()
{
    $this->hoge = new Hoge();
    $this->hoge->fuga = 'fuga'; // デフォルト値
}

親クラスでnewを使うのも微妙ですし、
わざわざ子クラスから渡すのも微妙ですし。。
もっと良い書き方があれば知りたいです。

そもそもコンストラクタインジェクションの使い方がおかしいのでしょうか？
ご教示頂けると有難いです。

Answer 1

コンストラクタインジェクションの使い方

どう使うのが正しいとか、難しいですよね・・・
そこそこlaravelを使っていますが、色々な部分でいまだに「これでいいのかな？もっといい方法があるのでは？」と思うコトが多々あります。

というわけで、「大正解」というものはちょっとわからないので、以下は個人的な見解としてお聞きください。。。

---

質問のようなことを実際によくしています。
なので可能かどうかについては、「可能」と回答できます。

ただ、DIは「単体テスト」に大きくメリットのあるテクニックですよね。
親クラスの中で単純にnewしてインスタンスを作った場合、親クラスの単体テストが出来なくなるというデメリットになりますので、newする場合は下記のようにすると良いのではないかと思います。

親クラス

protected $hoge;
public function __construct(Hoge $hoge = null)
{
    $this->hoge = $hoge ? $hoge : new Hoge();
    $this->hoge->fuga = 'fuga'; // デフォルト値
}

実際に私が実装してきたものの場合、
元々「親」にする予定のなかったクラスを、後の機能追加などで似た機能があるため「親」として継承させる、というようなケースが多かったので、親側の修正をしなくて済むよう「子クラスから渡す」方法をとることがほとんどでした。

なので実装のタイミング次第で、
・親クラスでnew
・子クラスから渡す
のどちらにするかは、やりやすい方を選択すればいいのではないかな～　と思っています。

Comments

[masa80]

>親クラスの中で単純にnewしてインスタンスを作った場合、親クラスの単体テストが出来なくなるというデメリットになりますので、newする場合は下記のようにすると良いのではないかと思います。

なるほど。
ありがとうございます！

Answer 2

1案として下記の実装は如何でしょうか？セッターメソッドを作る感じです。
子クラスでコンストラクタインジェクションをしようとすると使えませんが…

(親クラス)

<?php

namespace App\Http\Controllers;

use Illuminate\Http\Request;
use League\Flysystem\Adapter\AbstractAdapter;
use App\Test;

Abstract class AbstractTestController extends Controller
{
    protected $testClass;
    public function __construct(Test $testClass)
    {
        $this->testClass = $testClass;
    }
    /**
     * このメソッドでプロパティをオーバーライド可能にする
     *
     * @param string $value
     * @return void
     */
    protected function overrideProperty(string $value)
    {
        $this->testClass->property = $value;
    }
    /**
     * 設定されたプロパティをechoする。
     *
     * @return void
     */
    public function echoValue()
    {
        echo $this->testClass->property."<br>";
    }
}

子クラス

<?php

namespace App\Http\Controllers;

use Illuminate\Http\Request;

class ExtendTestContoller extends AbstractTestController
{
    public function index()
    {
        $this->echoValue();//ここはデフォルト値を表示
        $this->overrideProperty('テスト');
        $this->echoValue();//ここはオーバーライドされた値
        return view('welcome');
    }
}

結果