PHP ユーザー関数定義を使用した入力フォームのバリデーションチェック

Question

現在PHPで図鑑投稿アプリを作成しています。
生き物の写真とその生き物の名前、分類、説明を投稿できるアプリです。

create.phpにて、

-new.phpでPOSTされてきたユーザーの入力値を$pbookに渡す。
-入力値のバリデーションチェックを行う

この２点をfunction.phpで定義したユーザー関数を使用して実装を試みています。
うまくいかず手詰まりになってしまいました。
変数のスコープや引数の設定に問題があるのかと考えましたが、解決に至りそうにないので質問しました。
解決策やヒントをご教授していただきたいです。よろしくお願いいたします。

実現したい事
-POSTされてきた入力値が空欄もしくは文字数オーバーなど不備がある場合のみ、$errorの各項目に'blank','length','type'の値を渡し、エラー文を出力する。

発生している問題点
-POSTされてきた入力値にかかわらず$error配列全てに'blank'の値が渡ってしまう

以下が該当のコードです。

function.php

<?php 
function h($str){
    echo htmlspecialchars($str, ENT_QUOTES, 'UTF-8');
}

function hbr($str){
    echo nl2br(htmlspecialchars($str, ENT_QUOTES, 'UTF-8'));
}

function optionLoop($start, $end, $selection=''){
    echo '<option value="">選択してね</option>';
    for($i=$start; $i<=$end; $i++){
        $i = sprintf('%02d', $i);
        $selected = $i==$selection?'selected':'';
        echo "<option value=\"{$i}\" $selected>{$i}</option>";
    }
}

function emptyError(){
    $error = ['sp_name'=>'', 'team'=>'', 'picture'=>'', 'description'=>''];
    return $error;
}

function hasError($error) {
    return $error['sp_name'] != ''
        || $error['team'] != ''
        || $error['picture'] != ''
        || $error['description'] != '';
}

function assignmentPost(){
    $pbook['user_id'] = $_POST['user_id'] ?? '';
    $pbook['sp_name'] = $_POST['sp_name'] ?? '';
    $pbook['team'] = $_POST['team'] ?? '';
    $pbook['picture'] = $_FILES['picture']['name'] ?? '';
    $pbook['description'] = $_POST['description'] ?? '';
    return $pbook;
}
//
function validatePbook($p){
    $error = emptyError();
    // $pbook = $_POST;
    // $pbook = assignmentPost();
    if(isset($pbook['sp_name']) == ''){
        $error['sp_name'] = 'blank';
    }elseif(mb_strlen($pbook['sp_name']) > 50){
        $error['sp_name'] = 'length';
    }
    if(isset($pbook['team']) == ''){
        $error['team'] = 'blank';
    }
    if(isset($pbook['picture']) == ''){
        $error['picture'] = 'blank';
    }else{
    $filename = $pbook['picture'];
        if(!empty($filename)){
            $ext = substr($filename, -3);
            if($ext != 'jpg' && $ext != 'JPG' && $ext != 'png'){
                $error['picture'] = 'type';
            }
        }
    }
    if(isset($pbook['description']) == ''){
        $error['description'] = 'blank';
    }
    return $error;
}

create.php

<?php 
session_start();
require_once __DIR__ . '/vendor/autoload.php';

error_reporting(E_ALL);
require_once ('db_connect.php');
require_once('function.php');
require_once('init.php');
$message = "入力エラーがあります";


    if($_SERVER['REQUEST_METHOD'] != 'POST'){
        header('Location:room.php');
        exit;
    }
    $pbook = assignmentPost();
    
    $error = validatePbook($pbook);
   
    if($_SESSION['id'] != '' || hasError($error)){
        logD($pbook, 'pbook');
        logD($error, 'error');
        require_once('new.tpl.php');
        exit;
    }
    
        //$errorに値が一つも入っていなければDBに接続する
        if(!isset($error)){
            $picture = date('YmdHis') . $_FILES['picture']['name'];
            move_uploaded_file($_FILES['picture']['tmp_name'], '/Applications/XAMPP/xamppfiles/htdocs/pbook/files/'.$picture);

        //入力内容をDBに保存する
       
        $sql = 'INSERT INTO picture (sp_name, team, picture, description, user_id)
                VALUES (:sp_name, :team, :picture, :description, :user_id)';
        $stmt = $pdo->prepare($sql);
        $stmt->bindValue(':sp_name', $pbook['sp_name'], PDO::PARAM_STR);
        $stmt->bindValue(':team', $pbook['team'], PDO::PARAM_STR);
        $stmt->bindValue(':picture', $pbook['picture'], PDO::PARAM_STR);
        $stmt->bindValue(':description', $pbook['description'], PDO::PARAM_STR);
        $stmt->bindValue(':user_id', $pbook['user_id'], PDO::PARAM_INT);
        $stmt->execute();

        
        $pdo = null;
        $stmt = null;

        $team = $pbook['team'];
        $url = "{$team}/index.php";
        header('Location:'.$url);
        exit();
        }

new.php

<?php 
session_start();
require_once('function.php');
$pbook = ['id'=>'', 'sp_name'=>'', 'picture'=>'', 'description'=>'',
          'team'=>'', 'user_id'=>''];

if(!isset($_SESSION['id'])){
    $_SESSION['return_uri'] =  "http://localhost/pbook/new.php";
    header('Location:login/login.php');
    exit;
}

