【PHP】条件分岐のベストプラクティスを教えていただきたいです

Question

エラーは発生しておりません。

ただ
「どっちの方がより最適なんだろう？」
というのが判断できず教えていただきたいです、、、

質問内容

１.Bladeファイルの条件分岐はURLによる分岐で良いか
２．Controllerの条件分岐はinputの有無による分岐で良いか

条件分岐のベストプラクティス（エラーが出ない）について知りたいです。

DBへの登録機能を実装しております。

<form>タグで<input>の値を渡してcontrollerでDB登録といった基本的な形です。 APIは使っておりません。

異なるURLに対して、同じbladeファイル及びcontrollerファイルを使い回して、URLやinputの有無によって条件分岐させて「表示」及び「登録」機能を実装しております。

その際、「条件分岐」の方法がいくつかあるため、「いずれの方法がエラーが起きない設計になるんだろう？」の解がわからず、質問させていただきたいです。

既に問題なく動作は確認済みです。

【条件分岐例】
例１.createのcontrollerでsessionを渡す
例2.URLによる条件分岐
例3.inputの有無による条件分岐
例4.その他

現状

同じbladeファイル、同じcontrollerを使い回して登録機能を実装しております。

// ルーティング設定
Route::get('/register-teacher', [RegisteredController::class, 'create']);
Route::get('/register-student', [RegisteredController::class, 'create']);
Route::post('/register-teacher', [RegisteredController::class, 'store'])->name('register.user');
Route::post('/register-student', [RegisteredController::class, 'store'])->name('register.user');

<form method="POST" action="{{ route('register.user') }}">
	<input name="name" type="text">
	<input name="password" type="password">
	@if(Request::is('register-user'))
	<input name="student-name" type="text">
	@elseif(Request::is('register-user'))
	<input name="teacher-name" type="text">
	@endif
	<button type="submit">登録</button>
</form>

// 関数のみ抜粋
public function create(array $input)
{
	// 先生の場合
	if($input['teacher-name']) {
		// バリデーションチェック
		Validator::make($input, [
		    'name'         => ['required', 'string', 'max:255'],
		    'password'     => ['required', 'string', 'max:255' ,new Password, 'confirmed'],
		    'teacher-name' => ['required', 'string', 'max:255']
		])->validate();

		// DB登録フロー
		$account = Account::create([
		    'name'         => $input['name'],
		    'password'     => Hash::make($input['password']),
		    'teacher-name' => $input['teacher-name'],
		]);
	// 生徒の場合
	} elseif($input['student-name']) {
		// バリデーションチェック
		Validator::make($input, [
		    'name'         => ['required', 'string', 'max:255'],
		    'password'     => ['required', 'string', 'max:255' ,new Password, 'confirmed'],
		    'student-name' => ['required', 'string', 'max:255']
		])->validate();

		// DB登録フロー
		$account = Account::create([
		    'name'         => $input['name'],
		    'password'     => Hash::make($input['password']),
		    'student-name' => $input['student-name'],
		]);
	}
}

添付したコードに問題がある場合、またはより最適な方法がある等ありましたら、教えていただけると幸いです。

Comments

[m.ts10806]

現状でなにか問題が起きているのでしょうか

[saori777]

いえ、問題は起きておりません。
ただ、controllerの条件分岐について、URLやinputなど、条件分岐の方法がいくつかあるため、知見が深い皆様からどのように記述した方がより最適なのかを教えていただきたい次第です。

[m.ts10806]

１つのゴールへの道は１つではないので「最適」というのはなくて結局好みや現場でのルールに依存します。
なので「ベストプラクティス」のように曖昧に求めるより、具体的な方向性を示したほうがアドバイス得られやすくなると思います。

[hoshi-takanori]

ユーザーが 2 種類 (先生と生徒) あって、その処理をどう分けるべきかってことでしょうか。
分岐をどうするか以前に、登録に必要な情報や、ログイン後にアクセスできる範囲や権限が違ったりしますよね。
DB のユーザーテーブルを分けるか (分けた場合、同じ ID で先生と生徒の登録を許すか)、登録やログインのページを分けるか、似た内容のページだけど先生の方が情報量が多い場合、同じページでユーザーを見て出し分けるか、それとも別のページにするか、などなど、考えるべきことはいろいろありそう…。

[saori777]

ご丁寧に有難うございます。
テーブルは以下のように分けております。

accounts
teachers
students

accountsにログイン周り、teachersとstudentsにそれぞれ追加の情報を付与しております。
account_idをFKとして紐付けております。

登録情報などは基本的同じなので、同じcontrollerやbladeファイルを使用しており、bladeファイル及びcontrollerファイルはteacherかstudentによって表示などを切り分けております。

特に気になるのがcontrollerファイルの方で、現状は$input['teacher-name']の有無でteacherの処理かstudentの処理かを条件分岐しているのですが、
１．URLで分岐させる
２．sessionを持たせる
などなど、分岐させる方法は色々あるなと感じており、何が良いか検討がつかずご相談させていただいた次第です。
いずれの方法でもエラーは出ないと思いますが、何がより良いのかを教えていただきたかった次第です。

[退会済みユーザー]

teratailでは「ベストプラクティス」とか「最適」とかいう単語を使ってしまうと、超絶面倒くさいヤツを召喚してしまいます。気を付けてください！！

[hoshi-takanori]

「何がより良いのか」を判断するには、そもそも何のためにプログラムを作っていて、何を重視するかによります。
普通の学校のサイトなら、セキュリティが最重要かと思いますし、そもそも自分で登録する形にはならないような。
teratail のようなネット上でお互いに教え合うサイトなら、気軽に登録できるようにすることが重視されるでしょう。
また、実運用を考えないプログラミングの練習なら、お好きなように試行錯誤すれば良いのでは。

Answer 1

みなさま、ご丁寧に対応していただき有難うございました。
質問が抽象的すぎたので、みなさまの回答を持って終了とさせていただきます。