複数の変数のupdateについて

Question

各店舗(id)のリーダー(leaders)に、なる(value=2)ならない(value=3)をupdateしたいのですが以下のコードを実行してもカラムの値が変わることはありません。
正しいコードを教えて頂きたいのですが、どなたかご教授をよろしくお願い致します。

<form method="post" action="approval.php">

<?php
try
{

$dsn='mysql:dbname=xxx;host=xxx;charset=utf8';
$user='xxx';
$password='xxx';
$dbh=new PDO($dsn,$user,$password);
$dbh->setAttribute(PDO::ATTR_ERRMODE,PDO::ERRMODE_EXCEPTION);

$sql="SELECT * FROM user_list WHERE group_name=? && leader=0";
$stmt=$dbh->prepare($sql);
$data[]=$_SESSION['group_name'];
$stmt -> execute($data);

$dbh=null;

while(true)
{
	$rec=$stmt->fetch(PDO::FETCH_ASSOC);
	if($rec==false)
	{
		break;
	}
	 
print '<tr><td><input type="checkbox" name="leaders[';
print htmlspecialchars( $rec['id'], ENT_QUOTES);
print ']"  value="2" checked="checked"></td>';
print '<td><input type="checkbox" name="leaders[';
print htmlspecialchars( $rec['id'], ENT_QUOTES);
print ']"  value="3"></td>';
print '<td>';
print $rec['name'];
print '</td>';

print '<td>';
print $rec['request_day'];
print '</td></tr>';
print'<input type="hidden" name="ids[]" value="';
print htmlspecialchars( $rec['id'], ENT_QUOTES);
print'">';
}
print '</table >';
print '<div>&nbsp;</div>';
print'<input type="submit"  value="返答をメールする" name="change"/>';
print'</form>';
}
catch (Exception $e)
{
	 print '<div>&nbsp;</div>';
	 print '<div>&nbsp;</div>';
	 print '<font color="green">';
	 print 'ただいまシステム障害により本サービスを利用することができません。</br></br>大変ご迷惑をお掛けしております。';
	 print '</font>';	
 exit();
}
}
?>

<input type="submit"  value="返答をメールする" name="change"/>
</form>

approval.php

<?php
if($_SERVER["REQUEST_METHOD"] ==  'POST' ) {
$leaders = $_REQUEST['leaders'];
$ids = $_REQUEST['ids'];

$pdo = new PDO("mysql:host=xxx; dbname=xxx; charset=utf8", "xxx", "xxx", array( PDO::ATTR_EMULATE_PREPARES => false ) );

$ins = array();
for($i=0; $i<count($ids); $i++) {
    $ins[] = ":id{$i}";
}

$sql = "update user_list set leader=:leader where id in (" . implode(', ',$ins) . ")";
$condition = array(":leader" => $leaders);
for($i=0; $i<count($ids); $i++) {
    $condition[":id{$i}"] = $ids[$i];
}
$statement = $pdo->prepare( $sql );
$result = $statement->execute( $condition );
}
?>

Answer 1

approval.php

<?php

// 追加
ini_set('display_errors', '1');
error_reporting(E_ALL);

if($_SERVER["REQUEST_METHOD"] ==  'POST' ) {
$leaders = $_REQUEST['leaders'];
$ids = $_REQUEST['ids'];

$pdo = new PDO("mysql:host=xxx; dbname=xxx; charset=utf8", "xxx", "xxx", array( PDO::ATTR_EMULATE_PREPARES => false ) );

// 追加
$pdo->setAttribute(PDO::ATTR_ERRMODE,PDO::ERRMODE_EXCEPTION);

$ins = array();
for($i=0; $i<count($ids); $i++) {
    $ins[] = ":id{$i}";
}

$sql = "update user_list set leader=:leader where id in (" . implode(', ',$ins) . ")";
$condition = array(":leader" => $leaders);
for($i=0; $i<count($ids); $i++) {
    $condition[":id{$i}"] = $ids[$i];
}

// 追加
var_dump($sql);
var_dump($condition);

$statement = $pdo->prepare( $sql );
$result = $statement->execute( $condition );
}
?>

これで、変数$sql、$conditionのデータ構造がブラウザに表示され、
さらに何らかの PDO関連のエラーが発生した場合、ただちに処理を中断し、エラーメッセージをブラウザに表示するようになります。

この状態で再度、approval.php を実行し、ブラウザに表示されるメッセージを確認してください。

追記

<?php

ini_set('display_errors', '1');
error_reporting(E_ALL);

