forで無限ループになっていないかどうか

質問

各アイテムの最新3件だけデータベースに残したいと思い、下記のようにしてみました。

動作を確認したところ問題なかったのですが、何か（無限ループする可能性があるなど）問題があるようでしたら、ご指摘いただけないでしょうか。

よろしくお願いいたします。

for ($num = 1; $num < 21; $num++){
// アイテムが20件ある場合

$sql = "SELECT COUNT(id) AS cnt FROM item where item_id=$num ;";
$res = mysql_query($sql, $conn) or die;
$row = mysql_fetch_array($res, MYSQL_ASSOC);
$count = $row["cnt"];

if($count<3){
// アイテムが3件より少なかったら何もしない
}
else{

$delete_count=$count-3;

$sql = "delete from item where item_id=$num order by date limit $delete_count;";
$res = db_query($sql, $conn);

}

}

回答No.2

yambejp · ベストアンサー

感じからすると、テンポラリに最新3件を抽出して、itemをtruncateし、
テンポラリを戻すのが一番早くて確実だとは思います

お礼

ありがとうございました！

回答No.1

agunuz

item_id を forなんかで回さないで、DBからselectしてくるのはダメなんですか？

$sql = "select * from (SELECT item_id, COUNT(id) AS cnt FROM item group by item_id) as item_tbl where cnt > 3";

というか、ネイティブなMySQL関数はすでに非推奨です。いい加減使うのはやめましょう。

http://www.php.net/manual/ja/mysqlinfo.api.choosing.php