MySQLのデータをPHPで更新したい

Question

前提・実現したいこと

MySQL 5.7.26
PHP 7.3

テーブル名:syohin_data
id name tanka
1 りんご 300
2 ばなな 400

上記テーブルをPHPで更新したいのですが、実行時にエラーが出てしまいます。
SQL Server 2014で実行した場合,問題なく更新されるのですが、MySQLで実行するとエラーになってしまいます。update文を1つにした場合は問題なく更新されたので、2つある場合にエラーになるのは確認しました。MySQLではこのような形で更新するのは不可能なのでしょうか？
ご教授お願いいたします。

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

Fatal error: Uncaught PDOException: 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 'UPDATE SYOHIN_DATA1 SET TANKA1 = 600 WHERE ID = 2' at line 1 in

該当のソースコード

<?php
$sql = 'UPDATE SYOHIN_DATA1 SET TANKA1 = 500 WHERE ID = 1;UPDATE SYOHIN_DATA1 SET TANKA1 = 600 WHERE ID = 2';

  // DB接続情報
   $user = 'ユーザー名';
   $pass = 'パスワード';
   $dbnm = 'DB名';
   $host = 'ホスト名';
  // 接続先DBリンク
   $dsn = "mysql:host={$host};dbname={$dbnm}";


  // DB接続
	try {
	  $conn = new PDO($dsn, $user, $pass, array(PDO::MYSQL_ATTR_INIT_COMMAND => 'SET NAMES utf8'));
	  $conn->setAttribute(PDO::ATTR_EMULATE_PREPARES, false);
	  $conn->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
	} catch (Exception $e) {
	  echo "<p>DB接続エラー</p>";
	  echo $e->getMessage();
	  exit();
	}

  //実行
       $result = $conn->query($sql);
       if( $result === false )
       {
	echo mysql_errno($result) . ": " . mysql_error($result). "\n";
       }
  //接続クローズ
        $result =null;
        $conn = null;
?>

Comments

[m.ts10806]

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

[Orlofsky]

エラーメッセージやコードは https://teratail.com/help/question-tips#questionTips3-7 の [コード] に修正してください。

[Orlofsky]

エラーではありませんが、
ネーミング時、ローマ字表はヘボン式を使われては？
http://www.geocities.jp/kagemusyamk1/r01-hepburn.html

[Orlofsky]

[PDO]タグを追記されては？

Answer 1

MySQLではこのような形で更新するのは不可能なのでしょうか？

MySQLで、というよりPHPからの実行が不可能だと思います。
1つのSQLにつき1の結果を返します。
手元にSQLServerないので試してませんが。

設定次第で可能とのことです。

MySQL 関数 (PDO_MYSQL)

PDO::MYSQL_ATTR_MULTI_STATEMENTS (integer)
FALSE にすると、PDO::prepare() や PDO::query() でのマルチクエリの実行を無効にします。

つまり、trueを指定すると複数実行できるということになります。
trueにするリスクはあります。
いずれにしても後述のようにSQLインジェクション対策は必須です。

蛇足：
他にもイカン実装が幾つかあります。

• PDOならExceptionではなくPDOExeptionであるべき
• SQL実行までtry内に含むべき（接続時より実行時の方がエラーになりやすい）
• query()ではなくprepare->bindValue->executeの流れにすべき。

本来は値も固定ではなく可変になるのでそうなるとqueryでそのまま実行するとSQLインジェクションの脆弱性があります。

• DBに変更をかける処理なのでトランザクション→コミットorロールバックは入れておきましょう。
• PDOなのにmysql_errno()ではないでしょう。tryの中に含めば済む話かと。

というかmysql*系はPHP7では削除された機能です。

Comments

[maisumakun]

> 1つのSQLにつき1の結果を返します。

PDO-MySQLの設定次第で、複文を実行するかの選択が可能です。

https://blog.tokumaru.org/2016/12/pdo.html

[m.ts10806]

なんと。
貴重な情報ありがとうございます。

[maisumakun]

（以前に別件で複文にハマったことがありまして）

[退会済みユーザー]

記事だと標準で許可の模様？

[m.ts10806]

んーどうなんでしょう。
現状動いてないなら明示的にtrueにするか、やはり本数分実行するのが良いですね。
私なら後者。本数分実行します。

[退会済みユーザー]

今回は
PDO::ATTR_EMULATE_PREPARES, false
なんで、複文が使えないかと。

参考：
https://blog.tokumaru.org/2013/12/pdo-and-mysql-allow-multiple-statements.html
「PDO+MySQLで複文実行できる条件」のあたり
> PHP5.2.0以下を使っている場合、、あるいは PDO::setAttribute(PDO::ATTR_EMULATE_PREPARES, false); を呼んでいる場合は、PDO::queryによるSQL呼び出しであっても下記のようにいったんprepareされます。

ちなみに俺はここもイヤ。
PDO::MYSQL_ATTR_INIT_COMMAND => 'SET NAMES utf8'
多分実害はないだろうけど。。。

[m.ts10806]

te2jiさん
フォローありがとうございます。勉強になります。
やはり謎の実装は多いですね。
SET NAMESは私も気にはなりましたが、他の実装の方がまずそうなので、私も実害なしと思い結局スルーしました。

[kazumasa118]

ありがとうございます！！！解決しました！！
『PDO::ATTR_EMULATE_PREPARES, false』を消したらできるようになりました！

[m.ts10806]

今回はそこよりもまずい実装がたんまりあるのでご認識を。

[退会済みユーザー]

＊複文を使用しないで、分割すべきかと。

初学者は、エミュレーションを off にした実装を心がけたほうが良いです。

以下の違いがわかるようになってから、on を検討してください。
https://qiita.com/mpyw/items/b00b72c5c95aac573b71#%E3%82%A8%E3%83%9F%E3%83%A5%E3%83%AC%E3%83%BC%E3%82%B7%E3%83%A7%E3%83%B3%E3%81%AB%E9%96%A2%E3%81%99%E3%82%8B%E3%81%BE%E3%81%A8%E3%82%81

参考
https://qiita.com/te2ji/items/56c194b6cb9898d10f7f
https://teratail.com/questions/233051

[kazumasa118]

ご指摘ありがとうございます！
大変勉強になりました！

[m.ts10806]

PHPでデータベースに接続するときのまとめ

回答にも貼ってますが途中じゃなく、トップからリンク貼っておけばよかった。
この記事にはずっとお世話になってるなあ

Answer 2

バッドノウハウ

更新（ちゃんとデータ件数文分けて投げたほうが早いです。）

UPDATE t
  SET   a = CASE name 
    WHEN 'a' THEN 2 
    WHEN 'b' THEN 3
    ELSE a   END
WHERE name in ('a', 'b');

検証用データ作成

CREATE TABLE t ( name varchar(1) primary key, a int, b int, c int);

insert into t values 
( 'a', 1, 1, 1 ),
( 'b', 1, 1, 0 ),
( 'c', 1, 0, 1 ),
( 'd', 0, 1, 1 ),
( 'e', 0, 0, 1 ),
( 'f', 0, 1, 0 ),
( 'g', 1, 0, 0 ),
( 'h', 0, 0, 0 );

Comments

[kazumasa118]

ご指摘ありがとうございます。
勉強になりました。