Как расположить длинные условные выражения с отступом для 'если' операторы?

Архитекторы / дизайнеры программного обеспечения переоценены

Как разработчик, я ненавижу идею архитекторов программного обеспечения. В основном это люди, которые больше не пишут полный рабочий день, читают журналы и статьи, а затем рассказывают вам, как разрабатывать программное обеспечение. Это должны делать только люди, которые действительно пишут программное обеспечение на полную ставку. Мне все равно, если бы вы были лучшим программистом в мире 5 лет назад, прежде чем стать архитектором, ваше мнение для меня бесполезно.

Как это для спорных?

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

16
задан 2 revs 23 May 2017 в 11:46
поделиться

5 ответов

Как насчет этого?

bool isValid = isNull(value1) || isToLong(value1) || hasBadFormat(valule1);
if( isValid )
{
   doSomething();
}
else
{
   doSomethingElse();
}

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

24
ответ дан 30 November 2019 в 15:38
поделиться
if( isNull(value1) ||
    isToLong(value1) ||
    hasBadFormat(valule1))
{
    doSomething();
}
else
{
    doSomethingElse();
}

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

Конечно, я предпочитаю:

if( isNull(value1)
    || isToLong(value1)
    || hasBadFormat(valule1))
{
    doSomething();
}
else
{
    doSomethingElse();
}

: -)

13
ответ дан 30 November 2019 в 15:38
поделиться

Очевидное решение - переместить скобку на следующую строку, как и задумал Бог!

8
ответ дан 30 November 2019 в 15:38
поделиться

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

Я бы написал его так:

if(isNull(value1) ||    
   isToLong(value1) ||
   hasBadFormat(valule1))
{    
    doSomething();
}
else
{    
    doSomethingElse();
}

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

if(isNull(value1) || isToLong(value1) || hasBadFormat(valule1))
{    
    doSomething();
}
else
{    
    doSomethingElse();
}

И в этом случае, поскольку он кажется достаточно коротким, я бы сделал последнее.

2
ответ дан 30 November 2019 в 15:38
поделиться

Я обычно помещаю операторы в начало строки, чтобы они все отображались.

Итак, вот одно предложение:

if(isNull(value1)
   || isTooLong(value1)
   || hasBadFormat(valule1))
{
   doSomething();
} /* if */
else
{
   doSomethingElse();
} /* else */

Вот другое:

if(0
   || isNull(value1)
   || isTooLong(value1)
   || hasBadFormat(valule1))
/* ...etc... */

(Для &&, это было бы, если (1 && a && b) и т. д.)

Или это:

if
(
   isNull(value1)
   || isTooLong(value1)
   || hasBadFormat(valule1)
)
/* ...etc... */
1
ответ дан 30 November 2019 в 15:38
поделиться
Другие вопросы по тегам:

Похожие вопросы: