ラジオボタンとプルダウンの入力保持の仕方

前提・実現したいこと

入力→確認→登録の流れのフォームを作成中です。
確認画面の前に戻るボタンを押して、修正のため入力画面に戻った際に値の保持をしたいのですがラジオボタン(性別)とプルダウン(都道府県)のみ値の保持の仕方が分かりません。
性別は男を初期値に設定したいです。

ご教授お願いいたします。

発生している問題・エラーメッセージ

ラジオボタンは女のみ値保持ができています。ソースコードを見てもなぜか女を選択した時のみcheckedが入っています。
都道府県は値保持ができず空欄のままです。

該当のソースコード

regist.php(登録画面)
1
2<form method="post" action="confirm.php">
3<li>
4<label>性別</label>
5<input  type="radio" name="gender" value="0" <?php if( !empty($_POST['gender']) && $_POST['gender'] === "0" ){ echo 'checked'; } ?>>男
6<input  type="radio" name="gender" value="1" <?php if( !empty($_POST['gender']) && $_POST['gender'] === "1" ){ echo 'checked'; } ?>>女
7</li> 
8
9<li>
10<label>都道府県</label>
11<select class="dropdown" name="prefecture">
12<option value=""disabled style="display:none;" <?php if(empty($_POST['prefectur'])) echo 'selected'; ?>></option>            
13<option value="北海道" <?php if( !empty($_POST['prefectur']) && $_POST['prefectur'] === "北海道" ){ echo 'selected'; } ?>>北海道</option>
14<option value="青森県" <?php if( !empty($_POST['prefectur']) && $_POST['prefectur'] === "青森県" ){ echo 'selected'; } ?>>青森県</option>
15｜
16｜
17｜
18｜
19｜
20同様に47都道府県
21</form>
22

confirm.php(確認画面)
1
2<li>
3<label>性別</label>
4<?php 
5if ( $_POST['gender'] === "0" ){ echo '男'; }
6else { echo '女'; 
7}
8?>
9</li>
10
11<li>
12<label>都道府県</label>
13<?php 
14echo $_POST['prefecture'];
15?>
16</li>
17
18<form method="post" action="regist.php">
19<input type="submit" class="button1" value="前に戻る">
20<input type="hidden" value="<?php echo $_POST['gender']; ?>" name="gender">
21<input type="hidden" value="<?php echo $_POST['prefecture']; ?>" name="prefecture">
22</form>
23

行動規範の内容に同意します

回答1件

ベストアンサー

都道府県

nameとPOSTで指定しているkeyが合っていません。
name : prefecture
post : prefectur

あとはvar_dump()でデバッグしつつ、ですね。

HTML度外視でまずはやってみてください。
regist.phpのほうでもひとまず冒頭で選択した情報を出す。
同じ分岐で書いてみる。

php
1<?php
2if($_SERVER['REQUEST_METHOD'] === 'POST'){
3    echo $_POST['gender'];
4    echo $_POST['prefecture'];
5    if( !empty($_POST['gender']) && $_POST['gender'] === "0" ){
6       echo '男';
7    }
8    if( !empty($_POST['gender']) && $_POST['gender'] === "1" ){
9        echo '女';
10    }
11    if( !empty($_POST['prefecture']) && $_POST['prefecture'] === "北海道" ){
12        echo '北';
13    }
14    if( !empty($_POST['prefecture']) && $_POST['prefecture'] === "青森県" ){
15        echo '青';
16    }
17}
18?>
19<form method="post" action="confirm.php">

あとは機能の仕様。

empty()

返り値
var が存在し、かつその値が空や0でなければ、つまり boolean のコンテキストで false と見なされる場合、false を返します。

つまりempty($_POST['gender'])は「男」を選択した「0」のときは必ずempty()はtrueが返っています。
emptyを使う際の注意点です。
0を「空である」と判定します。

php
1var_dump(empty(0)); //true
2var_dump(empty('0')); //true
3var_dump(empty(1)); //false
4var_dump(empty('1')); //fals

ユーザーからの入力は何が来るか分からない…と言う点でempty()でチェックするのは悪手だと考えられます。
ブラウザのデベロッパーツールから書き換えればhiddenからだって0を送れます。prefectureのほうも改ざん可能です。
「keyが存在したら」とう観点ならisset()でも良いのですけど、
prefectureの方もですが、filter_input()の利用を強くすすめます。

php
1<input  type="radio" name="gender" value="0" <?php if( filter_input(INPUT_POST,'gender') === "0" ){ echo 'checked'; } ?>>男
2<input  type="radio" name="gender" value="1" <?php if( filter_input(INPUT_POST,'gender') === "1" ){ echo 'checked'; } ?>>女
3

あと、今回ミスってたprefectureのことも踏まえると、
入力項目は配列で定義しておいて出力したほうがいいんじゃないかなと。
手動での作業を増やすことは、それだけミスやバグの可能性を高めます（コピペも「コピーする範囲の選択」は手動です）。

php
1<?php
2$gender = ['0'=>'男','1'=>'女'];
3
4foreach($gender as $value=>$label){
5    $checked = (filter_input(INPUT_POST,'gender') == $value) ? ' checked' : '';
6    echo <<<GENDER
7    <input  type="radio" name="gender" value="{$value}"{$checked}>{$label}
8GENDER;
9}