PHP - If/else, for, foreach, while - without curly braces?

Something that really would like to know but never found out are shortcuts in PHP.

I am currently coding a function with a foreach loop with just a single statement inside. I tried to omit the curly braces as you can do in if/else control structures and it works. No errors.

foreach($var as $value)
    $arr[] = $value;

Now I tried to use it the same way but putting an if/else block inside it. Again, working and no errors.

foreach($var as $value)
    if(1 + 1 == 2) {
        $arr[] = $value;
    };

Then, I thought like "why is this working?" and omitted the closing semicolon. Still working. So I tried to use the if/else statement without curly braces inside the foreach loop and again, still working and no errors. But is the foreach loop really closed/ended right now?

foreach($var as $value)
    if(1 + 1 == 2)
        $arr[] = $value;

At least I omitted the closing semicolon again and (as expected) a parsing error occurred.

So my big question is: When can I omit the curly braces and in which structure/loop/function? I know that I can definitely do so in if and else. But what about while, for and foreach?

And yes, I know that it is not safe, smart, whatever to code without curly braces and there are shorthands like $condition ? true : false; and if: doSomething(); endif;, endfor; and endforeach;. I don't wanna learn about shorthands I just want to understand the conditions about when and where it is possible to omit the curly brackets.

Solution 1:

When you omit the braces it will only treat the next statement as body of the condition.

if ($x) echo 'foo';

is the same as

if ($x) { echo 'foo'; }

but remember that

if ($x)
  echo 'foo';
  echo 'bar';

will always print "bar"

Internally it's the other way around: if will only look at the next expression, but PHP treats everything in {} as a single "grouped" expression.

Same for the other control statements (foreach, and so on)

Solution 2:

There are places where you can, but you never should.

Explicit is always better than implicit.

Who knows when you're going to have to go back and modify old code. It's so easy to miss that stuff and there's nothing gained by doing it.

Solution 3:

It will work fine if you only have one argument inside!. But if you want to omit curly brace you can use colon and end. example:

if(a < 1 ) :
    echo "a is less than 1";
else :
    echo "a is greater than 1";
endif;

Solution 4:

As have said you don't wanna learn about shorthand's and accepted answer gives good example about omitting curly braces, but there is something to add. As you can see it's fine to omit curly braces in case of if ($x) echo 'foo';. There is nothing wrong with code, no performance or other issues and it is readable by other developers. And example also shows you that if you write

if ($x)
    echo 'foo';
    echo 'bar';

instead of

if ($x)
    echo 'foo';

echo 'bar';

You can run into unwanted results where bar is printed while you don't want it to be printed and if your code is full of such statements then it will make it harder for you to read your own code and even more harder for others to read it.

I don't wanna learn about shorthand's I just want to understand the conditions about when and where it is possible to omit the curly brackets.

These things are closely related so if you really want to understand where it is possible to omit curly brackets then that should be a must that you understand or are at least aware of shorthand's , have read

1. PHP Control Structures
2. The PHP ternary conditional operators and expressions in general

So my big question is: When can I omit the curly braces and in which structure/loop/function?

The curly brace is not required however, for readability and maintenance, many developers would consider it bad style not to include them. Previous 2 links should give you information needed to make your own decisions when you could omit curly brace. for example there is nothing wrong with following code snippets which all do exactly same thing.

With curly brace

if (PHP_VERSION_ID < 70000)
{
    print "PHP >= 7.0 required yours is ";
    print phpversion();
    print "\n";
    exit(1);
}

Is same as

if (PHP_VERSION_ID < 70000) :
    print "PHP >= 7.0 required yours is ";
    print phpversion();
    print "\n";
    exit(1);
endif;

Or you can use the dot operator

if (PHP_VERSION_ID < 80000)
    (print "PHP >= 7.0 required yours is ") . (print phpversion()) . (print "\n") . exit(1);

And you can make use of the ternary conditional operator and even omit if it self besides omitting curly braces

(PHP_VERSION_ID > 70000) ?: (print "PHP >= 7.0 required yours is ") . (print phpversion()) . (print "\n") . exit(1);

Since we only print we can shorten that and strip some print string functions which were here to represent more than one function in statement without curly braces

(PHP_VERSION_ID > 70000) ?: (print "PHP >= 7.0 required yours is " . phpversion() . "\n") . exit(1);

As from php 7 we can use Null coalescing operator

(PHP_VERSION_ID > 70000) ?: null ?? (print "PHP >= 7.0 required yours is ".phpversion() . "\n") . exit(1);

As one can see that there is many ways you can get exactly the same result. That not only applies for this if example but same can be practiced with structure/loop/function. So there is no one answer for your big question. One should consider mainly following.

1. Is the code you are writing easy to maintain.
2. Can you answer for your self is there something you win by omitting curly braces.