Conditional Statements in PHP code Between HTML Code
use braces {
and }
.
<?php if(check if user is logged in) { ?>
<display users profile in html>
<?php } else { ?>
<display an error>
<?php } ?>
You probably forgot the endif
of the alternative control structure syntax:
<?php if(check if user is logged in): ?>
<display users profile in html>
<?php else: ?>
<display an error>
<?php endif; ?>
Omitting the braces as you wrote is not possible. It is only possible if it is followed by a regular statement.
PHP has two styles of notation for if() blocks (and blocks in general).
Firstly, you have the wordy notation, which involves explicitly stating endif;
at the end of the if() block. It looks like this:
if(whatever):
do something
else:
do something else
endif;
The colons at the end of the if
and else
lines are important, because otherwise PHP thinks you're using the other notation (below).
Secondly, you have the curly-braces notation, which looks similar to C or Perl style code, and looks like this:
if(whatever) {
do something
} else {
do something else.
}
With this style notation, you are allowed to leave the pairs of curly-braces off if your block is only going to be one line long. (I personally think it's bad practice to leave them off like this, but plenty of people swear by it, and it is perfectly valid syntax. But I've seen PHP get confused over single-line blocks when you're switching between PHP code and HTML; this is why I always prefer to use braces even if I'm only writing one line).
The problem in your case is that you've mixed the two notations. You's trying to us the wordy notation, but don't have the colons on the lines, so PHP thinks you mean the braces notation. But because the braces notation allows for the braces to be missed out, PHP is seeing your code as valid syntax, even if it isn't going to work quite as you planned.
Your solution is to tidy it up so that you are definitely using one syntax or the other. Either add braces ({
and }
}) to the start and end of each block as shown in my example, or add colons and an endif;
line.
So your code should look like one of these two examples:
<?php if(check if user is logged in): ?>
<display users profile in html>
<?php else: ?>
<display an error>
<?php endif; ?>
or...
<?php if(check if user is logged in) { ?>
<display users profile in html>
<?php } else { ?>
<display an error>
<?php } ?>
Hope that helps.