条件分岐が多い場合のリファクタリング

Question

$id = (int)$request->id;
        switch (true) {
            case ($id === 1):
                $hoge = 'aaa';
                $foo = 'bbb';
                break;

            case ($id === 2):
                $hoge = 'aaa';
                $hogehoge = 'aaawww';
                $foo = 'bbb';
                break;

            case ($id === 3):
                $hoge = 'aaa';
                $foo = 'bbb';
                $foohoge = 'bbbrrr';
                break;

このような条件分岐があった場合に、仕様の変更に強いコードを書くにはどうすればいいのでしょうか？
例：仕様の変更に強いコードとは、idが変更されてもコードを最小限に修正するだけで済むことをさす。

現状は、以下のように書いています。

$id = (int)$request->id;
        switch (true) {
            case ($id === 1):
                $hoge = 'aaa';
                $foo = 'bbb';
        　　　　return view('admin.test', compact(
　　　　　　　　　　'hoge',
                  'foo',
        　　　　));
                break;

            case ($id === 2):
                $hoge = 'aaa';
                $hogehoge = 'aaawww';
                $foo = 'bbb';
        　　　　return view('admin.test', compact(
　　　　　　　　　　'hoge',
                  'hogehoge',
                  'foo',
        　　　　));
                break;

            case ($id === 3):
                $hoge = 'aaa';
                $foo = 'bbb';
                $foohoge = 'bbbrrr';
        　　　　return view('admin.test', compact(
　　　　　　　　　　'hoge',
                  'foo',
                  'foohoge',
        　　　　));
                break;

自分なりの問題点としては、数字がもし変更された場合に変数やretrunも修正する必要があって手間。
なので、数字の変更に強くてかつreturnをいちいち書かなくてもいいようなコードの書き方があればご教示ください。

Comments

[m.ts10806]

それぞれの値がどこからどのように来た情報として設定されるか次第では。
idの増減の法則や要件も具体的に記載してください。

[退会済みユーザー]

法則や要件は今回はそこまで具体的ではないですね...
あくまでも例として書いているのものなので

Answer 1

そもそも論ですが、なぜ数字で管理しているのでしょうか？
そこをリファクタリングするのもいいかも

どうしても数字にこだわるのであれば
皆さんが提示しているマジックナンバーとデータ自体をテーブルに入れておけば
コード変更は一切必要なくなります

Answer 2

仕様変更にもよるでしょうが、変更箇所がまとまっていてかつ修正しやすいのがいいかと。
例えばこんなのはどうでしょう？

$data = [
    1=>[
        'hoge'=>'aaa',
        'foo' =>'bbb'],
    2=>[
        'hoge'    =>'aaa',
        'hogehoge'=>'aaawww',
        'foo'     =>'bbb'],
    3=>[
        'hoge'   =>'aaa',
        'foo'    =>'bbb',
        'foohoge'=>'bbbrrr'],
];

$id = (int)$request->id;

if (isset($data[$id])) {
	return view('admin.test', $data[$id]);
}

Comments

[退会済みユーザー]

このような書き方もできるのですね,,勉強になります。

Answer 3

共通項目があるなら初期値をセットしておくのが妥当

$id=・・・;
$ret=["hoge"=>"aaa","foo"=>"bbb"];
$list=[
  1=>[],
  2=>["hogehoge"=>"aaawww"],
  3=>["foohoge" => 'bbbrrr'],
  ];
if(in_array($id,array_keys($list),true)){
  $ret=array_merge($ret,$list[$id]);
}
print_r($ret);
//return view('admin.test',$ret);

Comments

[退会済みユーザー]

おお...なかなか解読するのは難しそうなコードですが、もう少し他の方の意見も参考にしてみます。