データベースを更新したい。

Question

前提・実現したいこと

updateを使ってデータベースの内容を更新したいのですが、エラーが出てしまいます。

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

Syntax error or access violation: 1064 You have an error in your SQL syntax; check the manual that corresponds to your MariaDB server version for the right syntax to use near 'WHERE id = 2' at line 1

該当のソースコード

<?php
$user = 'suzuki';
$pass = 'Antaresu905';
if (empty($_GET['id'])) {
  echo 'IDを正しく入力してください。 ';
  exit;
} 
$id = (int)$_GET['id'];
$recipe_name = $_POST['recipe_name'];
$howto = $_POST['howto'];
$category = (int)$_POST['category'];
$difficulty = (int)$_POST['difficulty'];
$budget = (int)$_POST['budget'];
try {
  $dbh = new PDO('mysql:host=localhost;dbname=db1;charset=utf8', $user, $pass);
  $dbh->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
  $sql = 'UPDATE recipes SET recipe_name = ?, category = ?, difficulty = ?, budget = ?, howto = ?, WHERE id = ?';
  $stmt = $dbh->prepare($sql);
  $stmt->bindValue(1, $recipe_name, PDO::PARAM_STR);
  $stmt->bindValue(2, $category, PDO::PARAM_INT);
  $stmt->bindValue(3, $difficulty, PDO::PARAM_INT);
  $stmt->bindValue(4, $budget, PDO::PARAM_INT);
  $stmt->bindValue(5, $howto, PDO::PARAM_STR);
  $stmt->bindValue(6, $id, PDO::PARAM_INT);
  $stmt->execute();
  $dbh = null;
  echo 'ID: ' . htmlspecialchars($id,ENT_QUOTES) . 'レシピの更新が完了しました。 ';

} catch (PDOException $e) {
  echo 'エラー発生: ' . htmlspecialchars($e->getMessage(), ENT_QUOTES) . '<br>';
  exit;
}
?>

### 試したこと



### 補足情報（FW/ツールのバージョンなど）

ここにより詳細な情報を記載してください。

Answer 1

Whereの前の,があることで「まだ更新するカラムがある」という風にSQLが解釈されてるのでは。
ただ、PHPがやってるのはその形式の文字列をDBに届けて結果を受け取るだけなので、PHPからいきなりではなく、DBに直に実行して想定の結果が得られてからPHP（というかアプリケーション側）から実行してください。

Comments

[ryuguu]

すぐに回答していただきありがとうございました！
質問をさせて頂いたあとに類似の質問を見つけ、回答者様が直接DBで実行して問題なければphp
で実行してください。というのを見かけ、ｓｑｌでupdate構文？を打ち込んでいたところ","の部分でエラーが出たので気づけました。と同時に回答を頂いてしまったので大変申し訳ありません。調べが足らず早計に質問してしまいご迷惑をおかけしました。
これからもよろしくお願いします。

[m.ts10806]

PHP関係なら私かもしれません（各所で同じこと書いてるから）
最初のころは仕方ないとは思います。
参考書や記事はプログラムとSQL一緒くたに書いてありますから。
ただ、「単体実行が可能であるもの」という観点で見ると良いと思います。