コントローラをインジェクションできない

Question

コンストラクタに他のコントローラをインジェクションしたいのですが、うまくいきません。下記ソースです。

class HellosController extends AppController
{
  
  protected $name = Hello;
  protected $id;
  protected $list;
  protected $fbrl;


  public function __construct(FacadeBookResearchLogicController $fbrl = null){
      
    $this->fbrl = $fbrl ? $fbrl : new FacadeBookResearchLogicController();
  }

  public function index($id){
    $this->id = $id;

    $this->list = $this->fbrl->customerSearch($this->id,0);
   }
}

上記ソースを実行すると、

Error: Call to a member function init() on null

File: /var/www/html/cake/lib/Cake/Controller/Controller.php
Line: 643

となってしまいます。原因が分かる方教えていただけないでしょうか。

Answer 1

この記述だと、親クラスのコンストラクタをオーバーライドしてしまうため、親クラスでの初期化処理が実行されなくなるのが原因だと思います。
以下のように、親クラスのコンストラクタを呼び出す処理を追加してはいかがでしょうか？

	public function __construct(FacadeBookResearchLogicController $fbrl = null){

		$this->fbrl = $fbrl ? $fbrl : new FacadeBookResearchLogicController();
		parent::__construct();  // この処理を追加
	}

前提を覆すようで申し訳ないですが、そもそもコンストラクタでFacadeBookResearchLogicControllerを作成している特別な理由がありますか？
もし、コントローラから別のコントローラのactionを呼びたいのであれば、以下のような解決策もあると思います。

<?php
App::uses('FacadeBookResearchLogicController', 'Controller');

class HellosController extends AppController
{

	public $name = Hellos;
	public $id;
	public $list;
	public $fbrl;

	public function index($id){
		$fbrl = new FacadeBookResearchLogicController();
		$this->list = $fbrl->customerSearch($this->id,0);
	}
}

追記：
controllerの__constructは、$requestと$responseのパラメータが必要で、これはHellosControllerの__constructが呼び出される際にdispatcherから渡されるので、そのまま親クラスまで引き渡す必要があるようです。
以下のようなコードではいかがでしょうか？

	public function __construct($request = null, $response = null){
		$this->fbrl = new FacadeBookResearchLogicController();
		parent::__construct($request, $response);
	}

Comments

[tixure55]

回答ありがとうございます、parent::__construct();を追記しましたが、Method customerSearch does not exist

Error: An Internal Error Has Occurred.となってしまいます。

AppControllerを継承していないコントローラでかつ注入するクラスがコントローラでない場合は実装できたのですが、CakePHPでは無理なんですかね・・

タイプヒンティングの問題かなと思って、
public function __construct(){

$this->fbrl = $fbrl ? $fbrl : new FacadeBookResearchLogicController();
parent::__construct(); // この処理を追加
}
も試したのですが、Error: Call to a member function body() on null
File: /var/www/html/cake/lib/Cake/Controller/Controller.php
Line: 960となってしまいます。

[KatsumiTanaka]

なるほど、そうですか・・・
コンストラクタ内で別のコントローラのインスタンスを作成するという前提を覆すようで申し訳ないのですが、別のコントローラのactionを呼び出したいのであれば、追記したように利用する関数内でインスタンスを作成する方法もあります。
インスタンス内での処理についての、対応案も追記しました

[tixure55]

public function __construct($request = null, $response = null){
$this->fbrl = new FacadeBookResearchLogicController();
parent::__construct($request,$response);
}
とすることで、エラーが消えました。ありがとうございました。