Menu
Community /Pin to ProfileBookmark
@NogDogOct 01.2010 — #Do the strtotime() before the loop, and assign the result to a variable; then use that variable within the loop, so that you don't have to call the strtotime() function on every iteration.
Also, I might try using readdir() (instead of scandir) and compare its performance. It would not require populating an array with every file, so it [i]might[/i] work better with "thousands" of files -- or it might not, depending on available memory and other factors beyond my knowledge. ?
Delete Files Efficientlly
Hi All,
I have the following code which I intend to run once or twice a week to delete old files from my web servers uploads folder.
[code=php]$dir = ‘../uploads’;
if (is_dir($dir)) $files = scandir($dir);
foreach ($files as $file) {
if ($file == ‘.’ || $file == ‘..’) continue;
if (fileatime($dir . ‘/’ . $file) < strtotime(‘-90 days’)) unlink($dir . ‘/’ . $file);
}
I am interested in getting the best performance possible from this script as there are many thousands of files in the uploads folder.
I this code the most efficient way of getting the job done?
—
Thanks,
dai.hop
Sign in
to post a comment2 Comments(s) ↴
0
@NogDogOct 01.2010 — #Do the strtotime() before the loop, and assign the result to a variable; then use that variable within the loop, so that you don't have to call the strtotime() function on every iteration.Also, I might try using readdir() (instead of scandir) and compare its performance. It would not require populating an array with every file, so it [i]might[/i] work better with "thousands" of files -- or it might not, depending on available memory and other factors beyond my knowledge. ?