メソッドの引数の型について

Question

メソッド内の引数の数、型定義について検討をしています。
例えば、以下のPHPがあったとします。

use Illuminate\Http\Request;

$str1 = '文字列A';
$str2 = '文字列B';
$obj = new ClassA();
$obj->methodA($str1, $str2, $request);

class ClassA {
 public function methodA($str1:string, $str2:string, $request:Request) {
   // code
 }
}

$requestはオブジェクト変数なので、引数の数が少なくなりコード全体が見やすくなると思います。
コントローラー上からの利用するのであればいいと思うのですが、
一方でClassAのテストコードを作成すると、いちいちオブジェクトを作成する必要があり、
そうなるのであれば、各データ毎に変数を用意した方がいいのではと思ったりし、
最適解がわかりません。

どのような書き方をするのがベターなのでしょうか。
PHPを例にしていますが、言語には拘らない回答を求めています。

Answer 1

一方でClassAのテストコードを作成すると、いちいちオブジェクトを作成する必要があり、

そうなるのであれば、各データ毎に変数を用意した方がいいのかなと思ったりし、
最適解がわかりません。

「いちいちオブジェクトを生成」がデメリットであると思っているのであれば、オブジェクト指向というパラダイムを採用するべきではありません。

これは構造体でも同じですが、意味的な集まりをまとめて人間が理解しやすいようにするためにクラスや構造体を導入するものなので、要件を分析して得られた「意味的なグループ」があるのであれば、その設計に基づいて実装を行うだけです。

「各データ毎に変数を用意」するのが良いケースというのは、その各データ間の関係が疎で、このメソッド以外では別々に処理をする対象となっているなど、それなりの理由が先にあるはずです。

Comments

[退会済みユーザー]

ご返信が遅くなりました。
ご回答ありがとうございました。

Answer 2

What is the basis to determine if one is better? i think it will be best to execute depending on the design you're going for, rather than the grouping