random background image

[Anon]

I want to put a random background image in a specific spot on my page. I used a style sheet to have the image not be repeated. I am unsure of what code to put as the background.

Heres what i tried with no success.
background=\"$images[DisplayRandomImage()]\">\n");

Here is the code for the page.
<LINK REL=StyleSheet HREF="index4.css" TYPE="text/css" TITLE="Archanix Style">
</head>
<?php
$DIRNAME = "chicks/";
$images = array();
$file_handle=opendir($DIRNAME);
$x = 0;
while ($file = readdir($file_handle)) {
if($file == '.' || $file == '..') { continue; }
if(is_dir($DIRNAME.$file)) { continue; }
$images[$x] = $DIRNAME . $file;
$x += 1;
}
closedir($file_handle);

function DisplayRandomImage() {
global $images;
if (count($images) > 0) {
srand((double)microtime()*1000000);
$rnd_img = rand(0,(count($images) - 1));
return $rnd_img;
}
}
print("bgcolor=\"#FFFFFF\" text=\"#000000\" leftmargin=\"0\" topmargin=\"0\" marginwidth=\"0\" marginheight=\"0\" background=\"$images[DisplayRandomImage()]\">\n");
?>

[mithril]

Your main problem is the way you're calling the DisplayRandomImage() method. As you've written it, PHP will interpret it as a string instead of a method call. Try doing something like this:

&nbsp;&nbsp;background=\"" . $images[DisplayRandomImage()] . "\">\n");

You could also add a $count parameter to the DisplayRandomImage() method's argument list and get rid of that ugly global call. You'd then change

&nbsp;&nbsp;background=\"" . $images[DisplayRandomImage()] . "\">\n");

to

&nbsp;&nbsp;background=\"" . $images[DisplayRandomImage(count($images))] . "\">\n");

and

&nbsp;&nbsp;if (count($images) > 0) {

to

&nbsp;&nbsp;if ($count > 0) {

Other than the method call problem, your logic looks sounds. HTH.

geoff