[Amerowolf]

Ok, so. I have this code that prints the content of a directory to a website as links to webpages (since that's all that is in the directory). What I am having problems with is ordering the list alphabetically.

My original code:

<?php

 $dir="../zpress/pages"; // Directory where files are stored

 if ($dir_list = opendir($dir))
 {
 while(($filename = readdir($dir_list)) !== false)
 if(!is_dir($filename))
 {
  ?>
  <p><a href="../zpress/pages/<?php echo $filename; ?>"><?php echo $filename;
  ?></a></p>
  <?php
  }
  closedir($dir_list);
  }

  ?>

I also got some advice from a friend, and I created the code below, but it ends up just showing nothing on the page and I've not been able to troubleshoot it myself.

$the_files = array();
while(($filename = readdir($dir_list)) !== false) {
    if(!is_dir($filename)) {
        array_push($the_files,$filename);
    }
}
sort($the_files);
foreach($the_files as $the_file) : ?>
   <p><a href="../zpress/pages/<?php echo $filename; ?>"><?php echo $filename;?></a></p>
<?php endforeach; ?> 

[Macjohn]

You may wanna take a look at THIS and ALSO THIS...

for the second piece of code, here is a "correction" that should make it work...

$the_files = array();
while(($filename = readdir($dir_list)) != false) {
    if(!is_dir($filename)) {
        array_push($the_files,$filename);
    }
}
sort($the_files);
foreach($the_files as $the_file) ;
   echo '<p><a href="../zpress/pages/'.$filename.'"><'echo $filename.'></a></p>';
endforeach;

...Almost forgot...what do you use to code, an IDE I mean, NOTEPAD++, Eclipse, NetBeans.... try using something like NetBeans or so...It helps you alot with code errors, like the ":" instead of ";" in your second code and stuff like that.

This post has been edited by Macjohn: 12 April 2012 - 02:30 AM

[4D1]

@MacJohn the ':' is not an error it is PHP's alternate syntax.

This post has been edited by 4D1: 12 April 2012 - 02:34 AM

[Macjohn]

Corrected it!
Thanks

[4D1]

sorry m8 there is still an error, either the script wont run because of an unexpected endforeach; or the echo statement will never execute because foreach() is followed by a terminator ;

This post has been edited by 4D1: 12 April 2012 - 03:01 AM

[e_i_pi]

To the OP, it shows nothing on the page because you are referencing the wrong variable. $filename is a temporary variable used in the while, you should be using $the_file, like this:

$the_files = array();
while(($filename = readdir($dir_list)) !== false) {
    if(!is_dir($filename)) {
        array_push($the_files,$filename);
    }
}
sort($the_files);
foreach($the_files as $the_file) : ?>
   <p><a href="../zpress/pages/<?php echo $the_file; ?>"><?php echo $the_file;?></a></p>
<?php endforeach; ?> 

...or this...

$the_files = array();
while(($filename = readdir($dir_list)) !== false) {
    if(!is_dir($filename)) {
        array_push($the_files,$filename);
    }
}
sort($the_files);
foreach($the_files as $the_file) {
    echo '<p><a href="../zpress/pages/' . $the_file . '">' . $the_file . '</a></p>';
}

...which I tend to find more readable when mixing languages.