[PHP] Ставить ли скобки в условии if, если в блоке всего 1 действие?

Question

[Вадим]

Какое-то время назад друг, увидев мой код, указал на некоторые моменты которые, как ему показалось неверны(или просто так не принято).

if (!empty($fields))
    foreach ($fields as $key => $value) {
        /* Операции с массивом */
    }

Так вот этот кусок, по его мнению правильнее было бы оформить вот так:

if (!empty($fields)) {
    foreach ($fields as $key => $value) {
        /* Операции с массивом */
    }
}

Так как же всё-же *правильнее* ?

И еще 1 вопрос:

/* Так? */
if (!empty($fields)) {

}

/* Или так? */
if (!empty($fields))
{

}

Разницы особой нет, но всё-же интересно Ваше мнение.

Answer 1

[Василий]

Скобки лучше ставить т.к. по запарке потом можно вставить в это условие еще чтото и забыть про скобки, а потом тупить ват из фак.
Про второе считаю что первый вариант лучше т.к. не хавается лишняя строчка с скобкой и код получается более плотный и читабельный. Адепты С++ будут со мной несогласны.

Comments

[cmx]

Такой способ называется египетскими скобками и согласно PSR-2 используется везде за исключением объявления классов и методов.

Answer 2

[Vampiro]

Есть такое понятие, как "стандарт кодирования". Первый с гугла: www.skillz.ru/dev/php/article-Standarty_kodirovani...

Разные команды пишут по разным стандартам. Большинство редакторов умеют "отстандартить" код =) Так что дело вкуса.

ЗЫ на собеседованиях про psr-2 могут запросто спросить.

Answer 3

[xAockd]

Что бы не затупить не было втф
if пишу так
if($smth) return $smth;
И тогда мне кажется явно видно что if однострочник. Ну, а если надо расширить, то тогда скобки

Answer 4

[Александр Вульф]

Если вы пишите так:

if ($var) while($var--) {
    # ......
}

То скобки лучше не ставить.

Но если вы пишите так:

if ($var)
  while($var--) {
    # ....
  }

То лучше ставить:)
Имхо, конечно.

Comments

[cmx]

На моей практике более частый случай, по запарке наоборот закомментировал while, и думаешь - что черт возьми происходит?

Answer 5

[gro]

Начинающие программисты самовыражаются и самоутверждаются тем, как они раставляют скобки и как выравнивают блоки.

Потом начинают самовыражаться за счёт более интересных вещей, таких, как, например, хорошая архитектура или работающий проект.

Если первая стадия пройдена, нужно просто взять один из стандартов и следовать ему.
В PHP единственный стандарт по этому поводу, о котором следует говорить - PSR-2.

Впрочем, в любом другом вменяемом стандарте всегда ставятся скобки.

Answer 6

[anitspam]

я стал ставить скобки и переносить их на новую строку, когда увидел в промышленном коде, отвечающем за безопасность, примерно такое
с++

...
if (...)
    DEBUG('...')
    a = 1;
...

такие ошибки труднее найти визуально

Answer 7

[r1ch]

if (!empty($fields)) return $fields;
else rerurn false;

for(;;) echo "это бесконечный цикл";

в примерах выше обычно можно не ставить, код более линейный и понятный

Answer 8

[barbozze]

Вообще нужно смотреть по обстоятельствам, когда именно это необходимо
"Ставить ли скобки в условии if, если в блоке всего 1 действие?"
если синтаксис позволяет то на твоё усмотрение, лишь бы это было написано понятно

Comments

[barbozze]

я посмотрел и почитал 'PSR-2 – Рекомендации по оформлению кода', ни слова о том что они должны быть обязательно, только рекомендации по оформлению то бишь не жесткие правила, иначе синтаксис этого не учитывал и отсутствие их воспринималось как ошибка синтаксиса

так что ставить их необязательно, когда тело состоит из одной строки кода или имеет какую либо инструкции со своим телом

[barbozze]

разумеется это не относится к классам, интерфейсам, функция и т.б. ... только к инструкциям