[RESOLVED] PHP if statements

Mr_Moe

I am a 60 year newbie trying to learn some simple PHP.

I am having an trying to test for 3 different characters that someone enters into a post form. If they enter anything but the AA or O or JR I want the message to be display otherwise continue on to next code lines. If tried elseif and else statements and just gave up. I know one of you smart and young users can help me out.

if ($POST[Status] != "AA")
{
if ($POST[Status] != "O")
{
if ($POST[Status] != "JR")
{
if ($POST[Status] == "")
{
$message_add = "USDF Type field must be either AA=Adult Amateur or O=Open or JR=Junior";
extract($_POST);
include("add_member_section_form.php");
exit();
}
}
}
}

KateDaring52

Thank you, this is good news for new visitors.

Watch Horrible Bosses Online Free

dagon

couple of different ways do do this, but i would go with:

if ($_POST['Status'] != "AA" && $_POST['Status'] != "O" && $_POST['Status'] != "JR" ){
	$message_add = "USDF Type field must be either AA=Adult Amateur or O=Open or JR=Junior";
	extract($_POST);
	include ("add_member_section_form.php");
	exit();
}

&& = AND

with only 3 allowable options i would give the sue a select list to chose from, then they will have to pick one, rather than (I'm guessing) a txt input box.

laserlight

For three, I would already consider the in_array method, e.g.,

if (!in_array($_POST['Status'], array("AA", "O", "JR"))) {
    $message_add = "USDF Type field must be either AA=Adult Amateur or O=Open or JR=Junior";
    extract($_POST);
    include ("add_member_section_form.php");
    exit();
}

I am a little uncomfortable with your use of extract here as it essentially create the same potential security loopholes as having register_globals be on. If you really do want to use extract like that, use EXTR_SKIP and consider using a prefix.

Mr_Moe

Thanks to both of you for your replys.