Where is the extra closing bracket? - Very Easy. . .

Capoeirista

Hi There -

Am building an application and I have nested if statements and I've been chasing a bug for hours and hours and I've been working around this problem so long I can't think straight anymore. I need someone with fresh eyes to have a look at this.

I keep getting the same error - and the line number lands right in the middle of an html form.

Parse error: syntax error, unexpected '}' . . . in line 141.

It's telling me that I have an extra }, when I feel like I'm missing one. (I'm thinking I'm missing one to close the first if statement.)

I have five if statements. The very last one gives a fresh login screen. All of the others should occur if the login has been filled out. Should the first four be enclosed in the first if statement? If so, where do I put the bracket for the first? At the end of the fourth?

Here's the structure minus the code. I'm hoping that someone can see where I'm going wrong.

Thanks,

Lee

/********************* IF THE PASSWORD HAS ALREADY BEEN ENTERED *********/
if (isset($_POST['entered_Password'])) {
// MAKE POST VARIABLES INTO SCRIPT VARIABLES
// QUERY STRING
// PROCESS RESULTS

/********************* CHECK IF LOGIN INFO IS CORRECT *******************/

if ($entered_Password == $cust_Password AND (($cust_Fname != NULL) OR ($cust_Lname != NULL))) [B]{[/B]
	echo("You are logged in as $cust_Fname $cust_Lname. <br><br>");
	// REGISTER SESSION VARIABLES
[B]}[/B]

/********************* IF LOGIN INFO IS INCORRECT INCREMENT THE LOCKOUT NUMBER, AFTER THREE TRIES, THEY'RE LOCKED OUT /

else  if (($entered_Password !== $cust_Password) AND ($locked_num < 3)) [B]{[/B] 
	// SHOW THE FORM AGAIN
[B]}[/B]

/********************* IF LOCKED OUT *************************************/

else if ($locked_num > 3) [B]{[/B]
	// TELL THEM THEY'RE LOCKED OUT AND SET LOCKED FLAG IN DATABASE
[B]}[/B]

/********************* IF NO PASSWORD ENTERED YET (FRESH LOGIN SCREEN) **/

 else [B]{[/B] 
	// GIVE THEM THE VIRGIN LOGIN SCREEN
  [B]}[/B]

Capoeirista

GAH! I'm so sorry - I meant to post this in newbies.

laserlight

if (isset($_POST['entered_Password']))  {

There's no matching closing brace for the opening brace on this line.

Capoeirista

Thanks LaserLight,

That's the thing.

Given the structure of the page, where would you put the missing closing bracket? Should it include the first four if statements, or only the first one?

The first four are different cases where the password has been entered. It should default to the last one (fresh login screen) if the password hasn't been entered yet. In order for the middle three to occur, the stuff in the first one needs to have happened.

So, should they all be equal, or do the middle three need to be nested into the first one?

And why am I getting an error that's telling me I have an extra closing bracket when I'm missing one? Is this normal?

And that whole line numbering thing. I have a lot of extra white space, does PHP collapse it? Should I count only the lines that have code on them? The error puts the extra closing bracket right in the middle of an HTML form. Is this normal?

Capoeirista

My thinking now is that the first if statement should stand alone and the other four should be a concatenation of if...else statements.

When I do this, the page at least renders.

Thanks for your help. I apologize for mis-posting this in the wrong category. Have a great day,

Lee