is my syntax correct

Hoangsta

i just need a second eye to see if my syntax is correct...

  if ($feedback == 1 && $feedback2 == 1){      

	// Register the input with the value 
	$_SESSION['user_name'] = $_POST['user_name']; 
    $_SESSION['authr_user_name'] = $_POST['user_name'];  

    // On successful login, redirect to homepage 
    header("Location: A");
	 } else {
    $feedback_str = "<P class=\"errormess\">$feedback</P>";  

  }
  if ($feedback == 1 && $feedback2 != 1){
		// Register the input with the value 
		$_SESSION['user_name'] = $_POST['user_name']; 
    	$_SESSION['authr_user_name'] = $_POST['user_name'];  

    	// Redirect to page to fill out before redirecting to homepage.  

		header("Location: B");	
   		} else {
    $feedback_str = "<P class=\"errormess\">$feedback2</P>";  

  } 
} else {  

  $feedback_str = '';  

}

laserlight

Well, the last else does not have a corresponding if condition.

Also, ($feedback == 1 && $feedback2 == 1) and ($feedback == 1 && $feedback2 != 1) are complements.

This means that you have extra else statements corresponding to those conditions.

drawmack

I find that running things through the interpreter helps me find syntax errors.

Hoangsta

there were more on the code above,...

if ($LOGGED_IN = user_isloggedin()) {
  user_logout();
  $_COOKIE['user_name'] = '';
  unset($LOGGED_IN);
}

i forgot to add that to the top...but on my

($feedback == 1 && $feedback2 == 1) and ($feedback == 1 && $feedback2 != 1) are complements.

I don't know how to go about it....if i clear out all the businesses that register, it will work fine. it's funny. it detect that the business has a default value of 0(zero), so it shows the form to register your business. once one user register the business (1), it doesn't work anymore...it allows any user to bypass that inspectaion and go to the index.php.

if they are complements...i don't know how to write it another way...i'm just sooo tired...

if u wish to see the code that i written for determining my $feedback and $feedback2, i'll be more glad to show it...