PHPで別インスタンスの関数を使うときの方法が知りたい

Question

PHPでMVCの勉強をしており、以下に示すようなコードがあります。
AppクラスでRequestクラスとControllerクラスのインスタンスを作成し、call()で別のインスタンスから、他のインスタンスの関数を使いたい(今回はContollerからRequest)ときに
2つの方法を考えています。

方法①
・Appクラス内で自身の静的クラス$appを作って他のインスタンスを使いたいときはこの$appから参照する
方法②
・他のインスタンスを引数として渡す方法

このふたつの方法は以下のメリットがあると思っているのですが、他にメリットやデメリットはありますでしょうか？
もしくは使い分ける場面はありますでしょうか？

方法①のメリット
・いちいち引数を渡さなくてもよい
方法②のメリット
・用途の範囲を限定することができる

よろしくお願い致します。

<?php
// Your code here!
class App
{
    public static $app;
    public Request $request;
    public Controller $controller;
    
    public function __construct()
    {
        self::$app = $this;
        $this->request = new Request();
        $this->controller = new Controller();
    }
    
    public function call()
    {
        $this->controller->call_hello1();  //①の場合
        $this->controller->call_hello2($this->request);  //②の場合
    }
}

class Request
{
    public function hello()
    {
        echo "hello";
    }    
}

class Controller
{
    public function call_hello1()  //①の場合
    {
        App::$app->request->hello();        
    }
    public function call_hello2(Request $request)  //②の場合
    {
        $request->hello();        
    }    

}

$app = new App();
$app->call();

Comments

[退会済みユーザー]

Controllerクラスの下で下働きするクラスなら作るけど、別のControllerクラスを作る意味がピンと来ないのは私だけ？ Controllerクラスに持たせたメソッドをよそで使いたいということであれば、その処理はControllerクラスに持たせるんじゃなくて、汎用処理用のクラスに持たせてそれぞれのControllerクラスから利用するべきなんじゃないの？ Laravelのベストプラクティスを貼っておこう。https://github.com/alexeymezenin/laravel-best-practices/blob/master/japanese.md

[shimotani1028]

ありがとうございます。
url見て勉強します。

Answer 1

方法①はオブジェクト指向としてはありえないレベルで邪道です。
Appインスタンス2個以上必要な状況になったらどうするのって話になります。
そんなことはない、1つしか必要ないってことなら全部staticでいいじゃんってことになります。
(それができないならできないでシングルトンにしないと勝手に想定外のインスタンス作られて面倒なことになりかねない)

Comments

[shimotani1028]

ありがとうございます。
確かに全部staticでいいじゃんって思います。
もっとオブジェクト指向を勉強します。