The correct way to delete all files older than 2 days in PHP

Question

Just curious

        $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.

please use the search function before asking http://stackoverflow.com/search?q=delete+files+older+php — Gordon, Jan 23 '12 at 00:00
possible duplicate of [How to delete files from directory based on creation date in php](http://stackoverflow.com/questions/2205738/how-to-delete-files-from-directory-based-on-creation-date-in-php) — Gordon, Jan 23 '12 at 00:02
It's not duplicate. The one I wrote is simple and I only want to ask if this is correct or not which is a much easier question. — user4951, Jan 23 '12 at 01:49
@JimThio asking whether it works is not a real question. Write a UnitTest or simply try it in a test folder. And yes, this is a duplicate of the linked one. It covers exactly the same grounds. And there is more duplicates in the linked search. — Gordon, Jan 23 '12 at 08:28

score 127 · Accepted Answer · edited May 23 '17 at 11:47

127

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);
      }
    }
  }
?>

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.

edited May 23 '17 at 11:47

Community

1
1

answered Jan 22 '12 at 23:53

buschtoens

7,435
9
37
57

7

Works like a charm, but why u people always exclude brackets { } I love them :D i can't do without them. Otherwise code is naked – Luka Jan 21 '17 at 12:45
You really need to add the backets. This code is not readable and tidy. – Gfra54 Mar 29 '17 at 09:50
6

I wrote this answer back when I had a short-lived liaison with Python. Brackets have been added. ;) – buschtoens Apr 19 '17 at 06:29
If you are on php5.3+, check Maksim.T's answer. – Toto Sep 05 '18 at 13:36

score 74 · Answer 2 · edited Feb 03 '17 at 18:46

74

The easiest way is by using DirectoryIterator:

<?php
if (file_exists($folderName)) {
    foreach (new DirectoryIterator($folderName) as $fileInfo) {
        if ($fileInfo->isDot()) {
        continue;
        }
        if ($fileInfo->isFile() && time() - $fileInfo->getCTime() >= 2*24*60*60) {
            unlink($fileInfo->getRealPath());
        }
    }
}
?>

edited Feb 03 '17 at 18:46

Mojtaba

4,536
4
19
38

answered Dec 12 '13 at 15:58

Stefanie Janine Stölting

741
5
2

7

This works for me, but is slighlty faster if you just calculate time() and 2*24*60*60 once before the while loop and use variables in the loop. – Lisa DeBruine Apr 06 '15 at 11:51
I'd just like to say thanks. This honestly should be the chosen answer. – Bravo Delta May 03 '16 at 23:54
works great and learned about the DirectoryIterator which is awesome! – William Howley Nov 27 '16 at 23:22
1

Is the `isDot()` check necessary? Won't the `isFile()` check suffice for that? – ashnazg Jun 28 '18 at 15:38
1

Note: If your'e using WIndows `getCTime()` is the file creation time, so you might want to use `getMTime()` if you want to use last modified time instead of creation time. – IMB Jul 21 '18 at 11:43

score 20 · Answer 3 · edited Jun 20 '20 at 09:12

Another simplier and more modern way, using FilesystemIterator.

I'm using 'logs' directory as an example.

$fileSystemIterator = new FilesystemIterator('logs');
$now = time();
foreach ($fileSystemIterator as $file) {
    if ($now - $file->getCTime() >= 60 * 60 * 24 * 2) // 2 days 
        unlink('logs/'.$file->getFilename());
}

Main advantage is: DirectoryIterator returns virtual directories "." and ".." in a loop. But FilesystemIterator ignores them.

score 12 · Answer 4 · answered Apr 15 '20 at 05:06

12

I reckon this is much tidier and easier to read and modify.

$expire = strtotime('-7 DAYS');

$files = glob($path . '/*');

foreach ($files as $file) {

    // Skip anything that is not a file
    if (!is_file($file)) {
        continue;
    }

    // Skip any files that have not expired
    if (filemtime($file) > $expire) {
        continue;
    }

    unlink($file);
}

answered Apr 15 '20 at 05:06

Leon

121
1
3

Some people don't prefer guard clause... – 4unkur Nov 24 '21 at 04:42

