[Resolved] ifelse statement

[fr8]

The code is in my template.php. When the page is displayed with just urlhere/template.php the include("view.php"); still displays not the echo "working"; which is what i want to see.

Would my life be more easy if I did a switch instead?

$menulink=$_GET[menulink];

if($menulink=='Search')
{
include("search.php");
}
elseif($menulink='View')
{
include("view.php");
}
else
{
echo "working";
}

[mcgruff]

There's a minor typo - you should have "==" not "=" in the elseif($menulink = 'View') line.

[fr8]

Originally posted by mcgruff
There's a minor typo - you should have "==" not "=" in the elseif($menulink = 'View') line.

oh my god im blind. thank you for pointing that out.

Now that is fixed. would I be ok using the ifelse statement?

In time it will get bigger. As of right now its going to have 8 elseif in it. Should I go with the switch function instead?

[mcgruff]

You can use either. I'd personally tend to use a switch case for something simple like this with a long list of possible values.

[Weedpacket]

Yeah, go with the switch if you find yourself going elseif($thesamething==over and over)

Less typing and fewer chances for typos, for one thing 🙂

switch($menulink)
{
case 'Search': include("search.php");
    break;
case 'View': include("view.php");
    break;
default: echo "working";
}

[fr8]

well its settled then. two post in a row saying the switch would be better. that has to mean something. 🙂
so thats what im going to do. thanks for the help. ill mark this one resolved.