MYSQL DELETE文が実行されない

Question

DELTE文がうまく実行されない

php、mysqlを利用して簡易掲示板のユーザー登録機能を作っています。
そこで今実装したいのが仮登録（本登録フラグがFALSE）のまま１日以上経過したユーザーはデータベース上から削除するというプログラムです。

以下では、まず仮登録のユーザーをセレクトし、その投稿時刻が現在の時間より１日以上離れていれば、そのユーザーのID（ユニークな値）をもとにDELTEする、というような流れのつもりで記述しています。

<?php
$user = '*************';
$password = '********';
$dsn = "********************************************************";

$pdo = new PDO($dsn, $user, $password);
// if($pdo){
//   echo "繋げてはいます";
// }
$time_now = date('Y-m-d H:i:s');
//今の時間を表示してみる
//echo $time_now;

//仮登録の人をセレクト
$sql = "SELECT * FROM user_information WHERE registered = 0";
$stmt = $pdo->query($sql);
// if($stmt){
//   echo "実行できています";
// }
foreach($stmt as $row) {
  $time_post = $row["date"];
  $key = $row["id"];
  echo "ID:".$key."  ";
  $time_post_sec = strtotime($time_post);
  $time_now_sec = strtotime($time_now);
  $differences = $time_now_sec - $time_post_sec;
  //時間差を日という単位まで丸める
  $differences = gmdate("d",$differences) - 1;  //なぜか予定より１多い
  echo "時間差：".$differences."  ";
  if($differences > 0){
		$sql = 'DELETE FROM user_information WHERE id = "$key"';
    $stmt = $pdo->query($sql);
    if($stmt){
      echo "期限切れだったので消しました<br/>";
    }else{
      echo "期限切れだったけど消しませんでした<br/>";
    }
	}else{
    echo "仮登録後１日以内<br/>";
  }

}

 ?>

if文内ではユーザーはきちんと仕分けられ、登録後１日以上経っているユーザーに関しては”期限切れだったので消しました”を出力します。しかし、そこを通っているだけで、実際は消えていません。これはなぜでしょうか？DELETE文の書き方に問題がありますか？
ご教授いただけますと幸いです。

Comments

[Orlofsky]

DELTE → DELETE くらいは直しては？

[harajima]

ご指摘ありがとうございます。修正いたしました

[Orlofsky]

あと2ヶ所あります。

Answer 1

$sql = 'DELETE FROM user_information WHERE id = "$key"';

シングルクォートで囲んだ文字列の中では変数が展開されません。
つまり、$keyがそのまま文字列として出力されます。

全体をダブルクォートで囲み、$keyをシングルクォートで囲めばよいかと。

Comments

[harajima]

的確なご指摘ありがとうございました。最も早く望んでいた回答をいただけたのでベストアンサーとさせていただきました。

Answer 2

これだとidの値が文字列「$key」のものが削除されます。$key変数が展開されるようにsql文を作成してください。

deleteは0件でも正常なので質問に記載の動作になります。

  $sql = 'DELETE FROM user_information WHERE id = "$key"';

「期限切れだったので消しました」をechoしている所で、$sqlの内容もechoしてみればなにが起こっているか一発で解ると思います。

Comments

[harajima]

クオーテーションについて理解が逆になっていました…。
"deleteは0件でも正常"、"$sqlの内容をecho"とても参考になりました。今後にも活かします、ありがとうございました。

Answer 3

delete文自体に問題はありませんが、
phpの「文字列内での変数展開」についての理解と、delete結果の確認の仕方に問題があるのではないかと思います。

■phpの「文字列内での変数展開」
文字列内の変数展開は、ダブルコーテションで括られた文字列の中、で行われます。

$sql = 'DELETE FROM user_information WHERE id = "$key"';

では、シングルコーテーションで括られた文字列の中に変数がいますので、展開されません。

実際に実行されているSQLが

DELETE FROM user_information WHERE id = "$key"

になってしまっていると思います。

■deleteの結果の確認方法

delete文は、削除行数に関わらず、SQL的に問題がなければ、実行結果はいつでもtrueです。
例え条件で「削除対象行が0」だとしても、そのSQLが実行できたことには変わりないからですね。

なので、deleteは、SQL実行によって影響を受けた行数を取得した方が良いので、query()ではなくexec()で実行した方が良いかと思いますよ。

Comments

[harajima]

非常に丁寧な回答ありがとうございました。恐ろしく明快に説明していただき恐縮です。exec()での実行も勉強になりました。ありがとうございました。

Answer 4

DELETEを処理するメンテは１日１回で十分。
仮登録されてタイムスタンプ(datetime)に対して、その日の翌々日の0時に24時間以上前の
レコードを削除するメンテプログラムを仕込んでおいてください

仮に12/10のデータは12/12の0時に 12/11 0時以前のデータを削除する処理で削除します。
しかしこれでは実際には24時間を期限にできませんが
実際には仮登録ですから本登録処理の要求があったときに削除をします。

12/10 10:00に仮登録され、12/11 11:00に本登録の要求が来た場合、
要求に対して12/10 11:00以前のデータを削除した上で、本登録処理を走らせます
もし本登録が12/11 9:00にあれば、削除処理に絡まないので処理ができます

Comments

[harajima]

この次にちょうどやろうとしていたことでした。ご丁寧に教えていただき有難うございます、とても参考になりました。