if, else if, else

[Anon]

I guess I am thinking to much in vbscript. Can someone help me figure this one out? I am defining the variables at the top of the page - don't ask me why... It's just an automated process that I have going...

<?
$hotel='h';
?>

<?php
if (hotel='h')
{
include ("http://mysite.com/hotelinfo.php")
}

else if (hotel='c')
{
include ("http://mysite.com/condoinfo.php")
}

else
{
include ("http://mysite.com/noinfo.php")
}

end
?>

[mattw]

<?
$hotel='h';
?>

<?php
if (hotel=='h')
{
include ("http://mysite.com/hotelinfo.php")
}

elseif (hotel=='c')
{
include ("http://mysite.com/condoinfo.php")
}

else
{
include ("http://mysite.com/noinfo.php")
}

?>

For comparision you use a == not a =. The elseif doesn't have a space, and the end wasn't necessary.

Hope that helps,
Matt Wade
codewalkers.com

[Anon]

you need two = for a compare and a $ to specify a variable. like so:

if($hotel == 'h')

[Anon]

actually the space between the else and if do not make a difference, and is really how it should look.

[Anon]

I tried:

<?php
if ($hotel=='h')
{
include ("http://mysite.com/hotelinfo.php")
}
else if ($hotel=='c')
{
include ("http://mysite.com/condoinfo.php")
}
else
{
include ("http://mysite.com/noinfo.php")
}
?>

and recieved a parse error. Any Ideas the problem?

btw: I tried this also:

<?php
if (hotel=='h')
{
include ("http://mysite.com/hotelinfo.php")
}
else if (hotel=='c')
{
include ("http://mysite.com/condoinfo.php")
}
else
{
include ("http://mysite.com/noinfo.php")
}
?>

[Anon]

no spaces! you have to keep your if statements and include calls and such together. it should look like this:

<?php
if($hotel=='h')
{
include("http://mysite.com/hotelinfo.php")
}
else if($hotel=='c')
{
include("http://mysite.com/condoinfo.php")
}
else
{
include("http://mysite.com/noinfo.php")
}
?>

[Anon]

I am still getting a parse error. Anything else that might be wrong with the code? php3 is installed, could this be php4 code?

[Anon]

sorry, wasn't watching closely enough. you need to remember those semi-colons too. like this:

<?php
if($hotel=='h')
{
include("http://mysite.com/hotelinfo.php");
}
else if($hotel=='c')
{
include("http://mysite.com/condoinfo.php");
}
else
{
include("http://mysite.com/noinfo.php");
}
?>

[Anon]

Figured it out!

I was missing ";"

Thanks all!

[Anon]

What do you mean "should" look? What's wrong with elseif? It's a perfectly reasonable keyword in PHP, and makes indentation more consistent and keeps related comparisons at the same conceptual level:

if()
{
....code
....code
}
elseif()
{
....code
....code
}
else
{
....code
....code
}

Instead of

if()
{
....code
....code
}
else
{
....if()
....{
........code
........code
....}
....else
....{
........code
........code
....}
}

Of course, it's a matter of personal preference. I'm not saying you SHOULD use elseif.

[maxholman]

If you're simply comparing a single variable to multiple possible values, a switch statement is even nicer...

switch($value){

case "a":
//do stuff
break;

case "b":
//more stuff
break;

case "c":
//even more
break;

}

But, like you said... personal preference 🙂

Max

[Anon]

Yeah, I figured that one out to. Thanks alot!

[Anon]

well php is based on C++ which is based on C. the original sytax is:

if(something)
do this...
else if(something)
do this...
else
do this...

i know that elseif is a valid keyword and it is a matter of personal preference. i am just saying that in the beginning it looked like 'else if' and that having the space was not a syntactical error.

[Anon]

I think PHP has more Perl in its ancestry than C++ (decorated variable names, inbuilt memory management, loose typing, ...). Now Java, that's a clear C++ descendant...

'course, in Perl, it's "elsif"