if($_SERVER["REQUEST_METHOD"] == 'POST' ) {
    $leaders = $_REQUEST['leaders'];

    $pdo = new PDO("mysql:host=xxx; dbname=xxx; charset=utf8", "xxx", "xxx", array( PDO::ATTR_EMULATE_PREPARES => false ) );
    $pdo->setAttribute(PDO::ATTR_ERRMODE,PDO::ERRMODE_EXCEPTION);


    $sql = "update user_list set leader= :leader where id = :id";

    var_dump($sql);
    var_dump($leaders);

    $statement = $pdo->prepare( $sql );

    foreach ($leaders as $id => $leader) {
        $statement->bindValue(':leader', (int) $leader, PDO::PARAM_INT);
        $statement->bindValue(':id', $id, PDO::PARAM_INT);
        $results[$id] = $statement->execute();
    }

    var_dump($results);
}
?>

Comments

[alkeran]

2店舗のleaderカラムを"2"にupdateしようと実行したら、以下のエラーが表示されました。
どうすればよろしいでしょうか？
string(60) "update user_list set leader=:leader where id in (:id0, :id1)" array(3) { [":leader"]=> array(2) { [4]=> string(1) "2" [7]=> string(1) "2" } [":id0"]=> string(1) "4" [":id1"]=> string(1) "7" }
Notice: Array to string conversion in .../approval.php on line xxx

[KiyoshiMotoki]

それを見やすく整形すると、以下のようになります。

■ $sql のデータ構造
　string(60) "update user_list set leader=:leader where id in (:id0, :id1)"

■ $condition のデータ構造
　array(3) {
　　　[":leader"]=> array(2) {
　　　　　[4]=> string(1) "2"
　　　　　[7]=> string(1) "2"
　　　}
　　　[":id0"]=> string(1) "4"
　　　[":id1"]=> string(1) "7"
　}

■ エラーメッセージ
　Notice: Array to string conversion in .../approval.php on line xxx

「Notice: Array to string conversion」
とは、配列を文字列にキャストしようとした際に表示されるエラーメッセージです。
"in .../approval.php on line xxx" 部分により、どのソースファイルの何行目でそれが行なわれたかが分かります。

ただし、該当の箇所でキャストを実行していなくとも、
そこで呼び出しているメソッドの内部で実行されている場合もあります。

あとは、ご自分で考えてみてください。

[alkeran]

以下のコードに直したのですが、
string(54) "update user_list set leader=:leader where id in (4, 7)" array(1) { [":leader"]=> array(2) { [4]=> string(1) "2" [7]=> string(1) "2" } }
Notice: Array to string conversion in .../approval.php on line xxx
というエラーが出ました。

<?php

ini_set('display_errors', '1');
error_reporting(E_ALL);

if($_SERVER["REQUEST_METHOD"] == 'POST' ) {
$leaders = $_REQUEST['leaders'];
$ids = $_REQUEST['ids'];

$pdo = new PDO("mysql:host=xxx; dbname=xxx; charset=utf8", "xxx", "xxx", array( PDO::ATTR_EMULATE_PREPARES => false ) );

$pdo->setAttribute(PDO::ATTR_ERRMODE,PDO::ERRMODE_EXCEPTION);

$sql = "update user_list set leader=:leader where id in (" . implode(', ',$ids) . ")";
$condition = array(":leader" => $leaders);

var_dump($sql);
var_dump($condition);

$statement = $pdo->prepare( $sql );
$result = $statement->execute( $condition );
}
?>

[alkeran]

以下のコードで成功することができました。
ありがとうございました。

<?php

ini_set('display_errors', '1');
error_reporting(E_ALL);

if($_SERVER["REQUEST_METHOD"] == 'POST' ) {
$leaders = $_REQUEST['leaders'];
$ids = $_REQUEST['ids'];

$pdo = new PDO("mysql:host=xxx; dbname=xxx; charset=utf8", "xxx", "xxx", array( PDO::ATTR_EMULATE_PREPARES => false ) );

$pdo->setAttribute(PDO::ATTR_ERRMODE,PDO::ERRMODE_EXCEPTION);

$sql = "update user_list set leader=:leader where id in (" . implode(', ',$ids) . ")";
$condition = array(":leader" => implode(', ',$leaders));

var_dump($sql);
var_dump($condition);

$statement = $pdo->prepare( $sql );
$result = $statement->execute( $condition );
}
?>

[KiyoshiMotoki]

惜しいけど、それも違います。

エラーなく実行できますが、意図する結果にはなりません。
おそらく、一番上のチェックボックスの選択内容で、全ての更新対象のレコードを更新してしまうはずです。

