Ставить ли фигурные скобочки?

Question

[Даниил Демидко]

Понимаю, вопрос странный, но я не знаю что лучше когда требуется выполнить одно действие с условием.
Например:

if(any)
     {
            // строка кода
     }

Или так:

if(any)
          // строка кода

У Срауструпа в примерах ничего по этой теме неясно - в одних случаях он использует фигурные скобки после условия, даже если нужно выполнить всего одно действие, в других нет.
Красота или лаконичность?

Answer 1

[aminought]

Правилом хорошего тона считается постановка фигурных скобок в любом случае. Для вдохновения можете прочитать вот эту статью: Исключительная красота исходного кода Doom 3

Comments

[Даниил Демидко]

Спасибо, я так и думал но колебался между красотой и лаконичностью.

[Adamos]

Даниил Демидко: лаконичность в С++ - это ни в коем случае не экономия символов!
Лаконичность в С++ - это создание дополнительного уровня черновой работы в классе, работу которого (класса) в алгоритме потом можно будет читать, как роман. Потому что весь очевидный код убран в методы, имеющие очевидные названия.

Answer 2

[Vitaly]

Ответ на этот вопрос не так однозначен. Существуют стандарты оформления кода, согласно которым должен быть отформотирован весь исходный код. Это актуально и для коммерческих компаний и для открытых проектов. Поэтому "хорошо" или "плохо" сказать нельзя, нужно говорить "соотвествует стандарту или нет". Можете для своих личных проектов выбрать какой-нибудь стандарт кодирования и следовать ему. Вот пример стандарта: https://wiki.qt.io/Coding_Conventions . Помимо этого существуют программы, которые позволяют отформатировать весь код согласно набору правил (astyle, clang-format, uncrustify etc.). Использовать их далеко не всегда удобно, так как зачастую форматирование может быть не стандартным чтобы повысить читабельность кода, но в общем случае нормально.

Что касается вашего примера, я предпочитаю не ставить лишние скобки для тела if в одну инструкцию:

// OK
if (a > 3)
    ++a;

Но никогда не пишите эту инструкцию на той же строке -- не удобно отлаживать (может быть в какой-то IDE и удобно, но обычно каждый разработчик пользуется своей любимой...):

if (a > 3) ++a; // Как поставить бряку на "++a"?

Что касается циклов, то с for лучше всегда ставить скобки, т.к. некоторые IDE в режиме отладки вроде VS2008 (а возможно и более старших версий) не умеют заходить в тело однострочного цикла без скобок.

Comments

[Adamos]

При чем тут стандарты кода? Вложенное тело без скобок - это грабли для того, кто будет править код, позволяющие легко и неочевидно его испортить. При этом в плюсах у этого, так сказать, стандарта - только экономия нескольких байт в исходниках и, возможно, одной строчки. То есть можно считать, что плюсов у него вовсе нет.

[Vitaly]

Adamos: Сдаётся мне, что вы имеете ввиду конструкции типа:
if (...)
if (...)
// do
else
// do
где разработчик хотел чтобы ветка else выполнилась для первого if. Я встречал подобные ситуации, не то чтобы часто, раза два быть может за последние пять лет, но всё таки согласен что они имеют место быть. Правда это скорее просто исключение и скобки там нужны.
Поэтому я до сих пор считаю, что в нормально отформатированном коде в подавляющем большенстве случаев скобки для тела у if из одной инструкции не нужны. А так-то практически любой код при сопровождении может быть неочевидно испорчен и представляет собой потенциальную проблему. Но на то есть юнит тесты.

[Adamos]

Vitaly: нет, я имею в виду любой if или for, где одна строчка всегда может превратиться в две и наоборот. Для выделения тела цикла либо условной конструкции я всегда использую фигурные скобки, и это помогает не думать о такой фигне, занимаясь работой.

[Vitaly]

Adamos: Так это же очевидно, появились две строки -- поставили скобки. Две строки превратились в одну, прекрасно -- убрали скобки. Хотя конечно же на это есть стандарты кодирования в компании (если они описывают эту ситуацию), в противном случае это дело личного предпочтения программиста.

[Adamos]

