phpのPDOバインドで空の出力が出てしまいます。

Question

前提・実現したいこと

PHPでサービスを作ってます。
別のページでは動いているPDOの命令が動きません。
バインドに問題があると思いますが、いろいろ試しましたがわかりませんでした。
よろしくお願いします。

環境　PHP Version 7.4.1　　　　mamp使用　　phpMyAdmin　バージョン情報： 4.9.3

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

出力が空で返ってきます。

array(0) { }

該当のソースコード

<?php
$post_choose_offer = $_POST['choose_offer'];

echo $post_choose_offer;

□□□□□□□□□□   ここでは 5f5383012c89e が返ってきています。 □□□□□□□□□□ 

try {

  　　// データベースに接続
  require_once('../〇〇〇〇/db_connection.php');

  　　// sql文を作成
  　　// POSTで渡されたoffer_codeがデータベース内に存在するかを検索するsql文
  $prepare = $pdo->prepare('SELECT * FROM offer WHERE offer_code = :offer_code');



  　　// sql文のバインド処理
  　　// [ :offer_code ]を$post_choose_offerでバインド
  $prepare->bindValue(':offer_code', (string) $post_choose_offer, PDO::PARAM_STR);

  　　// sql文実行
  $prepare->execute();

  　　// データベースからの戻り値を受け取り、変数に格納
  $offer_data = $prepare->fetchAll(PDO::FETCH_ASSOC);



} catch (PDOException $e) {
  echo '接続エラー' . $e->getMessage() . "\n";
  exit();
}


  echo ' offer_data <br>  ';
  var_dump($offer_data);
□□□□□□□□□□  ここでarray(0) { }が返ってきています。 □□□□□□□□□□ 

試したこと

以下のコードで、バインドせずに直接文字列を入れると、DBから欲しい情報が返ってきます。

$prepare = $pdo->prepare('SELECT * FROM offer WHERE offer_code = "5f5383012c89e"');

念のためにPOSTを受けた直後にキャストもしましたが、変化なし。　

(string)$post_choose_offer 

fetchAll を　fetch でも変化なし。

###追記

var_dumpで出力した結果

var_dump($post_choose_offer);

string(14) " 5f5383012c89e"

変数に文字列を入れてバインドした結果。

$post_choose_offer = "5f5383012c89e";

こちらで欲しい情報が取れました。
ですので、バインドの問題では無いのかな？

$prepare->PDOStatement::debugDumpParams();

こちらで出力すると

SQL: [54] SELECT * FROM offer WHERE offer_code = :offer_code
Params:  1
Key: Name: [11] :offer_code
paramno=0
name=[11] ":offer_code"
is_param=1
param_type=2

出力の内容をまだ理解して無いので、調べている途中です。

むやみにキャストしてはいけないとご意見を頂いたので、以下に変更しました。
ありがとうございます。

$prepare->bindValue(':offer_code', $post_choose_offer, PDO::PARAM_STR); 

###解決

form側でスペースが混入していたのに気付きませんでした。
アホでした。

string(14) " 5f5383012c89e"

回答を頂いたみなさま、大変ありがとうございました。

Comments

[m.ts10806]

コードはマークダウンのcode機能を利用してご提示ください

[m.ts10806]

コードやエラーはマークダウンのcode機能を利用してご提示ください。
https://teratail.com/questions/238564

[heypo]

大変失礼しました。
ご指摘ありがとうございます。
使い方にまだ慣れていないもので申し訳ございません。
マークダウン記法に変更いたしました。

Answer 1

まずは、
PDOStatement::debugDumpParamsを使用して、狙った値がバインド出来ているか確認してみましょう。

また、PDOの接続部分でどの様なオプションを指定しているか不明ですが
必ず
PHP・PDO、SQL実行時のエラーをExceptionで捕捉する
の様な形でSQLに文法エラーがあった時は例外がthrowされるように設定してください。

こちらで試した限りだとコードに問題は無い様に見えるので
前述の
PDOStatement::debugDumpParamsに合わせて

$post_choose_offer = $_POST['choose_offer'];

echo $post_choose_offer;

を

$post_choose_offer = $_POST['choose_offer'];
var_dump($post_choose_offer);

として、値が欠けたりしていないかを確認してみてください。
おそらく現状でも

$post_choose_offer = "5f5383012c89e";

とした場合は動作すると思います。

Comments