if(isset($_SESSION['id']) && ($_SESSION['time']) + 3600 > time()){

$message = "図鑑登録";
require_once ('new.tpl.php');
var_dump($member);
}

new.tpl.php

<!DOCTYPE html>
<html>
<?php include('header_inc.php') ?>
<body>

<h2><?php h($message) ; ?></h2>
    <p></p>
<form action="create.php" method="post" enctype="multipart/form-data">
    <input type="hidden" name="user_id" value="<?php echo $_SESSION['id']; ?>">
    <label>種名</label>
    <input type="text" name="sp_name" value="<?php if(isset($pbook['sp_name'])){ echo $pbook['sp_name'];}?>">
    <?php if(isset($error['sp_name']) && $error['sp_name'] == 'blank'): ?>
    <p>*種名を登録してください*</p>
    <?php endif; ?>
    <?php if(isset($error['sp_name']) && $error['sp_name'] == 'length'): ?>
    <p>*種名は５０文字以内で登録してください*</p>
    <?php endif; ?>
    <label>班</label>
    <select name="team">
    <option value="">選んでね</option>
    <option value="sea"<?php if(isset($pbook['team']) && $pbook['team'] == 'sea'){echo 'selected';} ?>>海</option>
    <option value="kinoko"<?php if(isset($pbook['team']) && $pbook['team'] == 'kinoko'){echo 'selected';} ?>>きのこ</option>
    <option value="plant"<?php if(isset($pbook['team']) && $pbook['team'] == 'plant'){echo 'selected';} ?>>植物</option>
    </select>
    
    <?php if(isset($error['team']) && $error['team'] == 'blank'): ?>
    <p>*班を選んでください*</p>
    <?php endif; ?>
    <p></p>
    <label>写真</label>
    <input type="file" name="picture" />
    <p></p>
    <?php if(isset($error['picture']) && $error['picture'] == 'blank'): ?>
    <p>*写真を登録してください*</p>
    <?php endif; ?>
    <?php if(isset($error['picture']) && $error['picture'] == 'type'): ?>
    <p>*写真は「.gif」もしくは「.png」の形式で登録してください*</p>
    <?php endif; ?>
    <?php if(!isset($error['picture']) && !empty($error)): ?>
    <p>*もう一度写真を登録してください*</p>
    <?php endif; ?>
    <label>説明</label>
    <!-- name属性は$_POSTのキーとなる。$_POST['description'] -->
    <textarea name="description" cols="40" row="80"><?php if(isset($pbook['description'])){ echo $pbook['description'];}?></textarea>
    <p></p>
   <?php if(isset($error['description']) && $error['description'] == 'blank'): ?>
    <p>*説明を登録してください*</p>
    <?php endif; ?>
    <input type="submit" name="submit" value="登録する">
</form>
    <p><a href="room.php">班一覧に戻る</a></p>
    <?php if(isset($_SESSION)): ?>
    <p><a href="login/logout.php">ログアウトする</a></p>
    <?php endif; ?>

<?php include('footer_inc.php') ?>
</body>
</html>

Answer 1

この構成ならきちんとクラスを組んで、出力はテンプレートを作ってやるとよいでしょう。
ユーザーの入力をバリデーションするならfilter_inputするのが楽です

Comments

[1h_m]

回答ありがとうございます。
今回はvalidatePbookの引数を$pから$pbookに修正し、関数内のissetを除くことで解決できました。
まだ関数化のところで知識がおぼつかない状態でしたので、クラスを組んでの実装はまだ挑戦していないのですが次回はチャレンジしてみます。
確かにfilter_inputは便利そうなので次回は使ってみます。
新しい視点をくださりありがとうございました。

Answer 2

変数のスコープや引数の設定に問題があるのかと考えましたが、解決に至りそうにないので質問しました。
解決策やヒントをご教授していただきたいです。よろしくお願いいたします。

まずは現状で各箇所の変数がどうなっているかを確認してデバッグしてください。
多分今回の場合だと

function validatePbook($p){
    $error = emptyError();
    // $pbook = $_POST;
    // $pbook = assignmentPost();
    var_dump($pbook);
    die();
    if(isset($pbook['sp_name']) == ''){
        $error['sp_name'] = 'blank';

とすれば、blankを代入している直前の$pbookを確認できます。
想定していたものと違う値が入っていたら、それより前に遡って同じように確認していく。
というのを積み上げていけば想定した通りに動くようになります。

---

環境が不明なのでどこでも使えるvar_dump()によるデバッグを回答しましたが
var_dump()によるデバッグはソースコードを汚すなどの弊害や限界があるので、
リモートデバッグ環境を作ってブレークポイントを仕掛ける方が良いです。
余裕が出来たら
PHP リモートデバッグ xdebugといったキーワードに使用しているエディタやIDEをキーワードに追加して検索してみてください。

Comments

[1h_m]

回答してくださりありがとうございます。
順に値を確認していき、validatePbookの引数が$pとしていましたがこれが間違いだと気づき$pbookに修正しました。また関数内でissetの使用は不適切でしたので取り除いたところ想定通りの値が渡ってくれました。
リモートデバッグにも挑戦してみようと思います。
順を追ってどの場所でデバッグするか、とても大事なことを教えていただきました。ありがとうございました。

[tanat]

修正箇所のフィードバックありがとうございます。
解決して良かったです。
開発のうち、多くの時間はデバッグやテストで使われるというのは普通のことなので、
デバッグやのやり方を色々試してみるのも良いかもしれませんね。