正しく動作するコードを回答欄に追記しましたので、見比べてみてください。

[alkeran]

ありがとうございます。
$ids = $_REQUEST['ids'];
は不必要でしょうか？

[KiyoshiMotoki]

approval.php に
　var_dump($_REQUEST['leaders']);
という一行を追加し、リクエストパラメータ 'leaders' がどのような構造になっているか
確認してみてください。

その上で、ご自分で考えてみてください。

[alkeran]

本当にありがとうございます。
成功しましたが、idsという変数を渡しているのに、$idで認識されるのが不思議です。
じっくり考えたいと思います。

[KiyoshiMotoki]

Good luck!

Answer 2

とりあえずですが、不要な括弧を取り除きました。これでどうでしょうか？

<form method="post" action="approval.php">

    <?php
    try {

        $dsn = 'mysql:dbname=xxx;host=xxx;charset=utf8';
        $user = 'xxx';
        $password = 'xxx';
        $dbh = new PDO($dsn, $user, $password);
        $dbh->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);

        $sql = "SELECT * FROM user_list WHERE group_name=? && leader=0";
        $stmt = $dbh->prepare($sql);
        $data[] = $_SESSION['group_name'];
        $stmt->execute($data);

        $dbh = null;

        while (true) {
            $rec = $stmt->fetch(PDO::FETCH_ASSOC);
            if ($rec == false) {
                break;
            }

            print '<tr><td><input type="checkbox" name="leaders[';
            print htmlspecialchars($rec['id'], ENT_QUOTES);
            print ']"  value="2" checked="checked"></td>';
            print '<td><input type="checkbox" name="leaders[';
            print htmlspecialchars($rec['id'], ENT_QUOTES);
            print ']"  value="3"></td>';
            print '<td>';
            print $rec['name'];
            print '</td>';

            print '<td>';
            print $rec['request_day'];
            print '</td></tr>';
            print'<input type="hidden" name="ids[]" value="';
            print htmlspecialchars($rec['id'], ENT_QUOTES);
            print'">';
        }
        print '</table >';
        print '<div>&nbsp;</div>';
        print'<input type="submit"  value="返答をメールする" name="change"/>';
        print'</form>';
    } catch (Exception $e) {
        print '<div>&nbsp;</div>';
        print '<div>&nbsp;</div>';
        print '<font color="green">';
        print 'ただいまシステム障害により本サービスを利用することができません。</br></br>大変ご迷惑をお掛けしております。';
        print '</font>';
        exit();
    }
    ?>

    <input type="submit" value="返答をメールする" name="change"/>
</form>

Comments

[alkeran]

ありがとうございます。
できませんでした。すみません。

[s8_chu]

エラーが発生しているならば、それを掲載すると良いと思いますが、エラーは出ていないのでしょうか？

[alkeran]

エラーは出ていません。
DBは何も変化がありません。

[s8_chu]

実行できないということはエラーが発生していると考えられるのですが、PHPでエラーを表示する設定はしているのですか？

[alkeran]

はい。onにしています。
それぞれのidごとにleaderのデータを上書きしたいのですが...。

Answer 3

「実行することができません」とは具体的にどうなるのでしょうか？
-> エラーがでる？ or
-> 無反応？ or
-> 意図しないデータが挿入されている? or
-> 意図したelementと異なるものの値が更新されている？

またこの質問で得たい情報は何でしょう？
-> 何故動かないのかの理由？
-> 何が問題なのか？
-> 正しいコード？

とりあえず理由に関して
原因と思われる部分は以下です。

$sql = "update user_list set leader=:leader where id in (" . implode(', ',$ins) . ")"; //*3
$condition = array(":leader" => $leaders); //*1
for($i=0; $i<count($ids); $i++) {
    $condition[":id{$i}"] = $ids[$i]; //*2
}

*1 では$conditionの値は次のようになってるはずです。
array(":leader" => array(xxx,yyy,zzz)) ※中の配列には$leadersが入っている
2 $conditionの中は上記の通りで、[':id{$i}']の展開は[':id1']のようなデータだと思いますが、なんてキーは無いはずなので、恐らく何も起きないでしょう。
最終的に3のsqlの内容は次のようになると思われ、この通りだとSQLは正常に実行されないので、エラーになるか、何も実行しないかだと思います。

UPDATE user_list
SET leader=[:learer => array-object]
WHERE id IN ("1,2,3,4,5")

Comments

[alkeran]

無反応です。カラムの leader が何も変化しません。2 or 3 をupdateしたいのですが...。

[alkeran]

正しいコードを教えて頂きたいです。