Vitaly: Так это же очевидно - вложенное тело должно быть в фигурных скобках. Запомнили этот простой принцип, забыли о проверках, сколько там будет строчек. И при чтении тоже забыли. Экономим извилины для настоящего дела.

[Vitaly]

Adamos: это уже беспредметное обсуждение. Рассматривать нужно практические жизненные ситуации. Именно основываясь на этом я и посоветовал начинающему программисту взять любой распространённый стандарт и оформлять код согласно нему.
Нельзя просто взять и запомнить принцип и переключиться на что-то... И прочее. Я еще раз повторю: стандарт кодирования, если нет, то увольняйтесь делайте как вам удобно и как вы привыкли. Хоть вы тресните, а если в компании, в которую вы пришли работать есть стандарт, по которому положено не ставить скобки вокруг тела из одной инструкции, то вы будете писать как все, чтобы сохранить однородность кода и ваши личные предпочтения остануться за бортом. Код написаный не по стандарту даже на Code Review проверять не будут а просто завернут с пометкой "оформить как надо".

[Adamos]

Vitaly: как раз начинающему программисту лучше не портить свой стиль фантазиями, а применять good practices. Скорее всего, там, где будет применяться стиль, они же и будут применяться.

[Олег Цилюрик]

Adamos:

начинающему программисту лучше не портить свой стиль фантазиями

А почему вы так твёрдо уверены, что ваши фантазии лучше, чем фантазии начинающего программиста? ;-)

[Adamos]

Олег Цилюрик: потому что они проверены практикой. Я тоже когда-то экономил символы и строчки, а потом был вынужден вернуться к этому своему коду через несколько лет...

[Олег Цилюрик]

Adamos:
Здесь уже сказали:

это уже беспредметное обсуждение.

- если в конторе есть свой стиль кодирования - будете его использовать ... или отправитесь на улицу
- если нет, то лучше каждому то, как для него сложилось, как комфортно и привычно.
А всё остальное - это пустая болтовня ... которой в IT сложилось столько, как ни в одной технической дисциплине.

[Adamos]

Олег Цилюрик: а если мы говорим о начинающем программисте, то никакая контора ему стиль не навязывает, и никаких привычек у него еще нет. Почему бы ему не привыкнуть к тому варианту, который приводит к меньшим граблям? Тем более, что он сам спрашивает, что лучше.
Вы сами не понимаете, что только увеличиваете беспредметность спора, переводя его от конкретного вопроса к общим рассуждениям типа "shoulda coulda woulda"?

[Олег Цилюрик]

Adamos:

Почему бы ему не привыкнуть к тому варианту, который приводит к меньшим граблям?

Потому, что вы безапелляционно считаете, что ваш стиль является лучшим.
А я, например, так не считаю. А считаю (IMHO), при прочих равных обстоятельствах (бывают конечно всякие особые привходящие особенности), что код, который короче - он всегда и лучше.

[Adamos]

Олег Цилюрик: как я уже писал, у меня была прекрасная возможность убедиться в обратном.

[Олег Цилюрик]

Adamos:
бывает ;-)

Answer 3

[abcd0x00]

Руководствоваться нужно тем, что хорошо видно, и тем, что легко менять. Опыт показывает, что фигурные скобки для этого подходят лучше.

if (cond)
    ;
else
    statement;

if (cond) {}
else {
    statement;
}

Когда нужно добавить операторы в любую ветвь, то в первом случае нужно добавить фигурные скобки, а во втором случае фигурные скобки там уже есть.

Также фигурные скобки не дают частям конструкций приклеиться не туда. Представь, что кто-то незнакомый писал код, который тебе нужно дописывать.
Ты быстро пробегаешь его глазами и понимаешь неправильно, потому что такого кода три тонны.

if (cond)
    if (cond)
        if (cond) {
            statement;
        statement;
        }
    else
        statement;
else
    statement;

А вот он же

if (cond) {
    if (cond) {
        if (cond) {
            statement;
        statement;
        }
    else {
        statement;
    }
}
else {
    statement;
}
}

Ты поставил курсор на скобочку в случае сомнений - и редактор тут же тебе подсветил парную скобку.

Answer 4

[sim3x]

if(any) {
    // строка кода
}

Answer 5

[Noortvel]

if(any) // строка кода ;
?