im a bit stuck with an IF Statement..

[include_me]

Hi There,

Im trying to get this if statement to work but it keeps coming up with the following error when the page tries to load, I want it to check if a code has been entered correctly and if it has then to write the e-mail address to the txt file.

can anyone point out what im doign wrong?

The error message i am receiving is:

Parse error: syntax error, unexpected T_LOGICAL_AND in /home/public_html/members-area.php on line 20

I have tried putting && and AND but these just bring up different errors each time.

<?
$valcode=$_POST['vcode'];

$fn = "email.txt";
$file = fopen($fn, "a+");
$size = filesize($fn);

if($_POST['email']) and ($valcode=="1902")

{ 
	fwrite($file, $_POST['email']."; ");
	fclose($file);

	$message ="<p class='number'>Thank you, you will receive an e-mail with your user login and password shortly. Please check your junk mail if you do not receive an e-mail.</p>";
	} else {
	$message ="<p class='number'>Your validation code did not match that on our records, please re-enter and try again, if you keep encountering this problem please contact us.</p>";
}
?>

IF anyone could help, it would be great!

Thanks
Chris

[Houdini]

if($POST['email']) and ($valcode=="1902")

perhaps not closing the if before the and or &&

if($_POST['email'] && $valcode=="1902")

[ClarkF1]

Take out the open bracket before $valcode in the above code - it might throw up an error message

[Installer]

if($POST['email']) and ($valcode=="1902")

The conditions for an "if" block must be inside parentheses. Your first closing parenthesis makes PHP think you're done with the conditions; that's why the "and" is unexpected. Houdini's answer is almost right, but he forgot to remove the second opening parenthesis (as ClarkF1 says). Do this:

if ($_POST['email'] && $valcode == "1902")
// or
if (($_POST['email']) && ($valcode == "1902"))

[thorpe]

It should be...

if ($_POST['email'] and $valcode=="1902")

[thorpe]

Damn... too slow 🙂

[Houdini]

Note I edited my post a little late (still in a time warp)

[include_me]

Thanks everyone that worked, i knew it was something small like that :S

Can i be cheeky and ask 1 more question, when i load the page up the error message assigned to $message comes up before anyone clicks the submit button is there a way this will only appear if one of the if conditions isnt met?

I think I can see why its doing it but I just cant see a way around getting it to only come up when one of the fields isnt filled in. :rolleyes:

Thanks
Chris

[include_me]

Anyone? its really starting to annoy me now! 🙁

[Houdini]

Are you checking to see if that field is set ot empty or anything like that FIRST? I do not see it in your code so I will be Bold and say no.

[include_me]

no im not, where would i put that? another IF statement?

[Houdini]

It would be some along the line of

if($_POST['submit'] = somevalue{
//proceed with the process
}
  else{
  //return to form or whatever
  }

or you need to check somehow if the user actually clicked the submit button and that would be the first thing that you check for otherwise do something else.

[include_me]

Thanks! I just sorted it as you posted it, did it a slightly diff way but it works now! thanks for all your help!

Chris