Hello,

I have a little code to fetch all the images out of a directory and display them on the page. However I would like to order them ascending image001, image002, image003.

Can anybody help me to achieve this?

PHP Code:

<?php

$dir = "images/portfolio/architecturephotography/";

if ($opendir = opendir($dir))
{
while (($file = readdir($opendir)) !== FALSE)
{
if ($file!="."&&$file!="..")
echo "<img src='$dir/$file' /><br />";
}
}
?>

Would be much appreciated.

You're going to have to insert the files into an array and then sort them:

PHP Code:

$files = array();

$dir = "images/portfolio/architecturephotography/";

if ($opendir = opendir($dir))
{
     while (($file = readdir($opendir)) !== FALSE)
     {
          if ($file!="."&&$file!="..")
               $files[] = $file;
     }
}

natsort($files);

foreach($files as $file)
{
     echo "<img src='$dir/$file' /><br />";
} 


Thanks a lot Matt, worked a treat.

Don't forget to close the handle that you opened.

if ($opendir = opendir($dir))
{
     while (($file = readdir($opendir)) !== FALSE)
     {
          if ($file!="."&&$file!="..")
               $files[] = $file;
     }
     closedir($opendir);
}

Is this just good practice Ronnie?

Originally Posted by jamiek

Is this just good practice Ronnie?

Um, yes.

Technically the opendir() function returns a handle to an object in memory.

You can use that handle to perform other functions, such as readdir($handle).

When you are done with the handle, discard it by closing it. This removes the object from memory.

If not discarded, the object remains in memory. Every call to this part of the code creates another object, which in turn gets added into memory. And so on.

Imagine the page that this portion of code resides on all of a sudden winds up on the front page of Digg. What do you think will happen with a surge of page hits and that handle is not closed properly?

Ah I see like a memory leak, many thanks.

Originally Posted by jamiek

Ah I see like a memory leak, many thanks.

No. Unreleased memory.

And to be more explicit in your code with variable naming:

PHP Code:

if ($dir_handle = opendir($dir))
{
     while (($file = readdir($dir_handle)) !== FALSE)
     {
          if ($file!="."&&$file!="..")
               $files[] = $file;
     }
     closedir($dir_handle);
}