エラー表示 General error: 1366 Incorrect integer value: '' for column 'id' at row 1

現在mysqlにphpのPOST形式でデーターを入れようとしているのですが

エラーがでてしまい調べても修正方法がわからないため質問させていただきました

『エラーコード』
メンテナンス中の為ご迷惑をおかけいたします
id変数の展開->
PDOException: SQLSTATE[HY000]: General error: 1366 Incorrect integer value: '' for column 'id' at row 1 in C:\MAMP\htdocs\database\receive.php:34 Stack trace: #0 C:\MAMP\htdocs\database\receive.php(34): PDOStatement->execute(Array) #1 {main}

id変数の展開->で一度$id変数を表示してみましたがnull?みたいです
数字はinputで確認に入れたのですが...

html
1 <form method="post" action="http://localhost:80//database/receive.php" name="formName">
2
3
4 <div>商品ID</div>      
5        <input type="number" name="id" id="id">
6
7
8  <input type="submit" value="登録" name="register" id="register" />
9
10
11    </form>

php
1<?php
2
3ini_set('mbstring.internal_encoding','UTF-8');
4
5
6
7try{
8
9
10$id=$_POST["id"];
11
12
13$id=htmlspecialchars($id,ENT_QUOTES,"UTF-8");
14
15
16
17
18if($id=""){
19
20print "商品IDを入れてください"; 
21print '<br>';
22
23}
24
25require_once "../confidential/detabaseAccses.php";
26
27$sql="insert into ilysb(id) values(?)";
28
29$stmt=$dbh->prepare($sql);
30
31$data[]=$id;
32
33$stmt->execute($data);
34
35$dbh=null;
36
37}catch(exception $e){
38
39print "メンテナンス中の為ご迷惑をおかけいたします";
40
41print "<br>";
42
43print "id変数の展開->";
44
45print "$id";
46
47print "<br>";
48
49print $e; 
50
51exit();
52}
53
54?>

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

回答1件

ベストアンサー

エラーをきちんと読めば見えてくることもあるかと思います。

General error: 1366 Incorrect integer value: '' for column 'id' at row 1

Google翻訳：一般エラー：1366行1の列 'id'の整数値が不正です： ''

おそらくidの型がintegerなどの整数型で定義されているが、空文字が投入されたということなんでしょう。

エラーはもちろんですが、エラーメッセージの意味が分かったのであれば
そこから「どういう対策があるか」より、調査対象を自身のコードにした方が良いかと思います。
問題が起きているのは他でもない、自身のコードです。

良くコードを見てみましょう。

POST値を受け取っていますが

php
1$id=$_POST["id"];

if内の式で上書きしています

php
1
2if($id=""){

簡単に書くと、こう。

php
1<?php
2$a = "ああああああああああああああああああ";
3if($a=""){}
4echo $a;

ただ「未入力だったらエラーを出す」のはいいとして

php
1if($id=""){
2
3print "商品IDを入れてください"; 
4print '<br>';
5
6}
7

これだと、未入力でもこのエラーメッセージの文字列は出力されますが、
そのまま実行されますよ。

要件次第ですが、何かしらで処理の続行を止める必要があるのでは。
簡単にするなら、die()とかexit()ですね。

id変数の展開->で一度$id変数を表示してみましたがnull?みたいです
数字はinputで確認に入れたのですが...

確認する場所間違ってます。
「POSTで情報が来ているか」なら$_POSTをvar_dump()です。
別の変数に代入されてますが、変数はPOSTからきた情報かどうかはプログラムは知りません。

蛇足：
５点

１点目：
コードインデントはきちんとされたほうが良いかと思います。
可読性の面で悪くなりますし、スコープも見えにくくなります。
改行しすぎもよろしくありません。
コードフォーマット機能のあるエディタ（IDEが良い）を利用しましょう。

２点目：

php
1    $id = htmlspecialchars($id, ENT_QUOTES, "UTF-8");

DBにおさめる値なのであれば、HTMLエスケープしてはいけません。
「文字列の加工」にもなります。HTMLに表示するときのみ利用すべきです。
この直後に$idが空になるか確認していることから、「送信された情報」ではなく「PHPで加工した情報」に対するチェックになってしまっています。

type=numberの送信であること、空文字のチェックであることから
現状問題ないようには見えますが、この癖がつくと他のところで思わぬ穴に落ちます。

３点目:
細かいですが初期化してない配列にいきなり代入してます。

php
1 $data[] = $id;

↓

php
1$data = [];
2$data[] = $id;

４点目：
PDOなのであれば、PDOExceptionとするのが正確です。

php
1}catch(exception $e){

５点目：

C:\MAMP\htdocs\database

Windows でMAMP
MacでXAMPP　は悪手です。

teratailでもよくあがるお悩み相談です。
WindowsならXAMPP　MacならMAMP　で解決することも非常に多いです。
「どうしてもWindowsでMAMPでなければならない理由」ってないと思います。
だからと言って確かに「XAMPPでなければならない理由」もないのですが、
MAMPはWindowsに対して正規化されたものではないため、何か問題が起きた時に見つかる情報が少ないのです。

理想は、そういったOS関係なく利用できる仮想環境でLinuxでWebサーバーを立てることですね。LAMP環境が最も環境に左右されにくいです。

投稿2020/07/09 21:46

m.ts10806

総合スコア80850

nguyenseiji

2020/07/10 00:15

ご連絡ありがとうございます if($id="")を if($id=="")にしたら問題なく動きました変数の中身が空白と一緒ならだったので完全にスペルミスになります。本題の方は解決したのですが別のところでもご指揮頂き少し気になったので質問させていただきます『質問1』 if($id="")の()の中は条件を判断するだけの処理ではなく実行されてしまうのでしょうか例えばif($id="")とした場合 $idは空白か判断するだけでなく $idに空白を代入してしまうのでしょうか『質問2』 htmlspecialcharsは mysqlにデータを入れる時は使用しない mysqlから取り出してhtmlで表示する時のみ使用するの認識であっておりますでしょうか

nguyenseiji

2020/07/10 00:55

こちらについても回答頂ければ嬉しいです https://teratail.com/questions/276493