score 5 · Answer 5 · edited Jun 16 '17 at 14:00

5

/* Delete Cache Files Here */
$dir = "cache/"; /** define the directory **/

/*** cycle through all files in the directory ***/
foreach (glob($dir."*") as $file) {
//foreach (glob($dir.'*.*') as $file){

/*** if file is 24 hours (86400 seconds) old then delete it ***/
if (filemtime($file) < time() - 172800) { // 2 days
    unlink($file);
    }
}

Hope it help you.

edited Jun 16 '17 at 14:00

Martin

20,858
7
60
113

answered Jun 16 '17 at 13:34

Lachit

61
1
4

score 4 · Answer 6 · answered Jan 22 '12 at 23:44

4

Looks correct to me. I'd just suggest you replace 172800 with 2*24*60*60 for clarity.

answered Jan 22 '12 at 23:44

Niet the Dark Absol

311,322
76
447
566

27

or just add `// 2 days` beside it, for even more clarity – mrlee Jan 22 '12 at 23:46
1

Or just create constants at the beginning of your project like DAY_IN_SECONDS, WEEK_IN_SECONDS, etc. lol – Nicholas Cardot Jun 13 '20 at 01:50

score 3 · Answer 7 · answered Nov 10 '17 at 20:13

Here is an example of how to do it recursively.

function remove_files_from_dir_older_than_x_seconds($dir,$seconds = 3600) {
    $files = glob(rtrim($dir, '/')."/*");
    $now   = time();
    foreach ($files as $file) {
        if (is_file($file)) {
            if ($now - filemtime($file) >= $seconds) {
                echo "removed $file<br>".PHP_EOL;
                unlink($file);
            }
        } else {
            remove_files_from_dir_older_than_x_seconds($file,$seconds);
        }
    }
}

remove_files_from_dir_older_than_x_seconds(dirname(__file__).'/cache/', (60 * 60 * 24 * 1) ); // 1 day

Excellent you save my time, thank you so much – Rachid Loukili Sep 16 '21 at 17:38 — Rachid Loukili, Sep 16 '21 at 17:38

score 2 · Answer 8 · answered Jan 23 '12 at 00:02

2

Be aware that you'll run into problems if you have a very large number of files in the directory.

If you think this is likely to affect you, consider using a lower level approach such as opendir.

answered Jan 23 '12 at 00:02

John Carter

52,342
26
107
142

1

What kinds of problems might we be running into? – vertigoelectric Jun 19 '13 at 20:05
@vertigoelectric out of memory problems – DarkSide Apr 15 '14 at 11:32

score 1 · Answer 9 · answered May 07 '21 at 10:43

@maksim-t answer can be improved a bit:

Calculate the oldest "allowed" from the start, instead of calculating it on every iteration.
Use Modification time, instead of creation time, because otherwise you may remove files, that were updated recently.
Use getPathname instead of getFilename, so that there will be not need for concatenation of the path.

#Initiate iterator
$fileSI = new FilesystemIterator($this->files);
#Get the oldest allowed time
$oldest = time() - ($maxFileAge * 86400);
#Iterate the files
foreach ($fileSI as $file) {
    #Check time
    if ($file->getMTime() <= $oldest) {
        #Remove the file
        @unlink($file->getPathname());
    }
}

score 0 · Answer 10 · answered Jun 21 '17 at 09:57

/** It deletes old files.
 *  @param string $dir Directory
 *  @param int $secs Files older than $secs seconds
 *  @param string $pattern Files matching $pattern
 */
function delete_oldfiles($dir,$secs,$pattern = "/*")
{
    $now = time();
    foreach(glob("$dir$pattern") as $f) {
      if (is_file($f) && ($now - filemtime($f) > $secs)) unlink($f);
    }
}

score 0 · Answer 11 · answered Jan 03 '21 at 17:55

(time()-filectime($path.$file)) < 172800 //2 days

If the current time and file are changed time are within 172800 seconds of each other, then.

 if (preg_match('/\.pdf$/i', $file)) {
     unlink($path.$file);
 }

Well, if its there still any confusion you just need to change the operator from the first line of code.

The correct way to delete all files older than 2 days in PHP

11 Answers11

Linked