Одна строка, если операторы

Question

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

Считаете ли вы, что заявление должно быть написано как:

if(condition) 
{ 
    statement = new assignment; 
} 

ИЛИ

if(condition) 
    statement=new assignment; 

пожалуйста обеспечивают хорошую причину своего решения.

Answer 1

, если вы действительно должны использовать одну строку, если

if(condition) statement=new assignment; 

будет лучше, так как его одной линии, она должна содержать одну операцию.

Answer 2

Я всегда был поклонником брекетов. Если кто-то изменить oneline если заявление так:

if(condition) statement=new assignment; 

в

if(condition) 
statement = new assignment; 
another statement; 

Вы не получите ожидаемое поведение.

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

Answer 3

Я всегда делаю однострочные if заявления без санкций. Наличие скобок указывает (синтаксически правильно), что «о, я могу сделать что-то еще здесь ...», и мне не нравится соблазн. Все, что связано с несколькими утверждениями, должно быть разбито на несколько строк с соответствующими скобками.

Answer 4

Я всегда использую прилагаемые фигурные скобки, чтобы уменьшить риск того, что кто-то (включая меня) позже представит ошибку, отредактировав код вокруг инструкции if, не обращая при этом внимание на то, какие строки принадлежат как часть условия if ,

EDIT:

Вот живой пример, если это я просто случайно наткнулся на какой-то старый код:

if (form.validateUpload (messages, this)) 
    return getErrorOutcome (ctx, messages); 
    if (LOG.isInfoEnabled()) 
     LOG.info ("CREATING UPLOAD"); 

Обратите внимание, как и «если» заявления в основном блоке кода, но из-за к плохому форматированию, на первый взгляд они кажутся вложенными. Конечно, любой «хороший» программист должен быстро увидеть, что происходит, но зачем вызывать лишнюю путаницу?

Answer 5

Я бы пошел без скобок.

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

Звучит как пустая трата аргумента.

Answer 6

if(condition) 
    statement=new assignment; 

или

if(condition) statement=new assignment; 

Answer 7

Как правило, я отрекаюсь сослагательного наклонения однострочные за исключением в этом случае Perl

operation if condition; 

Answer 8

Это действительно зависит от стиля кодирования вашей группы. Группа должна иметь последовательные стандарты кодирования. Для моей текущей группы мы всегда используем:

if (condition) { 
    statement = new assignment; 
} 

Мы делаем это, чтобы избежать ошибок, вызванные забывая скобки после, если заявления, например:

if (condition) 
    statement1; 
    statement2; 
//statement2 is not part of the if statement, but it looks like it because of wrong indentation 

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

if (condition) 
    statement1; 

Лично мне не нравится это так, потому что это менее явная; но самое главное - придерживаться стандартного стандарта кодирования для вашей группы или проекта, так что код, который вы пишете, выглядит как код, который пишут ваши коллеги, и так же легко читается для всех в группе.

Соглашения вашей среды IDE или среды могут обеспечить хорошую основу для ваших стандартов кодирования и даже могут быть адаптированы к стилю вашей группы.

Answer 9

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

Answer 10

if (condition) 
{ 
    statement = new assignment; 
} 

- это то, что я хотел бы написать. А именно потому, что мне нравится аккуратный код, который экономит время на чтение/редактирование/понимание.

В очень редких случаях, я хотел бы сделать исключение, обычно только тогда, когда я быстро и грязно кодирования что-то для отладки и т.д.

А одну строку, если оператор всегда очень легко испорчены, как помещается точка с запятой.

Answer 11

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

if(condition) { 
    statement = new assignment; 
} 

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

http://java.sun.com/docs/codeconv/html/CodeConventions.doc6.html#449

Примечание: если заявления всегда использовать фигурные скобки {}. Избегайте следующих ошибок склонными вид:

if (condition) //AVOID! THIS OMITS THE BRACES {}! 
    statement; 

Использование скобок предотвращения ошибок: некоторые еще можно добавить позже новые предложения, которые suposed должны быть выполнены, если условие и забвение скобок