PHP 未入力でPOSTした時にエラーメッセージを出したい。

前提・実現したいこと

PHPで、入力値をPOSTしデータベースに挿入するコードの練習をしています。
表示結果としては、挿入した入力値を表示し、以下のデータが挿入されたという旨を表示します。
値が未入力のまま送信された場合に、「〇〇を入力していません」という文を表示します。

発生している問題

「$name」、「$age」、「email」が入力してもらう値であり、そのうち「name」と「age」が入力されていなかったときに「$err」に入れてそれぞれのエラーを出したいのですが、なかなかうまくいかず、行き詰っています。

###エラーメッセージ
Cannot use isset() on the result of an expression (you can use "null !== expression" instead)

該当のソースコード

PHP
1<?php
2    if(isset(($_POST["name"]) || mb_strlen($_POST["name"])) === 0){
3        $err = "nameを入力してください";
4    }elseif(isset(($_POST["age"]) || mb_strlen($_POST["age"])) === 0){
5        $err = "ageを入力してください";
6    }
7
8    $dsn = 'mysql:dbname=sample;host=localhost;charset=utf8';
9    $user = 'root';
10    $password = '';
11    $name = htmlspecialchars($_POST["name"]);
12    $age = htmlspecialchars($_POST["age"]);
13    $email = htmlspecialchars($_POST["email"]);
14
15    $data = [];
16
17    try{
18
19    $dbh = new PDO($dsn, $user, $password);
20    $dbh->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
21    $sql = "INSERT INTO user (name, age, email) VALUES (:name, :age, :email)";
22    $stmt = $dbh->prepare($sql);
23    $stmt->bindValue(':name', $name, PDO::PARAM_STR);
24    $stmt->bindValue(':age', $age, PDO::PARAM_STR);
25    $stmt->bindValue(':email', $email, PDO::PARAM_STR);
26    $stmt->execute();
27
28    }catch (PDOException $e){
29       echo($e->getMessage());
30       die();
31    }
32?>
33
34<html>
35    <body>
36        <h1>以下のデータを挿入しました</h1>
37        <?php
38        if(isset($err)){
39        foreach($err as $date){
40                echo $date;
41            }
42            exit();
43        }else{
44            echo "name:".$name."<br>".
45                "age:".$age."<br>".
46                "email:".$email;
47            }
48        
49        ?>
50    </body>
51</html>

試したこと

主にissetのあたりのコードを色々試しました。
①
if(isset(($_POST["name"]) === TRUE || mb_strlen($_POST["name"])) === 0){
$err = "nameを入力してください";
}elseif(isset(($_POST["age"]) === TRUE || mb_strlen($_POST["age"])) === 0){
$err = "ageを入力してください";
}
②
if(isset((($_POST["name"]) || mb_strlen($_POST["name"]) === 0) || (($_POST["age"]) || mb_strlen($_POST["age"])) === 0){
$err = "nameとageを入力してください";
}
上記の2つのコードも同じエラーが出ます。

ご教授お願い致します。

ITU

2020/08/10 10:41

追記確かに、issetの中はorではなくANDが正しいです。修正を重ねているうちに誤ったコードのまま気付かずに投稿してしまったようです。申し訳ありませんでした…。皆様、丁寧な回答をありがとうございます。提案していただいたコードは確かに正常に動作しますが、 if( isset( $_POST["name"] ) && mb_strlen( $_POST["name"] ) === 0 ){} この形は崩さずに実現したいと思っています。不可能でしょうか…

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

回答3件

関数の開始と閉じの位置がおかしいです。
スペースやインデントを適宜活用し、1個ずつ書けば間違いは起きません。

php
1if(  )
2
3if(  ){}
4
5if( isset(  )  ){}
6
7if( isset( $_POST["name"] )  ){}
8
9if( isset( $_POST["name"] )  || mb_strlen(  ) ){}
10
11if( isset( $_POST["name"] )  || mb_strlen( $_POST["name"] ) === 0 ){}
12
13

しかし、
if( isset( $_POST["name"] ) || mb_strlen( $_POST["name"] ) === 0 ){}

これだと、$_POSTにnameがある時点で、「"nameを入力してください"」に入りますけど、そういう仕様で問題ないですか？
「未入力」なら||(or)ではなく&&(and)条件にすべきではないでしょうか。
それか||のままで !isset() || ( isset() && mbstrlen() )

filter_input()使った方が良さそうに思います。

その他：
・htmlspecialchars()あてる場所そこじゃダメです。画面出力のところにのみ入れてください。
・ exit();そこに書いてしまうと</body>とか出力されませんよ。

投稿2020/08/07 05:16

編集2020/08/07 05:25

m.ts10806

総合スコア80875

ざっくりこんな感じで

投稿2020/08/07 05:29

yambejp

総合スコア116441

yambejp

2020/08/07 05:29

<?PHP $name=filter_input(INPUT_POST,"name"); $email=filter_input(INPUT_POST,"email"); $age=filter_input(INPUT_POST,"age",FILTER_VALIDATE_INT); $err=["name"=>"","email"=>"","age"=>""]; if(empty($name)) $err["name"]="名前いれて"; if(empty($email)) $err["email"]="メールアドレスいれて"; if(empty($age)) $err["age"]="年齢いれて"; if($_SERVER["REQUEST_METHOD"] =="POST"){ if(count(array_filter($err,function($x){return $x!=="";}))>0){ print_r($err); }else{ print "complete"; } } function h($x){ return htmlspecialchars($x); } ?> <form method="post"> <input name="name" value="<?=h($name)?>" placeholder="name"><br> <input name="email" value="<?=h($email)?>" placeholder="email"><br> <input name="age" value="<?=h($age)?>" placeholder="age"><br> <input type="submit" value="send"> </form>

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

スマートな書き方ではありませんが、質問者さんがわかりやすいように書くならば・・・

php
1if (isset($_POST["name"]) == FALSE)
2{
3	if (isset($_POST["age"]) == FALSE)
4	{
5		$err = "name と age を入力してください";
6	}
7	else
8	{
9		$err = "name を入力してください";
10	}
11}
12else
13if (isset($_POST["age"]) == FALSE)
14{
15	$err = "age を入力してください";
16}