Is this function written right?

[run2web]

I'm new to writing functions and this one is giving me errors. Can someone lead me in the right direction please?

<?php function Order ($order) {
if ($order == "OwnerID"){
	return $order;
	}
elseif ($order == "LastName"){
	return $order;
	}
elseif ($order == "FirstName"){
	return $order;
	}
elseif ($order == "City"){
	return $order;
	}
elseif ($order == "State"){
	return $order;
	}
elseif ($order == "Zip"){
	return $order;
	}
else ($order == "hidden"){ //line 20
	return $order;
	}
?>

I am getting this error: Parse error: parse error, unexpected '{' in /home/hagerstown1/www/HagerstownCemetery/sort.php on line 20

[buck2bcr]

the problem is missing brackets. Using large if else's makes it a little hard to keep track of the brackets. If I need to use a bunch of if else (which isn't often, you may want to take a look at using switch instead). I make sure to line up the brackets.

<?php 
function Order ($order) 
{
    if ($order == "OwnerID")
    {
        return $order;
    }
    else if ($order == "LastName")
           {
                return $order;
           }
           else if ($order == "FirstName")
               {
                   return $order;
               }
               else if ($order == "City")
                      {
                           return $order;
                      }
                      else if ($order == "State")
                             {
                                  return $order;
                             }
                             else if ($order == "Zip")
                                    {
                                         return $order;
                                    }
                                    else ($order == "hidden")
                                    { 
                                         return $order;
                                     }
}
?>

HTH

[buck2bcr]

Here is what a switch would look like, I don't think you need the breaks in there since it is returning, I just put them there to be safe.

function Order ($order)
{
    switch($order)
    {
        case "OwnerID":
            return $order;
            break;
        case "LastName":
            return $order;
            break;
        case "FirstName":
            return $order;
            break;
        case "City":
            return $order;
            break;
        case "State":
            return $order;
            break;
        case "Zip":
            return $order;
            break;
        case "hidden":
            return $order;
            break;
    }
}

HTH

[run2web]

Thank you buck2bcr! The switch statement worked beautifully. 😃

Honestly for some reason Switch statements have always intimidated me.