バリデーションに引っ掛かったらカテゴリーを選択してくださいが消えてしまう

Question

前提・実現したいこと

バリデーションに引っ掛かったらカテゴリーを選択してくださいが消えてしまうので、
バリデーションに引っ掛かっても「カテゴリーを選択してください」が消えないようにしたい。

以下のようにカテゴリーボックスを作成してます。

現在の状況だと、バリデーションに引っ掛かったら「カテゴリーを選択してください」が消えてしまいます。

なので、他のところでバリデーションに引っ掛かっても「カテゴリーを選択してください」が消えないようにしたい。

なお、カテゴリーを選択した上で他のバリデーションにひっかかる場合は以下のようにカテゴリーが選択された状態なので実装上問題ないです。

該当のソースコード

post.php

session_start();
require_once(ROOT_PATH .'/Models/Post.php');
require_once(ROOT_PATH .'function.php');
ini_set('display_errors', "On");
$obj = new Post();

// ログインしているユーザid
$company_id = $_SESSION['login_company']['id'];

$err = [];

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

      $title = filter_input(INPUT_POST, 'title');
      $text = filter_input(INPUT_POST, 'text');
      $tags = filter_input(INPUT_POST, 'tags');
      $category = filter_input(INPUT_POST, 'category');
      $price = filter_input(INPUT_POST, 'price');
   
      if(empty($category)) {
        $err['category'] = "カテゴリーを以下のセレクトボックスから選択してください";
      }
    
      // エラーがなければ、DBにデータを保存し、リスト一覧にページ遷移
      if(count($err) === 0) {
        $j_tags = implode('/', array_column(json_decode($_POST['tags']), 'value'));
        $result = $obj->createList($company_id,$_POST,$save_path, $j_tags);
        $_SESSION['list_msg'] = "投稿が完了しました！";
        header('Location: index.php');
        exit;
      }
        <!-- カテゴリー -->
        <h6 id="p-category"><label for="kome"><font color="red">*</font></label>カテゴリーを選択してください</h6>
        <?php if(isset($err['category'])): ?>
            <p class="err_one"><font color="red"><?php echo $err['category'] ?></font></p>
          <?php endif; ?>
        <select class="form-select" id="category-select" name="category">
          <?php if(isset($_POST['category'])): ?>
            <option selected><?php echo h($obj->setCategoryName($category)); ?></option>
          <?php else: ?>
            <option selected value="0">カテゴリーを選択してください</option>
          <?php endif; ?>
          <option value="1">求人</option>
          <option value="2">人材育成・研修</option>
          <option value="3">集客・マーケティング</option>
          <option value="4">Webサイト制作</option>
          <option value="5">Webコンテンツ制作</option>
          <option value="6">イラスト</option>
          <option value="7">営業・コールセンター代行</option>
          <option value="8">ビジネス相談・経営コンサル</option>
          <option value="9">士業(税理士・行政書士など)</option>
          <option value="10">翻訳・語学</option>
          <option value="11">クリーニング</option>
          <option value="12">音楽・ナレーション</option>
          <option value="13">その他</option>
        </select>

Post.php

  /** カテゴリーIDからカテゴリー名に変換
   * @param int $category_id
   * @return string
   */

  public function setCategoryName($category)
  {
    if($category === '1') {
      return '求人';
    } elseif($category === '2') {
      return '人材育成・研修';
    } elseif($category === '3') {
      return '集客・マーケティング';
    } elseif($category === '4') {
      return 'Webサイト制作';
    } elseif($category === '5') {
      return 'Webコンテンツ制作';
    } elseif($category === '6') {
      return 'イラスト';
    } elseif($category === '7') {
      return '営業・コールセンター代行';
    } elseif($category === '8') {
      return 'ビジネス相談・経営コンサル';
    } elseif($category === '9') {
      return '士業(税理士・行政書士など)';
    } elseif($category === '10') {
      return '翻訳・語学';
    } elseif($category === '11') {
      return 'クリーニング';
    } elseif($category === '12') {
      return '音楽・ナレーション';
    } elseif($category === '13') {
      return 'その他';
    }
  }

試したこと

以下の部分を削除し、デフォルトで最初からカテゴリーを選択している状態にしてみた。

<option selected value="0">カテゴリーを選択してください</option>

バリデーションに引っ掛かっても1回目はデータがデフォルトの物が送信されるが、2回ひっかかると空欄になってしまう。

Comments

[itagagaki]

filter_inputの内容も示していただけますか？

[退会済みユーザー]

filter_inputの内容とは、他のバリデーションメッセージ等のことでしょうか？

[itagagaki]

あ、いや、なんでもないです。すみません。

Answer 1

setCategoryName()に0だったときの選択肢がないです。

というか、selected属性をつければ別に一番上にある必要はないので、
そもそものセレクトボックスの作り方間違えてます。
今の実装だとバリデーション掛からなかったときに同じvalueが2つ並ぶことになりますよ。

同じ情報を複数個所に持つメンテナンス性の悪さも加味すると、配列なりデータベースで設定情報持っておいて、それぞれに一致するかどうかでselected出力するだけでいいのでは。

<?php

$selectlist = ["選択して","a","b","c"];
$category = filter_input(INPUT_POST, 'category');



echo '<select>'.PHP_EOL;
foreach($selectlist as $n=>$v){
    echo '<option value="'.$n.'"'.(($category==$n)?' selected':'').'>'.$v.'</option>'.PHP_EOL;
}
echo '</select>'.PHP_EOL;

Comments

[退会済みユーザー]

ご回答ありがとうございます。
このようなやり方もあるのですね。勉強になります。
実装してみてわからない点があればまたご質問させていただきます。

[m.ts10806]

本件の直接の原因は１行目でこたえていますが、そちらはどうでしょうか。

[退会済みユーザー]

結論、2回目バリデーションに引っ掛かったときに空欄になってしまいます。

elseif($category === '0') {
return 'カテゴリーを選択してください。';
}
setCategoryNameにvalueが0だった場合に上記追加してやってみたのですが、1回目バリデーション引っ掛かったときは問題なくカテゴリーを選択してくださいが表示されるんですが、2回目以降バリデーションにひっかかると空欄になります。

[m.ts10806]

デバッグしてください。
書いたとおりにしか動きませんので。
回答読んでないのかもしれませんが、下記も少なからず影響しているはずです。

>今の実装だとバリデーション掛からなかったときに同じvalueが2つ並ぶことになりますよ。