Koşullar uzunsa biçimlendirmenin en kolay yolu? [kapalı]

ifMümkünse uzun sarım koşullarından kaçınılmalıdır, ancak bazen hepimiz bunları yazarız. Çok basit bir durum olsa bile, ilgili ifadeler bazen çok endişe vericidir, bu nedenle tüm koşul çok uzundur. Bunları formatlamanın en kolay yolu nedir?

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

veya

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

veya

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

veya

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

if (thudable) {
    thud();
}

ya da başka bir tercih?

Çoğunlukla, eğer koşul yeniden düzenlemeye ihtiyaç duyan kodun işareti ise, fakat bazen bundan kaçınamazsınız. Bu gibi durumlarda, ilkini tercih ederim:

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

Çünkü bir satırda neler olduğunu söyleyebiliyorsun. Ancak, mümkün olduğunda böyle bir şey yapmayı tercih ederim:

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

if (foo()) { ... }

Devam edenleri belirtmek için operatörleri sonunda tutmayı seviyorum:

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

Anlamlı değişken isimlerinin büyük bir hayranıyım:

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

if (isInAStrangeCondition) {
    thud();
}

Veya yukarıda belirtildiği gibi bir fonksiyon olarak refaktör.

Messier alt ifadelerini veya hepsini bool değişkenleri olarak ayırırım. Sonra 'if' ifadesinin üst düzey boolik mantığı açıklığa kavuşturulabilir. Yaptığım işlerde, her zaman ORed veya ANDed gibi birkaç şey olmaz.

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

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

Bu, özellikle 'if' komutunu çalıştırmadan önce tüm bölümlere bakabileceğim bir hata ayıklayıcıda iyidir.

Operatörleri yeni satırların başında hizalama eğilimindeyim, böylece terimleri nasıl birleştirdiğimi hatırlıyorum (hem uzun mantık hem de uzun aritmetik için). Bunun gibi, böyle:

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

Bu, yalnızca 2 boşlukla girintili olursam veya ortamımı çok satırlı tahminleri girintiye ayarladıysam çalışır, aksi halde yüklemenin bittiği ve kullanışlı kodun nerede başladığını söylemek zor olabilir.

Aşağıdakilerin hayranıyım:

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

Bu şekilde, yine de bir if ifadesi gibi görünür, ifadede ise parçalara ayrılmaz. Girinti, önceki satırın bir devamı olduğunu göstermeye yardımcı olur.

Açma braketi önceki satırın sonuna gelinceye kadar girintili yapabilirsiniz, böylece olması gerektiği gibi if ifadesinin sonunda olur.