フォームを作る

質問

現在フォームを作成しているのですが、
ふと気になったので質問させてください。

フォームのフローとして　入力→確認→確定　が大体の場合当てはまると思いますが入力チェック等、再利用する処理は別スクリプトに書くとして、
フォーム自体はなるべく１つのファイルで完結させた方がいいのでしょうか？
それとも画面ごとにスクリプトを分けるような形がいいのでしょうか？

「そうじゃなくてこうした方が」「自分はこうしてる」等など、
ご意見、アドバイス等よろしくおねがいします。

回答No.2

hogehoge78 · ベストアンサー

確認画面がはさんである以上、最初の入力がされたときに値が正しいか検証をして、その後、確認画面に含まれた値が正しいか再検証を行う必要は必ずあります。（hiddenで含まれた値の書き換え、cookieで与えた場合でも書き換えがありうるし、sessionで引いた場合でも万が一ということもあるので。）
そうなると、それらの値のチェックを関数なり検証用クラスなりを使って処理すると思うのですが、
index.php⇒confirm.php⇒thanks.php
と遷移する場合、POST先であるconfirm.phpとthanks.phpで同じ検証用プログラムを走らせる必要がある上に、値が間違っていた場合は、confirm.phpでは、index.phpに記述したHTMLをまた含める必要があることを考えると、tom223さんの仰っているようにアクセスされるPHPファイルは一つにしてあったほうがやりやすい気はします。
-------index.php--------
<?php

if(strtoupper($_SERVER['REQUEST_METHOD'] == 'POST')){

$flag = validate($_POST); //何らかの入力された文字が正しいかノチェック

if(isset($_POST['confirm']) && $flag == true){ //<input type="submit" name="confirm" />が叩かれた場合

require_once '../template/confirm.html'; //入力項目確認画面

}elseif(isset($_POST['submit']) && $flag == true){ //<input type="submit" name="submit" />が叩かれた場合

require_once '../template/thanks.html'; //登録完了画面
}else{
require_once '../template/index.html';
}

}else{

require_once '../template/index.html'; //HTMLファイル（通常のフォームのもの）を読み出す
}

?>
簡単ですが、こんな感じでしょうか。

お礼

回答ありがとうございます。
やはりアクセスするファイルは１つにするのが良い感じなんですね。
恥ずかしながら入力チェックは確認画面でしか行ってませんでした。
これを機に最後の処理の前にもチェックするよう作成する事にします。