データベースが更新できない

Question

前提・実現したいこと

PHP+MySQLでクライアントの情報を管理するシステムを作っています。
UPDATE文で情報を更新する機能を作成しましたが、更新ができません。

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

エラーメッセージは出ません。
if文で、更新に成功したら「更新完了」、失敗したら「更新できませんでした」が
表示されるようにしていますが、「更新完了」になりません。

該当のソースコード

if (isset($post['id'])){
    $id = $post['id'];
}

$name            = $post['name'];
$genre           = $post['genre'];
$url             = $post['url'];
.
.
.



$pdo = new PDO('mysql:host=xxxxxx;dbname=xxxxxx;charset=utf8','user','password');

.
.
.

if (isset($id)){
    $sql = $pdo->prepare('update client set name=?,genre=?,url=?..., where id=?');

            
                        
    if ($sql->execute([$_REQUEST['name'], $_REQUEST['genre'], $_REQUEST['url'], ... $_REQUEST['id']]))
                        {
                            echo '<p>更新完了</p>';                    
                            exit();
                        } else {
                            echo '<p>更新できませんでした。もう一度やり直してください。</p>';                            
                            exit();
                        }                    
}

### 試したこと

いろいろと調べてみましたが、初心者の為知識が乏しく、
これといった打開策がみつけられません。
phpmyadmin上でupdate文を実行してみましたが成功しました。
よろしくお願いします。

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

Answer 1

可能性としては更新のために渡されたパラメータがおかしい、のだとは思いますが、確実に何が起こっているか知るために、errorInfo() を呼び出しましょう。

$sql = $pdo->prepare('update client set name=?,genre=?,url=?..., where id=?');

if ($sql->execute([$_REQUEST['name'], $_REQUEST['genre'], $_REQUEST['url'], ... $_REQUEST['id']]))
{
  echo '<p>更新完了</p>';                    
  exit();
} else {
  print_r($sql->errorInfo());
  exit();
}                    

Comments

[3.a.m.op]

ご回答ありがとうございます。
自分で調べたときにerrorInfo()の使い方がよくわからなかったので助かりました。
実行したところ、
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 'other='' where id='4'' at line 3
と出ました。
これは、where id=?の辺りに記述ミスがあるということになるのでしょうか？

[yambejp]

otherの前にカンマがないとかその手の文法エラーたぐいでは？

[3.a.m.op]

まさにその通りでした！
otherの前にカンマがありませんでした！
初心者丸出しのケアレスミスでお恥ずかしい限りです。
初めての投稿で緊張しておりましたが、丁寧に教えていただき助かりました。
ありがとうございました。

Answer 2

if (isset($post['id'])){

$postは$_POSTなのでしょうか？

$id    = filter_input(INPUT_POST,'id');
$name  = filter_input(INPUT_POST,'name');
$genre = filter_input(INPUT_POST,'genre');
$url  =  filter_input(INPUT_POST,'url');

if(!is_null($id)){
  $sql='update client set name=?,genre=?,url=? where id=?';
  $stmt = $pdo->prepare($sql);
  $stmt->execute([$name,$genre,$url,$id]);
}

追記

念のためちゃんとかくと、pdoを使うときには必ずtry~catchでやるのが原則です

try{
  $dsn = 'mysql:host=localhost; dbname=mydb;charset=utf8;';
  $user = 'myuser';
  $password = 'mypassword';
  $pdo = new PDO($dsn, $user,$password);
  $pdo->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION); 
  $pdo->setAttribute(PDO::MYSQL_ATTR_INIT_COMMAND,'SET NAMES utf8');
  $pdo->setAttribute(PDO::ATTR_DEFAULT_FETCH_MODE, PDO::FETCH_ASSOC);//updateのときは意味ない
  $pdo->setAttribute(PDO::ATTR_EMULATE_PREPARES,false);
  if(!is_null($id)){
    $sql='update client set name=?,genre=?,url=? where id=?';
    $data=[$name,$genre,$url,$id];
    $stmt = $pdo->prepare($sql);
    $stmt->execute();
    $sql="SELECT * FROM tbl ORDER BY l";
    $stmt = $pdo->prepare($sql);
    $stmt->execute($data);
  }
}catch(PDOException $e){
  die($e->getMessage());
}

Comments

[3.a.m.op]

ご回答ありがとうございます。
はい、$_POSTになります。
上記の前に外部ファイルからのサニタイズをしており、
そこを変更し忘れてそのまま入力しておりました。
失礼しました。

filter_inputは初めて知りました。
調べて打開策に繋げられそうかやってみます。

[退会済みユーザー]

$pdo->setAttribute(PDO::MYSQL_ATTR_INIT_COMMAND,'SET NAMES utf8');
？？？

[yambejp]

$dsnに「charset=utf8」を指定する処理が効かないバージョンがあり
それでもSET NAMEをコマンドとして発行したくないときのおまじないです。
安定バージョンでやる場合は不要でしょうね

[退会済みユーザー]

コピペミスかと思ってました＾＾；
下位フォローか。失礼。

[3.a.m.op]

>te2ji様
ご回答ありがとうございます。