mysql5.7でのプリペアドステートメント構文のエラー

Question

料理レシピアプリを構築中、エラーに悩まされ詰まっております。

こちらのコードで次のエラーが出ます。
何度見直してもどこが悪いのかわかりません。

構文が悪いと出てますが、どこが悪いかご指摘くださいませんでしょうか？

<?php
require_once('functions.php');
require_once('config.php');
ini_set('display_errors', TRUE);
error_reporting(E_ALL);
var_dump($_POST);

$user_name = $_POST['user_name'];
$recipe_name = $_POST['recipe_name'];
$howto = $_POST['howto'];
$category = (int) $_POST['category'];
$difficulty = (int) $_POST['difficulty'];
$budget = (int) $_POST['budget'];

try{
  if(empty($_POST['id'])) throw new Exception('ID不正');
  $id = (int) $_POST['id'];

  $dbh = new PDO(
      DB_DSN,
      DB_USER,
      DB_PASS,
      [
          PDO::ATTR_EMULATE_PREPARES, false,
          PDO::ATTR_ERRMODE => PDO::ERRMODE_EXCEPTION,
          PDO::ATTR_DEFAULT_FETCH_MODE => PDO::FETCH_ASSOC,
      ]
  );

  $sql = "update recipes set user_name = ?, recipe_name = ?, category = ?, difficulty = ?, budget = ?, howto = ?, where id = ?";
  $stmt = $dbh->prepare($sql);
  $stmt->bindValue(1, $user_name, PDO::PARAM_STR);
  $stmt->bindValue(2, $recipe_name, PDO::PARAM_STR);
  $stmt->bindValue(3, $category, PDO::PARAM_INT);
  $stmt->bindValue(4, $difficulty, PDO::PARAM_INT);
  $stmt->bindValue(5, $budget, PDO::PARAM_INT);
  $stmt->bindValue(6, $howto, PDO::PARAM_STR);
  $stmt->bindValue(7, $id, PDO::PARAM_INT);
  $stmt->execute();
// print_r($stmt->fetchAll());
  $dbh = null;
  echo "レシピの編集が完了しました。";

} catch(Exception $e){
  echo "エラー発生： " . h($e->getMessage(), ENT_QUOTES, 'UTF-8') . "<br>";
  die();
}
// <a href="index.php">トップページに戻る</a>

エラーと「var_dump($_POST);」の結果。

array(7) { ["user_name"]=> string(7) "paise33" ["recipe_name"]=> string(9) "うんこ" ["category"]=> string(1) "1" ["difficulty"]=> string(1) "1" ["budget"]=> string(3) "334" ["howto"]=> string(13) "tttyyyy mmmm" ["id"]=> string(2) "19" } エラー発生： SQLSTATE[42000]: Syntax error or access violation: 1064 You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'where id = 19' at line 1

よろしくお願いします。

Answer 1

whereの前の,が不要では。

, where id = ?

setとbindValueのところを手書きするのではなく配列に詰め込んでimplodeするとかのほうがコード量も少なくなりますし確実性が増します。

$set = [];
$bind = [];
if(!is_null($_POST['user_name'])){
  $set[] = " user_name = ? ";
  $bind["user_name"]["value"] = $_POST['user_name'];
  $bind["user_name"]["type"] = PDO::PARAM_STR;
}
if(!is_null($_POST['category'])){
  $set[] = " category = ? ";
  $bind["category"]["value"] = $_POST['category'];
  $bind["category"]["type"] = PDO::PARAM_INT;
}

//...中略

$sql = "update recipes set ".implode(",",$set)." where id = ?";
$stmt = $dbh->prepare($sql);
$bind_num = 1;
foreach($bind as $bind_name=>$bind_info){
 $stmt->bindValue($bind_num, $bind_info["value"], $bind_info["type"]);
 $bind_num++;
}
$stmt->execute();

Comments

[rude_rockers]

ありがとうございます！全てが解決いたしました！！毎度お恥ずかしい。脳がエラーになっておりますw
教えていただいた書き方もやってみようと思います。

[m.ts10806]

いかに短いやり方もで最大の効果を出すかというのはもの作りの永遠のテーマだと思いますが、配列もうまく使えばスリムなコードを書くのに役立ちます。
単にデータをループするだけが配列の脳ではないので今回のやり方も発想のひとつとして自身に取り込んでみてください。
(selectの動的なwhere句を作るときにも同じような考え方が使えます)

[rude_rockers]

脳が溶けそうですが、なんとか進みます。

[m.ts10806]

厳しいこと言うと
そういうわけのわからない例えを出してる暇があったらミニマムコードのひとつでも書いて動かしてエラーで一喜一憂したほうが良いです。
書かないコードは動かないしエラーを乗り越えないと成長もしません。
失敗したぶんだけ成長するのはスポーツと同じと考えます

[rude_rockers]

す、すいません・・・
書くことの大切さを痛感しています。

Answer 2

直接の原因は既に指摘のあるとおりですが、以下もおかしいですね。

PDO::ATTR_EMULATE_PREPARES, false,

正しくは以下かと思います。

PDO::ATTR_EMULATE_PREPARES => false,

Comments

[rude_rockers]

あ、ありがとうございます！！
調べ直してみます！！

[rude_rockers]

本当だ！！ありがとうございます！！
コピペした部分がバレてしまいましたww

Answer 3

update文のwhereの前にカンマがありますが、不要です。

  $sql = "update ～略～ howto = ?, where id = ?";

より詳細の調査になれば、テーブル構造も提示願います。

Comments

[rude_rockers]

ありがとうございます！！解決できましたm(_ _)m

Answer 4

よんでエラーの通り、
その文脈だとwhere句を置く場所ではなくなっているから
その構文付近を見直してみましょう

Comments

[rude_rockers]

ありがとうございます！
。。。本当ですね。
さっきも翻訳したはずなのに、whereのカンマに気づけませんでした。。。

しかも、データに「うんこ」なんて幼稚なワードでテストしてたのを公開しているなんて、思いもしませんでした・・・・・！！！