Якщо ви хочете, щоб якийсь код виконувався на основі двох або більше умов, який найкращий спосіб відформатувати оператор if?
перший приклад: -
if(ConditionOne && ConditionTwo && ConditionThree)
{
Code to execute
}
Другий приклад: -
if(ConditionOne)
{
if(ConditionTwo )
{
if(ConditionThree)
{
Code to execute
}
}
}
що найпростіше зрозуміти та прочитати, маючи на увазі, що кожна умова може бути довгою назвою функції або чимось.
Відповіді:
Я віддаю перевагу варіанту А
bool a, b, c;
if( a && b && c )
{
//This is neat & readable
}
Якщо у вас є особливо довгі змінні / умови методу, ви можете просто розбити їх
if( VeryLongConditionMethod(a) &&
VeryLongConditionMethod(b) &&
VeryLongConditionMethod(c))
{
//This is still readable
}
Якщо вони ще більш складні, то я б подумав про те, щоб робити методи умови окремо поза оператором if
bool aa = FirstVeryLongConditionMethod(a) && SecondVeryLongConditionMethod(a);
bool bb = FirstVeryLongConditionMethod(b) && SecondVeryLongConditionMethod(b);
bool cc = FirstVeryLongConditionMethod(c) && SecondVeryLongConditionMethod(c);
if( aa && bb && cc)
{
//This is again neat & readable
//although you probably need to sanity check your method names ;)
}
IMHO Єдиною причиною для варіанту "B" було б, якщо у вас є окремі elseфункції для запуску для кожної умови.
напр
if( a )
{
if( b )
{
}
else
{
//Do Something Else B
}
}
else
{
//Do Something Else A
}
Інші відповіді пояснюють, чому перший варіант зазвичай найкращий. Але якщо у вас кілька умов, спробуйте створити окрему функцію (або властивість), виконуючи перевірку умов у варіанті 1. Це полегшує читання коду, принаймні, коли ви використовуєте хороші імена методів.
if(MyChecksAreOk()) { Code to execute }
...
private bool MyChecksAreOk()
{
return ConditionOne && ConditionTwo && ConditionThree;
}
Якщо умови покладаються лише на локальні змінні області, ви можете зробити нову функцію статичною і передати все, що вам потрібно. Якщо є суміш, передайте місцеві речі.
isOkToDoWhateverяк властивості має багато сенсу.
ifчитаність висловлювань, а швидше про читаність умов.
Перший приклад є більш "легким для читання".
Насправді, на мою думку, вам слід використовувати лише другий, коли вам потрібно додати якусь «інакшу логіку», але для простого Умовного використовуйте перший смак. Якщо вас турбує довгий стан, ви завжди можете використовувати наступний синтаксис:
if(ConditionOneThatIsTooLongAndProbablyWillUseAlmostOneLine
&& ConditionTwoThatIsLongAsWell
&& ConditionThreeThatAlsoIsLong) {
//Code to execute
}
Щасти!
На запитання було задано і досі відповідали так, ніби рішення повинно прийматись виключно на "синтаксичній" основі.
Я б сказав, що правильна відповідь про те, як ви викладаєте низку умов в if, також повинна залежати від "семантики". Отже, умови слід розбити та згрупувати відповідно до того, що відбувається «концептуально».
Якщо два тести насправді є двома сторонами однієї медалі, наприклад. якщо (x> 0) && (x <= 100), то складіть їх разом на одному рядку. Якщо інша умова концептуально набагато більш віддалена, наприклад. user.hasPermission (Admin ()), а потім поставте його на власний рядок
Напр.
if user.hasPermission(Admin()) {
if (x >= 0) && (x < 100) {
// do something
}
}
if ( ( single conditional expression A )
&& ( single conditional expression B )
&& ( single conditional expression C )
)
{
opAllABC();
}
else
{
opNoneABC();
}
Форматування декількох умовних виразів у операторі if-else таким чином:
// disable any single conditional test with just a pre-pended '//'
// set a break point before any individual test
// syntax '(1 &&' and '(0 ||' usually never creates any real code
if ( 1
&& ( single conditional expression A )
&& ( single conditional expression B )
&& ( 0
|| ( single conditional expression C )
|| ( single conditional expression D )
)
)
{
... ;
}
else
{
... ;
}
Другий - це класичний приклад антивізеру стрілки, тому я б його уникав ...
Якщо ваші умови занадто довгі, витягніть їх у методи / властивості.
Перший з них простіший, тому що, якщо ви читаєте його зліва направо, ви отримуєте: "Якщо щось І щось інше І щось інше ТОГО", - це легко зрозуміле речення. Другий приклад читає "Якщо щось THEN, якщо щось інше THEN, якщо щось інше THEN", що є незграбним.
Крім того, подумайте, чи хочете ви використовувати деякі АБО у своєму реченні - як би ви це зробили у другому стилі?
{
( VeryLongCondition_1 ) or last;
( VeryLongCondition_2 ) or last;
( VeryLongCondition_3 ) or last;
( VeryLongCondition_4 ) or last;
( VeryLongCondition_5 ) or last;
( VeryLongCondition_6 ) or last;
# Guarded code goes here
}
Якщо якась із умов не виконується, вона просто продовжиться, після блокування. Якщо ви визначаєте будь-які змінні, які ви хочете зберегти після блоку, вам потрібно буде визначити їх перед блоком.
Я давно стикаюся з цією дилемою і досі не можу знайти належного рішення. На мій погляд, єдиним хорошим способом є спочатку спробувати позбутися умов раніше, щоб ви не раптом порівняли 5 з них.
Якщо альтернативи немає, тоді, як пропонували інші, - розбийте її на окремі та скоротіть імена або згрупуйте їх, і, наприклад, якщо все має бути істинним, використовуйте щось на кшталт "якщо в масиві x немає false, тоді запустіть".
Якщо все не вдається, @Eoin Campbell дав досить хороші ідеї.
Коли стан справді складний, я використовую такий стиль (приклад реального життя PHP):
if( $format_bool &&
(
( isset( $column_info['native_type'] )
&& stripos( $column_info['native_type'], 'bool' ) !== false
)
|| ( isset( $column_info['driver:decl_type'] )
&& stripos( $column_info['driver:decl_type'], 'bool' ) !== false
)
|| ( isset( $column_info['pdo_type'] )
&& $column_info['pdo_type'] == PDO::PARAM_BOOL
)
)
)
Я вважаю, що це приємніше і читабельніше, ніж вкладання декількох рівнів if(). І в деяких випадках, подібних до цього, ви просто не можете розбити складні умови на частини, оскільки в іншому випадку вам доведеться if() {...}багато разів повторювати ті самі оператори в блоці.
Я також вважаю, що додати трохи «повітря» в код - це завжди гарна ідея. Це значно покращує читабельність.