dynamic array population in a while loop

[sethadam1]

This function returns an error. I can't figure out why, but I get this OVER AND OVER AND OVER:

Warning: 1 is not a valid Directory resource in /home/firsttube.com/public_html/code.php on line 52

Warning: 1 is not a valid Directory resource in /home/firsttube.com/public_html/code.php on line 58

I sense this is a very simple piece I'm overlooking, probably with defining the array as empty, but it's also the first time I've written an array that's being dynamically populated (I've tried array_push() but couldn't get that to work either. Help is VERY MUCH appreciated if anyone gets a second!

-Adam
[e-mail]

<?php

$codearray = array ();
$handle=opendir('./code/');
echo "Available Scripts:<p>";
while (false !== ($file = readdir($handle))) {
if ($file != "." && $file != ".." && $file != "index.php") {
//add each file to an array so I can sort
// foreach ( $file as $codefile ) {
$codearray[] = $codefile;
}
closedir($handle);
}
//sort results of new array
sort($codearray);

//one last foreach loop, this time to display results of new array
foreach ( $codearray as $showcode ) {
echo "<a href=display.php?file=$showcode><font color=\"black\">";
$filename = (str_replace("_"," ",$showcode));
$filename = (str_replace(".","(",$showcode));
echo $filename .")</font></a><br>\n";
}

?>

[van__]

Try this bit instead...
<?
//one last foreach loop, this time to display results of new array
foreach ( $codearray as $showcode=>$file) {
echo "<a href=display.php?file=$file>";
$filename = (str_replace("_"," ",$file));
$filename = (str_replace(".","(",$file));
echo $filename .")</a><br>\n";
}
?>

if that doesn't work, take a look at the array you are creating...
<?
print_r($codearray);
?>

[sethadam1]

Thanks for the really quick response, but the problem appears to be higher up in the code. I guess I'm not sure if I'm declaring an empty array too soon, since even after I comment out the foreach loops, I get the same error, and it's always on these two lines:

(line 52)
$codearray = array();

(line 58)
$codearray[] = $file;

I must not be getting this array to work properly...doesn't this look like valid syntax?

-Adam

Van wrote:

Try this bit instead...
<?
//one last foreach loop, this time to display results of new array
foreach ( $codearray as $showcode=>$file) {
echo "<a href=display.php?file=$file>";
$filename = (str_replace("_"," ",$file));
$filename = (str_replace(".","(",$file));
echo $filename .")</a>\n";
}
?>

if that doesn't work, take a look at the array you are creating...
<?
print_r($codearray);
?>

[van__]

Well, let's rip this apart...
<?php
//here you declare an empty array...
$codearray = array ();

//we open a directory...
//you could point to 'code' as itself...
$handle=opendir('./code/');
echo "Available Scripts:<p>";

//here we have some probs with the use of
//false as well as an invalid operator '!=='
/
while (false !== ($file = readdir($handle))) {
/
//Try..
while ($file = readdir($handle)) {

//now you look for files...
if ($file != "." && $file != ".." && $file != "index.php") {
//you had this commented out...
//add each file to an array so I can sort
// foreach ( $file as $codefile ) {

/
I haven't seen a var called $codefile anywere except that commented code... this MUST be wrong...
$codearray[] = $codefile;
/

//try
array_push($codearray,$file);

//and close the if loop...
}

//we close the dir.
closedir($handle);
}

//sort results of new array
sort($codearray);

[Anon]

Van -

I GOT IT. Thanks for your help. As for the variable $codefile, that was a typo when I changed some variables to make it more logical for this forum. I won't make that mistake again next time!

Thanks again,

-Adam