複数レコードのアップデートについて

Question

<?php print "<pre>"; print_r($_POST); print "</pre>"; //formから受け取り //Array //( // [checkall] => on // [uid] => Array // ( // [0] => 21 // [1] => 20 // [2] => 19 // ) // // [ucheck] => Array // ( // [0] => OK // [1] => OK // [2] => OK // ) // // [update] => 更新 //) try { $db = new PDO('mysql:host=localhost;dbname=project;charset=utf8', 'phpusr', 'phppass'); if (isset($_POST['update'])) { $stt = $db->prepare('UPDATE project SET ucheck=:ucheck WHERE uid=:uid'); $stt->bindValue(':ucheck', $_POST['ucheck']); $stt->bindValue(':uid', $_POST['uid']); $stt->execute(); } } catch(PDOException $e) { die('エラーメッセージ：'.$e->getMessage()); } //DBのログ //Query UPDATE project SET ucheck='Array' WHERE uid='Array' PHP初心者でmysqlの複数レコードの更新の方法が分からないので お手数ですが教えてください。 formから[uid]と[ucheck]を受け取りupdate処理をしたいのですが 繰り返し構文を作ろうとしてもうまくいかない為お願いします。

Comments

[退会済みユーザー]

テーブルの定義がわからないと答えようがありません。定義を提示してください。

Answer 1

Before

try { 
  $db = new PDO('mysql:host=localhost;dbname=project;charset=utf8', 'phpusr', 'phppass'); 
  if (isset($_POST['update'])) { 
    $stt = $db->prepare('UPDATE project SET ucheck=:ucheck WHERE uid=:uid'); 
    $stt->bindValue(':ucheck', $_POST['ucheck']); 
    $stt->bindValue(':uid', $_POST['uid']); 
    $stt->execute(); 
  } 
} catch(PDOException $e) { 
  die('エラーメッセージ：'.$e->getMessage()); 
}

After

// デバッグコード
// 開発中は常にコレを記述することをお勧めします。
error_reporting(E_ALL);
ini_set('display_errors', 1);

try { 
  $db = new PDO('mysql:host=localhost;dbname=project;charset=utf8', 'phpusr', 'phppass'); 

  // 他の箇所で特に設定していない場合、これを実行しないとエラー発生時に例外が投げられない
  $db->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);

  if (isset($_POST['update'])) { 
    $stt = $db->prepare('UPDATE project SET ucheck=:ucheck WHERE uid=:uid'); 

    for (int $i = 0; $i < count($_POST['uid']); $i++) {
      $stt->bindValue(':ucheck', $_POST['ucheck'][$i]); 
      $stt->bindValue(':uid', $_POST['uid'][$i]); 
      $stt->execute(); 
    }
  } 
} catch(PDOException $e) { 
  die('エラーメッセージ：'.$e->getMessage()); 
}

ちなみに、本当はリクエストパラメータをサニタイズせずプログラム内で使用するのはよろしくありません。
（$stt->bindValue(':ucheck', $_POST['ucheck'][$i]); などの部分のこと）
http://www.weblio.jp/content/%E3%82%B5%E3%83%8B%E3%82%BF%E3%82%A4%E3%82%BA

その対策については、以下などを参考にまずはご自分で考えてみてください。
http://php.net/manual/ja/function.filter-input.php
http://php.net/manual/ja/function.filter-input-array.php

Answer 2

教えて頂いてありがとう御座います。
回答頂いたもので希望通りの動作が確認出来ました。セキュリティ対策もこれから覚えていきます。

まだ１年も経たない若輩者ではありますがこれからもお願いします。

Answer 3

try {
  $db = new PDO('mysql:host=localhost;dbname=project;charset=utf8', 'phpusr', 'phppass');
  $stt = $db->prepare('UPDATE project SET ucheck=:ucheck WHERE uid=:uid');

  if (isset($_POST['update'])) {
    foreach ($_POST as $i => $data) {
        $stt->bindValue(':ucheck', $data['ucheck'][$i]);
        $stt->bindValue(':uid', $data['uid'][$i]);
        $stt->execute();
    }
  }

} catch(PDOException $e) {
  die('エラーメッセージ：'.$e->getMessage());
}