Eğer / else ifadeleri davaların azlığı veya onlarla başa çıkma zorluğu ile düzenlenmeli mi?

Şu anda yazıyorum bazı kod, böyle bir şey var:

if (uncommon_condition) {
    do_something_simple();
} else {
    do();
    something();
    long();
    and();
    complicated();
}

Bir parçam "Yazıldığı gibi iyi. Basit vakalar önce gitmeli, daha karmaşık vakalar da gitmeli." Ama başka bir bölüm şöyle diyor: "Hayır! elseKod if, ifalışılmadık davalarla elseuğraşmak ve diğer tüm davalarla uğraşmak içindir." Hangisi doğru veya tercih edilir?

Düzen olma olasılıklarına göre düzen. En yaygın, en olası vb. Durumlar önce yapılmalıdır.

"Onlarla başa çıkma zorluğu" örnekte kod yapısı, soyutlama vb. İle ele alınmalıdır. ifMaddelerinizin aynı soyut seviyede olmasını istiyorsunuz .

if ( ! LotteryWinner ) {
    GoToWorkMonday();
} else {
    PlanYearLongVacation();
}

Okunabilirliği artırmaya çalışın. Bunun bir yolu daha uzun kod bloğunu diğer parçaya yerleştirmektir.

if (s == null)
     // short code
else 
     // long 
     // code
     // block

daha okunabilir

if (s != null)
    // long
    // code
    // block
else
    // short code

Kullanım hakkında duyduğum gibi sabit bir kural yok ama bunu takip ediyorum

if(usual)
{
(more often)
}
else (unusual)
{
(rarely occurring)
}

Ama her ikisi de farklı özelliklere sahip aynı işleve sahipse, daha sonra olağandışı ilk önce normalden sonra gidin, böylece bir talimat kaydedebilirsiniz.

if(x == 0)  // 1
  {x = 1;}  // 2
else
  {x = 2;}  // 3

Yukarıdaki kod montaj kodu için böyle bir şey olacaktır:

1. 000d 837DFC00        cmpl    $0, -4(%ebp)
   0011 7509            jne .L2

2. 0013 C745FC01        movl    $1, -4(%ebp)
   001a EB07            jmp .L3

    .L2:
3.001c C745FC02         movl    $2, -4(%ebp)

        .L3:

İçerideki koşul doğruysa akış 1-> 2 (4 yapı)
İçeride koşul yanlışsa akış 1-> 3'tür (3 yapı)

Bu yüzden, her seferinde bir talimat kaydedebilmemiz için başka bir bölüm ve normal durumda olağandışı veya nadiren meydana gelen olayları koymak daha iyidir ;-)

Ben tam ters desen kodunu okumak daha kolay yol açar ve ifadeler iç içe azaltır veya ortadan kaldırır bulduk. Ben buna "dayağı" deseni denir. (Hikaye anlatmada, bir dayağı, son görev tamamlanmadan önce başarılı bir şekilde yerine getirilmesi gereken bir dizi zorluk olacaktır.) Önce kenar davalarınızı ele alarak, kodunuzun ana gövdesinin temiz ve özlü olmasına izin verirsiniz:

if(gauntlet_1){ handle the first gauntlet condition }; 
if(gauntlet_2){ handle the second gauntlet condition };
...
// All preconditions (gauntlets) have been successfully handled

// Perform your main task here

Benim için yürüttükleri kodun karmaşıklığından çok koşullar hakkında. İlk önce olağandışı koşulları tuzağa düşürmek için daha sonra daha yaygın olanları sipariş etmeniz gerekir. Ayrıca, başka kod gerçekten uzun ve karmaşıksa, koşullu kısmı okuyucu için açık tutmak için muhtemelen bir alt yapardım.

Sabit bir kuralım yok, ama genel olarak bunu takip ediyorum - önce, bunu hesaba katacak kaçırılmış bir tasarım deseninin nerede olduğunu düşünün. Değilse, eğer yazarımın durumu en açık şekilde anlaşılacak şekilde yazarım. Yani, çift negatiflerden ve benzerlerinden kaçınmak.

Bu tür senaryolara gelince, bunun için bir başparmak kuralı yoktur. Ancak muhtemelen if bloğundaki pozitif veya büyük olasılıkla kodu yazıp diğer blok veya varsayılan durumlarda dışarıda bırakabiliriz.