Является ли плохой практикой использование оператора if без фигурных скобок? [закрыто]

Я видел такой код:

if(statement)
    do this;
else
    do this;

Однако я думаю, что это более читабельно:

if(statement){
    do this;
}else{
    do this;
}

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

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

С точки зрения ремонтопригодности всегда разумнее использовать вторую форму.

РЕДАКТИРОВАТЬ: Нед указывает на это в комментариях, но я думаю, что здесь тоже стоит сделать ссылку. Это не просто гипотетическая чушь про башню из слоновой кости: https://www.imperialviolet.org/2014/02/22/applebug.html

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

if(one)
    if(two)
        foo();
    else
        bar();

Из этого:

if(one)
    if(two)
        foo();
else
    bar();

Мой общий шаблон таков: если он умещается в одной строке, я сделаю:

if(true) do_something();

Если есть предложение else или если код, который я хочу выполнить, trueимеет значительную длину, полностью используйте фигурные скобки:

if(true) {
    do_something_and_pass_arguments_to_it(argument1, argument2, argument3);
}

if(false) {
    do_something();
} else {
    do_something_else();
}

В конечном итоге все сводится к субъективному вопросу стиля и удобочитаемости. Однако общий мир программирования в значительной степени разделяется на две части (для языков, в которых используются фигурные скобки): либо используйте их все время без исключения, либо используйте их все время без исключения. Я принадлежу ко второй группе.

Я использую средство форматирования кода среды IDE, которую использую. Это может отличаться, но его можно настроить в настройках / параметрах.

Мне нравится этот:

if (statement)
{
    // comment to denote in words the case
    do this;
    // keep this block simple, if more than 10-15 lines needed, I add a function for it
}
else
{
    do this;
}

Я следую «правилу»:

Если оператор «if» выполняет тестирование, чтобы что-то сделать (функции вызова IE, переменные настройки и т. Д.), Используйте фигурные скобки.

if($test)
{
    doSomething();
}

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

Если оператор «if» выполняет тестирование, чтобы прекратить что-либо делать (управление потоком IE внутри цикла или функции), используйте одну строку.

if($test) continue;
if($test) break;
if($test) return;

В этом случае для программиста важно быстро обнаружить исключительные случаи, когда вы не хотите, чтобы код запускался, и все это покрывается $ test, а не блоком выполнения.

Наличие фигурных скобок с первого момента должно помочь вам никогда не отлаживать это:

if (statement)
     do this;
else
     do this;
     do that;

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

if (someFlag) {
 someVar= 'someVal1';
} else {
 someVar= 'someVal2';
}

Гораздо приятнее выглядит так:

someVar= someFlag ? 'someVal1' : 'someVal2';

Но используйте тернарный оператор только в том случае, если вы абсолютно уверены, что больше ничего не нужно добавлять в блоки if / else!

Я предпочитаю использовать подтяжки. Добавление фигурных скобок упрощает чтение и изменение.

Вот несколько ссылок по использованию скобок:

• Предпочитать многострочный, если

• Отсутствие скобок: вопрос не только в стиле

• Переполнение стека

По моему опыту, единственное (очень) небольшое преимущество первой формы - читабельность кода, вторая форма добавляет «шума».

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

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

Одно из самых важных правил, которое следует помнить при написании кода, - это согласованность. Каждая строчка кода должна быть написана одинаково, независимо от того, кто ее написал. Строгость предотвращает "возникновение" ошибок;)

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

Когда мои студенты принимают этот факт, они перестают бороться со своим исходным кодом и начинают рассматривать кодирование как действительно интересное, стимулирующее и творческое занятие. Они бросают вызов своему разуму, а не нервам!

Это вопрос предпочтения. Я лично использую оба стиля, если я достаточно уверен, что мне больше не нужно будет добавлять операторы, я использую первый стиль, но если это возможно, я использую второй. Поскольку к первому стилю нельзя добавлять больше операторов, я слышал, что некоторые люди рекомендуют его не использовать. Однако второй метод требует дополнительной строки кода, и если вы (или ваш проект) используете такой стиль кодирования, первый метод очень предпочтителен для простых операторов if:

if(statement)
{
    do this;
}
else
{
    do this;
}

Однако я думаю, что лучшее решение этой проблемы - в Python. С блочной структурой на основе пробелов у вас нет двух разных методов создания оператора if: у вас есть только один:

if statement:
    do this
else:
    do this

Хотя здесь есть «проблема», заключающаяся в том, что вы вообще не можете использовать фигурные скобки, вы получаете то преимущество, что больше нет строк, чем первый стиль, и у него есть возможность добавлять больше операторов.

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

Не получится:

если (оператор) сделать это; и это; иначе сделайте это;

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

Пример:

if (argument == null)
    throw new ArgumentNullException("argument");

if (argument < 0)
    return false;

В противном случае я использую второй стиль.

Лично я предпочитаю использовать сочетание пробелов и скобок, например:

if( statement ) {

    // let's do this

} else {

    // well that sucks

}

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

Я согласен с большинством ответов в том, что лучше указывать в коде явно и использовать фигурные скобки. Лично я бы принял набор стандартов кодирования и убедился бы, что все в команде знают их и соблюдают. Где я работаю, мы используем стандарты кодирования, опубликованные IDesign.net для проектов .NET.

Я предпочитаю ставить фигурную скобку. Но иногда помогает тернарный оператор.

Вместо того :

int x = 0;
if (condition) {
    x = 30;
} else {
    x = 10;
}

Просто нужно сделать: int x = condition ? 30 : 20;

Также представьте себе случай:

if (condition)
    x = 30;
else if (condition1)
    x = 10;
else if (condition2)
    x = 20;

Было бы намного лучше, если бы вы вставили фигурную скобку.