Найбільш читаний спосіб форматування довгого, якщо умови? [зачинено]

ifЯкщо взагалі можливо, слід уникати тривалих умов намотування , але іноді ми все пишемо їх. Навіть якщо це дуже проста умова, заяви, що стосуються, іноді просто дуже багатослівні, тому вся умова закінчується дуже тривалою. Який самий читабельний спосіб їх форматування?

if (FoobarBaz::quxQuux(corge, grault) || !garply(waldo) || fred(plugh) !== xyzzy) {
    thud();
}

або

if (
    FoobarBaz::quxQuux(corge, grault)
 || !garply(waldo)
 || fred(plugh) !== xyzzy
) {
    thud();
}

або

if (FoobarBaz::quxQuux(corge, grault)
    || !garply(waldo)
    || fred(plugh) !== xyzzy) {
    thud();
}

або

thudable = FoobarBaz::quxQuux(corge, grault);
thudable ||= !garply(waldo);
thudable ||= fred(plugh) !== xyzzy;

if (thudable) {
    thud();
}

чи будь-які інші уподобання?

Часто, якщо умова є знаком коду, який потребує рефакторингу, але іноді не вдається уникнути цього. У цих випадках я віддаю перевагу першому:

if (bar || baz || quux) { ... }

Тому що ти можеш сказати, що відбувається з одним рядком. Однак я б швидше скоїв щось подібне, коли це можливо:

function foo() {
  return bar || baz || quux;
}

if (foo()) { ... }

Мені подобається тримати операторів в кінці, щоб вказати на продовження:

if (the_function_being_called() != RETURNCODE_SUCCESS &&
    the_possibly_useful_recovery_strategy() == RETURNCODE_EPICFAIL &&
    this_user_has_elected_to_recieve_error_reports)
{
    report_error();
}

Я великий фанат значущих імен змінних:

const bool isInAStrangeCondition =
    FoobarBaz::quxQuux(corge, grault) ||
    !garply(waldo) ||
    fred(plugh) !== xyzzy;

if (isInAStrangeCondition) {
    thud();
}

Або рефактор як функція, як згадувалося вище.

Я розбиваю медьє супрекспресії, або всі вони, як змінні bool. Тоді булева логіка висловлювання "якщо" верхнього рівня може бути зрозумілою. У роботі, яку я роблю, це не завжди декілька речей ORED або ANDed.

bool goodblah = some_mess < whatever;
bool frobnacious = messy_crud != junky_expression;
bool yetanother = long_winded_condition;

if (goodblah || (frobnacious && yetanother))   {
    ...
}

Це особливо добре в налагоджувальній машині, де я можу переглянути всі булі, перш ніж виконати "якщо".

Я схильний вирівнювати операторів на початку нових рядків, тому пам’ятаю, як я поєдную терміни (як для довгої логіки, так і для довгої арифметики). Подобається це:

if (first_attempt(data) == SUCCESS
    || (reusable(data) && second_attempt(data) == SUCCESS)
    || (still_reusable(data) && third_attempt(data) == SUCCESS))
  return SUCCESS;

Це працює лише в тому випадку, якщо я відступаю через 2 пробіли або встановлюю для мого оточення багаторядкові предикати відступів, інакше важко буде сказати, де закінчується предикат і корисний код.

Я фанат наступного:

if (really_long_expression && another_really_really_long_expression && 
            another_very_long_expression_OMG_id_it_long){
    bugs();
}

Таким чином, вона все ще виглядає як вираз if, а не розбита на частини, якщо вираз. Відступ допомагає показати, що це продовження попереднього рядка.

Ви також можете відступити його до тих пір, поки не відкриється дужка в кінці попереднього рядка, щоб вона була в кінці виразу if, як належить.