IF() statement giving me Fits! Help!

[mrhinman]

I've got this SELECT in an HTML form (multipart/form-data) passing on a value called agemail:

<select name="agemail" class="bodyText" id="agemail">
<option value="[e-mail]" selected>Toby Piper</option>
<option value="[e-mail]">Linda Terry</option>
<option value="[e-mail]">Shawna Hutchins</option>
<option value="[e-mail]">Luann McCluskey</option>
<option value="[e-mail]">Bobby Fisher</option>
<option value="[e-mail]">Joel Hamlin</option>
<option value="[e-mail]">Brian Clark</option>
</select>

Then, in the PHP on the following page, I have it doing this:

$agemail = $_POST['agemail'];
if ( substr($agemail,0,4) == "toby" ) {$agent = "Toby Piper";} else {$agent="Pecan Valley Real Estate";}
if ( substr($agemail,0,4) == "luan" ) {$agent = "Luann McCluskey";} else {$agent="Pecan Valley Real Estate";}
if ( substr($agemail,0,4) == "shaw" ) {$agent = "Shawna Hutchins";} else {$agent="Pecan Valley Real Estate";}
if ( substr($agemail,0,4) == "lind" ) {$agent = "Linda Terry";} else {$agent="Pecan Valley Real Estate";}
if ( substr($agemail,0,4) == "bobb" ) {$agent = "Bobby Fisher";} else {$agent="Pecan Valley Real Estate";}
if ( substr($agemail,0,4) == "joel" ) {$agent = "Joel Hamlin";} else {$agent="Pecan Valley Real Estate";}
if ( substr($agemail,0,4) == "bria" ) {$agent = "Brian Clark";} else {$agent="Pecan Valley Real Estate";}

before it adds both fields (agemail and agent) to the database. BUT, my AGENT field is not coming through. I get the ELSE every time!!! I'm going crazy here. Am I just leaving something out???

[Piranha]

To explain what is happening look at the first if-statement. If that is true you will get agent Toby Piper. Then look at the next if-statement, and it is false. Then the agent will become Pecan Valley Real Estate.

A better way to do this would be to use [man]switch[/man], a link to another page describing this is here.

[mrhinman]

Aha! Yup, I guess I wasn't paying attention to the false values! Duh! Thanks for the extra "eyes" on it!

[mrhinman]

You mean like this:

switch (substr($agemail,0,4))

{
case toby:
$agent = "Toby Piper";
break;

case luan:
$agent = "Luann McCluskey";
break;
default:
$agent = "Pecan Valley Real Estate";
}

[Installer]

Except, of course, put quotes around the first name strings.

Here's a way that might make code repair, modification, and expansion easier:

$agent_arr = array('toby' => 'Toby Piper',
                   'luann' => 'Luann McCluskey',
                   'shawna' => 'Shawna Hutchins');
                   //etc.

list($email_name) = explode('@', $_POST['agemail']);
if (array_key_exists($email_name, $agent_arr)) {
    $agent = $agent_arr[$email_name];
} else {
    $agent = 'Pecan Valley Real Estate';
}

Edit: BTW, you should edit your first post to change the email domain to "@example.com", to avoid an unwanted mail server influx.