[heypo]

返信ありがとうございます。

```HTML出力
var_dump post_choose_offer
string(14) " 5f5383012c89e"
```
POSTで渡しているのはこの情報だけですので、問題なさそうです。

```PHP
$post_choose_offer = "5f5383012c89e";
```
こちらで欲しい情報が出力されました。

```PHP
$prepare->debugDumpParams();
```

で出力した結果
```HTML出力
SQL: [54] SELECT * FROM offer WHERE offer_code = :offer_code Params: 1 Key: Name: [11] :offer_code paramno=0 name=[11] ":offer_code" is_param=1 param_type=2
```

出力された内容をまだ理解できていないので、調べている途中です。

[tanat]

string(14) " 5f5383012c89e"

頭にスペースが入ってませんか?

[heypo]

解決しました！
ありがとうございます。
アホなミスで申し訳ございませんでした。

HTML form 側の select option value で変数をforeachで回してたりして複雑になっていたので、見落としていました。
大変助かりました。
本当にありがとうございました。

[tanat]

解決してよかったです。
var_dumpの良いところは文字数や型も表示してくれるところですね。

切り分けも含めた作りの定石としては、POST値をバリデーションする(今回のケースだと文字数やスペースのチェックして、問題があればDBに渡す前にエラーにする)事なのでその辺りも調べてみることをおすすめします

[heypo]

ありがとうございます。
配列はvar_dumpで出して、変数はechoで出す。
なんかそんなクセがついていました。
大変参考になりました。

Answer 2

まずはSELECT * FROM offer WHERE offer_code = '5f5383012c89e'
で想定のデータがとってこれるかどうかから。
※ダブルクォーテーションはMySQLだけだと思ってください。シングルクォーテーションで覚えた方が良いです。

これはPHPからではなくMySQLに対して直接実行してください。
これに限らずアプリケーションから実行する前に、想定の結果を得られるSQLを先に作ってください。
でないと「バインドに問題がある」という根拠になりません。
問題切り分けのために「どこまで正しく動くか」確認するのは必須です。
※SQLに関しては製造時ではなく設計時に形を作っておくことの方が多いです

あと、安易なキャストはバグの元です。
初心者を戒めるPHP#安易にキャストするな
echo $post_choose_offer;で確認できているなら、そのまま渡すべきです。

$prepare->bindValue(':offer_code', (string) $post_choose_offer, PDO::PARAM_STR); 

↓
$prepare->bindValue(':offer_code', $post_choose_offer, PDO::PARAM_STR); 

キャストしなくともPDO::PARAM_STRできちんと文字列として認識してくれますし、文字列として認識されるかどうか気になるのでしたらSQLに突っ込む前にバリデーションで弾くべきです。
想定する値があるなら尚更です。キャストで対応する案件ではありません。

Comments

[heypo]

返信ありがとうございます。
質問に追記を入れました。
キャストはクセでやってました。
ご指摘ありがとうございます。

[heypo]

解決しました！
form 側のミスでスペースが混入していました。
ありがとうございます。
アホなミスで申し訳ございませんでした。

[m.ts10806]

誰か言ってますか？アホなミスって

[heypo]

いえいえ、自責の念を込めて、です。

Answer 3

$prepare = $pdo->prepare('SELECT * FROM offer WHERE offer_code = :offer_code');

を

$prepare = $pdo->prepare('SELECT * FROM offer WHERE offer_code = ":offer_code"');

で試してみましょう。

Comments

[heypo]

返信ありがとうございます。
↓↓↓↓↓↓↓ offer_data ↓↓↓↓↓↓↓
array(0) { }
空で返ってきました。
残念です。

[seastar3]

場当たり的な回答で申し訳ありません。
こういう場合は、echo デバッグで原因を切り分けていくのが確実でしょう。今後のプログラミングスキル向上のためにも、粘り強く取り組んで見ることを推奨します。

[heypo]

いえいえ、ありがとうございます。
別のページで全く同じ仕組みが動いているので、迷宮に入っているところでした。
そのページからのコピペでも動かないので、別のところに原因があるのかもしれません。
がんばって解決してみます。

[m.ts10806]

>":offer_code"
これだとバインド結果
"'abcd'" のようになるのでは・・

[heypo]

解決しました！
form 側のミスでスペースが混入していました。
ありがとうございます。
アホなミスで申し訳ございませんでした。