SQL文が利いていない？

インターネット上での商品管理システムの勉強をしています。

<!DOCTYPE html>
<html>
<head>
<meta charset="UTF-8">
<title>ろくまる農園</title>
</head>
<body>




<?php

try{
$pro_code = $_POST['code'];
$pro_name = $_POST['name'];
$pro_price = $_POST['price'];

$pro_code = htmlspecialchars($pro_code, ENT_QUOTES, 'UTF-8');
$pro_name = htmlspecialchars($pro_name, ENT_QUOTES, 'UTF-8');
$pro_price = htmlspecialchars($pro_price, ENT_QUOTES, 'UTF-8');

$dsn = 'mysql:dbname=shop;host=localhost;charset=utf8';
$user = 'root';
$password= '';
$dbh = new PDO($dsn,$user,$password);
$dbh -> setAttribute(PDO::ATTR_ERRMODE,PDO::ERRMODE_EXCEPTION);

$sql = 'UPDATE mst_product SET name=?,price=? WHERE code=?';
$stmt = $dbh->prepare($sql);
$data[] = $pro_code;
$data[] = $pro_name;
$data[] = $pro_price;
$stmt->execute($data);

$dbh = null;


print '修正しました。<br />';

}
catch (PDOException $e){
print 'ただいま障害発生中です。ご迷惑おかけします。';
echo $e->getMessage();
exit();

}

?>
<a href="pro_list.php">戻る</a>

</body>
</html>

このようなコードを写したのですが、データベース上のデータが書き換わらない
ようで、戻るボタンを押しても商品名と価格も変化しません。
UPDATEの部分が利いてないような気がするのですが、
コードのミスなども含めてみていただきたいです。
似たような質問をしている方の記事を見つけたので
var_dump($data);
exit;
を打ち込んでみました。
array(3) { [0]=> string(1) "2" [1]=> string(6) "高菜" [2]=> string(3) "180" }
どうやら、書き換えたいデータは取得できているようです。

Zuishin

2017/05/27 09:11

書き写したならオリジナルがあるはずです。それと比べたらミスは自分でチェックできます。

kei344

2017/05/27 09:30

参考にした記事／書籍について、URLを記載ください。

退会済みユーザー

2017/05/27 10:09

書籍は「気づけばプロ並PHP」というものを使っています。

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

回答1件

ベストアンサー

$data[] = $pro_code;
$data[] = $pro_name;
$data[] = $pro_price;

の順番じゃないですかねぇ。

WHERE code=?
がマッチしていないんじゃないかと。

余談ですが、POST と htmlspecialchars の使い方があんまりよろしくないです。
[PHP] 自分の主要な記事のまとめ
を起点に色々読んでみるとイイです。

投稿2017/05/27 09:19

編集2017/05/27 09:19

退会済みユーザー

総合スコア0

退会済みユーザー

2017/05/27 10:14

今回もありがとうございました！ SQL文の指令と取得順合わせて書かないといけなかったんですね・・・知りませんでした

退会済みユーザー

2017/05/27 10:16

追伸：まとめてくださってる記事見られないです・・・qiitaのサーバー落ちてますか？

退会済みユーザー

2017/05/27 10:24

私の環境では特に問題ないですねぇ。ちなみに、リンク先は私がまとめたものではなく、mpyw さんのまとめです＾＾；

退会済みユーザー

2017/05/27 10:30

今、質問への追記・修正を見て気がついたのですが、以前結構指摘しませんでしたか？そろそろその教本、諦めたほうがイイですよ。

退会済みユーザー

2017/05/27 11:05

諦めたいとも思ってるんですけど、最低限の事もわかってないので…事実さっきのsqlとの対応記入順も知らなかったですし最近の構文、セキュリティに関する事も今使ってる本終わったら上書きしていこうと思ってるんですが、それじゃ遅いですか?

退会済みユーザー

2017/05/27 11:24

本読み後に一括で上書きででもイイですが、ここで質問されるとそのたびに反応してしまうので、できれば都度上書きを目指してほしいです。＊個人的な感想ですが、サンプルとして提示されているソースは、見るたびにイラッとくるんです。少なくとも、副読本での指摘は反映した状態で、質問していただけるとうれしいです。

退会済みユーザー

2017/05/27 11:35

申し訳ないです。ただ、質問してる時は前の章で使ったファイルの書き換えでミスしてるところ、本のなかでコメントもされてないところがあり正常に動作していないところです。（この記事もさっきの通り、記述順で書き換えが完了しない問題が起きたので...)この記事あげる前にも、いくつも書き換え忘れなどをチェックして潰したんですが、最後の書き換えができなかったんです。

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