在PHP中删除所有超过2天的文件的正确方法

2021-12-20 00:00:00 file caching php

只是好奇

        $files = glob(cacheme_directory()."*");
        foreach($files as $file)
        {
            $filemtime=filemtime ($file);
            if (time()-$filemtime>= 172800)
            {
                unlink($file);
            }
        }

我只想确定代码是否正确.谢谢.

I just want to make sure if the code is correct or not. Thanks.

推荐答案

你应该添加一个 is_file() 检查，因为 PHP 通常会列出 .和 ..，以及可能驻留在您正在检查的目录中的子目录.

You should add an is_file() check, because PHP normally lists . and .., as well as sub-directories that could reside in the the directory you're checking.

此外，正如此答案所建议的，您应该用更具表现力的符号替换预先计算的秒数.>

Also, as this answer suggests, you should replace the pre-calculated seconds with a more expressive notation.

<?php
  $files = glob(cacheme_directory()."*");
  $now   = time();

  foreach ($files as $file) {
    if (is_file($file)) {
      if ($now - filemtime($file) >= 60 * 60 * 24 * 2) { // 2 days
        unlink($file);
      }
    }
  }
?>

或者，您也可以使用 DirectoryIterator、如本答案所示.在这种简单的情况下，它并没有真正提供任何优势，但它会是 OOP 方式.

Alternatively you could also use the DirectoryIterator, as shown in this answer. In this simple case it doesn't really offer any advantages, but it would be OOP way.

相关文章