Simple, yet so difficult

mike0682

I'm by no stretch of the imagination a good programmer. So i copied this code from this forum, and added some line I thought might make it work. All i want is this index.php file to redirect the user to one of 4 diffrent spalsh pages, randomly, on my site. I though this might work, but i get an error return. Any suggestions?

<?php
srand((double)microtime()*1000000);
$randstring = "";
for ($i=0;$i<1;$i++) {
$randstring = $randstring.rand(1,4);
}
if ($randstring == "1") {header("location:/redhtml.htm");}
elseif ($randstring == "2") {header("location:/greenhtml.htm");}
elseif ($randstring == "3") {header("location:/bluehtml.htm");}
elseif ($randstring == "4") {header("location:/kakihtml.htm");}
exit;
?>

Thank you in advance.

nostradamus

Why do you use a string?
Try :

<?php
srand((double)microtime()*1000000);
$rand = 0;
for ($i=0;$i<1;$i++) {
$rand = rand(1,4);
}
if ($rand == 1) {header("location:/redhtml.htm");}
elseif ($rand == 2) {header("location:/greenhtml.htm");}
elseif ($rand == 3) {header("location:/bluehtml.htm");}
elseif ($rand == 4) {header("location:/kakihtml.htm");}
exit;
?>

nostradamus

What do you need

for ($i=0;$i<1;$i++) {
$rand = rand(1,4);
}

for, just use

$rand = rand(1,4);

mike0682

Ok, thanks for responding by the way, i put this and still get an error.

<?php
srand((double)microtime()*1000000);
$rand = 0;
$rand = rand(1,4);
if ($rand == 1) {header("location:/redhtml.htm");}
elseif ($rand == 2) {header("location:/greenhtml.htm");}
elseif ($rand == 3) {header("location:/bluehtml.htm");}
elseif ($rand == 4) {header("location:/kakihtml.htm");}
exit;
?>

any suggestions

jesus_hairdo

Can you tell us the error message and line number?

Anon

whats the error ?

try replacing the header bit with
include('bluehtml.htm'); ?
and/or get rid of the exit ?

now what errors ?
Al

mike0682

it dosent say an error line, just a blank page with the word "error".

mike0682

how does this look:

<?php
srand((double)microtime()*1000000);
$rand = rand(1,4);
if ($rand == 1) {include('redhtml.htm');;
} elseif ($rand == 2) {include('bluehtml.htm');
} elseif ($rand == 3) {include('greenhtml.htm');
} elseif ($rand == 4) {include('kakihtml.htm');
}
?>

bad huh?

mike0682

Hey Al, THAT WORKED!!!!!

Hurray

Thanks a million

jesus_hairdo

it does look bad, you've got 2 ; at the end of the first if statement

this might look a bit neater using the switch function instead of the ifs, so it would be

switch($rand)
{
case "1":
header("Location: /redhtml.htm");
break;
case "2":
header("Location: /bluehtml.htm");
break;
case "3":
header("Location: /greenhtml.htm");
break;
case "4":
header("Location: /kakihtml.htm");
break;
}
I've just noticed that you had no spaces between the : at the end of Location and the /. Headers can be really picky so i'm wondering if thats the problem.

Have you tried doing a full url in the Location part (ie http://server/redhtml.htm)??

jesus_hairdo

You are probably better using readfile($filename) rather than include,
as using include tries to parse $filename as part of the php file,
whereas readfile just outputs the contents of $filename and as such should be better performance wise.