0

我正在尝试删除数据库中过去 2 个月的数据。

我在我的数据库中保存这样的格式date("n-Y");zo 输出将是8-2013

现在我正在尝试像这样删除过去 2 个月：

代码：

echo "<h2>Opruimen.</h2>";

$AFGELOPENMAAND = date("n-Y",strtotime("-1 Months"));
$AFGELOPENMAAND2 = date("n-Y",strtotime("-2 Months"));

$result = $pdo->prepare("SELECT * FROM photos WHERE geupload = $AFGELOPENMAAND OR geupload = $AFGELOPENMAAND2");
$result->execute();
$hoeveel = $result->rowCount();

if($hoeveel != 0){
    foreach($result as $row){
        unlink($row["thumb"]);
        unlink($row["location"]);

        $resultDEL = $pdo->prepare('DELETE FROM photos WHERE id = :id');
        $resultDEL->execute(array(':id' => $row["id"]));

        echo 'De vorige 2 maanden zijn verwijdert.';
        echo '<meta http-equiv="refresh" content="2; URL=admin.php">';  
    }
}else{
    echo 'Deze actie is al uitgevoerd of het is niet nodig!!';
}

但是我可以用我的其他人看到，如果他发现了什么。它在我的数据库中找不到行，但其中有一些过去 2 个月的行。

我的查询有问题吗？

格兹

4

4 回答 4

2

你必须引用你在mysql中搜索的字符串，目前你正在搜索month_value-year_value，一个负整数

于 2013-09-09T12:40:05.457 回答
1

你有一些问题...

当绑定数据发生变化时，不必为每次迭代都准备好语句：

if($hoeveel != 0){
    $resultDEL = $pdo->prepare('DELETE FROM photos WHERE id = :id'); //put it here
    foreach($result as $row){
        unlink($row["thumb"]);
        unlink($row["location"]);

        $resultDEL->execute(array(':id' => $row["id"]));

        echo 'De vorige 2 maanden zijn verwijdert.';
        echo '<meta http-equiv="refresh" content="2; URL=admin.php">';  
    }
}else{
   echo 'Deze actie is al uitgevoerd of het is niet nodig!!';
}

但是您也可以直接删除它们：

$result = $pdo->prepare("DELETE FROM photos WHERE geupload = $AFGELOPENMAAND OR geupload = $AFGELOPENMAAND2");
$result->execute();

经验法则：当您从 DB 中读取某些内容，并且必须处理每一行以便生成 DB 查询时，100 个案例中有 99 个您可以完全在 DB 中完成，从而节省大量带宽和时间。

编辑AdrianBR 发现问题...不要将日期存储为字符串。数据库系统擅长处理自己定义的数据类型，使用它们。

于 2013-09-09T12:38:58.233 回答
1

替换这个：

$result = $pdo->prepare("SELECT * FROM photos WHERE geupload = $AFGELOPENMAAND OR geupload = $AFGELOPENMAAND2");
$result->execute();

有了这个

$result = $pdo->prepare("SELECT * FROM photos WHERE geupload = '".$AFGELOPENMAAND."' OR  geupload = '".$AFGELOPENMAAND2."'");
$result->execute();
于 2013-09-09T12:45:38.207 回答
0

我的建议是，将您的“geupload”字段更改为 DATETIME（或 DATE）字段而不是 VARCHAR，使用起来要容易得多，并且数据库可以像处理真实日期而不是字符串一样处理该字段（如前所述）。

您可以像这样删除两个月之前的行：

DELETE FROM photos WHERE TIMESTAMPDIFF(MONTH, geupload, NOW()) > 2

您可以像这样轻松地将当前时间插入或更新到此字段中：

INSERT INTO photos (.., .., geupload) VALUES (.., .., NOW())

UPDATE photos SET geupload = NOW()
于 2013-09-09T12:50:34.510 回答