PHP deleteが反映されない

Question

投稿したデータを削除するため、削除リンクを踏み

index.php

<!-- リスト表示 -->
<article>
  <?php while($list = $lists->fetch()): ?>
    <p><?php print('名前:  ' . $list['name'] . 'さん  ' .  '     ' .  '出身:  '  . $list['from_japan'] . '    ' .  $list['age'] . '歳');?></p>
    <time><?php print($list['created']); ?></time>
    <!-- 管理者のみ表示　今回はスルー -->
    <a href="update.php?id=<?php print($list['id']);?>">編集</a>
    ｜
    <a href="delete.php?id=<?php print($list['id']);?>">削除</a>
    <hr>
  <?php endwhile; ?>

delete.phpにidを渡し、

<pre>
  <?php
  // 値が渡らない
  require('dbconnect.php');

  if (isset($_REQUEST['id']) && is_numeric($_REQUEST['id'])) {

    $id = $_REQUEST['id'];
    $stmt = $db->prepare('DELETE FROM to_pass_away WHERE id=?');
    $stmt->execute(array('id'));
  }
  header('Location: index.php');
  ?>
</pre>

delete.phpに飛ばされるも、データが削除されず残ったままです。
憶測がつかないので、ご教授いただければ幸いです。

Answer 1

  $stmt->execute(array('id'));

せっかく受け取ろうとしている
$id
はここでexecute()に渡してあげて下さい。

  $stmt->execute([$id]);

($idに想定通りにidの値がわたってる事も確認してください)
execute()の直前で

var_dump($id);
die();

等で確認。(確認できたら削除)

---

まずは
PHP デバッグ 方法
などで検索してみて、デバッグ方法を確立することをおすすめします

Comments

[退会済みユーザー]

みおとしてーた

[majka]

無事解決しました。
確かにデバックを考慮せず進めていました。
ありがとうございました。

[tanat]

解決してよかったです。

リダイレクトがある場合はどうしてもデバッグしにくくなってしまうので、
IDEを使ってブレークポイントで止めたり(PHP xdebug ブレークポイントあたりで検索してみてください)、回答の様にdie()で止めたりする必要がありますね。

Answer 2

まず、GETで受ける前提であるなら、$_GET['id']を使うべきです。
一旦var_dump($_GET['id']);をphpの冒頭に入れて
パラメータを受信できているかを確認してください。

そして、dbconnect.phpでどんなことをやっているかわかりませんが、
トランザクション処理を有効にしているならば、
コミット処理を入れないと反映されません。

/* 変更をコミットする */
$db->commit();

それと、ていねいにやるなら、
PHP: PDOStatement::bindValue - Manual
でプリペアドステートメントのパラメータを文字列か数値か指定すると改善したりして。

    $id = $_GET['id'];
    $stmt = $db->prepare('DELETE FROM to_pass_away WHERE id=?');
    $stmt->bindValue(1, $id, PDO::PARAM_INT);
    $stmt->execute();

これで $id が渡ります。
あるいは、

    $stmt->execute(array($id));  // not 'id'

Comments

[majka]

回答ありがとうございます。
var_dumpの結果は string(2) "11"でした(11はid)
また、dbconnect.phpではただmysqlからデータを取得する処理だけなので、
<?php
try {
$db = new PDO('mysql:dbname=deceased;host=127.0.0.1;charaset=utf8', 'root', '');
} catch (PDOException $e) {
echo 'DB接続エラー: ' . $e->getMessage();
}
?>

idを数字に指定すれば通ったりしますかね、？

[退会済みユーザー]

受信はできているから、ちゃんとexecute()のところまで渡るように、気を抜かずコードを見直しを。
（回答に見直しをかけたので、ご確認を。）

[majka]

executeを正しく理解せずに使用していたので非常に助かりました。
ありがとうございました。