Sorting array of folder list

Peuplarchie

Good day to you all,
I'm working on a script thta list folder in a specific folder.
My problem is that it won't sort.

Here is my code :




	echo '<div style="width:620px; margin-left:auto; margin-right:auto; padding:0px; ">';
$dirr="Categories/".$cat."/".$val."/";
$dh  = opendir($dirr);
while (false !== ($filename = readdir($dh))) {
if ($filename != "." && $filename != "..") { // remove the move up directory commands. (. and ..)
    $files[] = $filename;

echo '<div style=" width:620px;  margin-left:auto; margin-right:auto; color:#FFFFFF; ">';
echo '<div style=" width:520px; margin-left:auto; margin-right:auto; color:#FFFFFF; float:left;"><img src="snippet_icon.png" border="0"/> '.$filename.'</div><br style="font-size:2px;"/>';
echo '<div style=" width:100px;  margin-left:auto; margin-right:auto; color:#FFFFFF; float:right; text-align:center;">';
echo '<a href="view_snippets.php" onclick="load(\'view_snippets.php?cat='.$cat.'&lang='.$val.'&code='.$filename.'\',\'page\');return false;" title="View Snippet '.$filename.'"><img src="view_icon.png" border="0" title="View Snippet '.$filename.'"/></a>';
echo '</div></div><br>';
}
}
sort ($files); // sort the file list in the array


echo "</div>";

echo "<br/><br/>";

Thanks!

bradgrafelman

Looks fine to me... why do you say that it isn't working? Once you sort the folder list, you never do anything with it..

EDIT: Also, you could simplify your code a lot if you just used [man]glob/man. For example, all of this:

$dh  = opendir($dirr); 
while (false !== ($filename = readdir($dh))) { 
if ($filename != "." && $filename != "..") { // remove the move up directory commands. (. and ..) 
    $files[] = $filename; 

// ...
}

sort($files)

could be replaced by this:

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

Weedpacket

bradgrafelman wrote:
Looks fine to me... why do you say that it isn't working? Once you sort the folder list, you never do anything with it..

If you look waaaaay off to the right there's a few places where $filename is being echoed.

Of course, this happens while the array is still being constructed, which is sorted afterwards.

So, Peuplarchie, the solution is the same one that stops your clothes getting soaked every time you take a shower: Don't construct the array and display it at the same time then sort it; construct the array then sort it then display it.