ควรจัดทำงบ / อื่นโดยความหายากของกรณีหรือความยากลำบากในการจัดการกับพวกเขา?

ในรหัสที่ฉันกำลังเขียนตอนนี้ฉันมีสิ่งนี้:

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

ส่วนหนึ่งของฉันคิดว่า "ไม่เป็นไรวิธีเขียนกรณีง่าย ๆ ควรไปก่อนและกรณีที่ซับซ้อนกว่าควรไปต่อไป" แต่อีกส่วนหนึ่งกล่าวว่า: "ไม่! elseรหัสควรอยู่ภายใต้ifเนื่องจากifสำหรับการจัดการกับกรณีที่ผิดปกติและelseสำหรับการจัดการกับกรณีอื่น ๆ ทั้งหมด" ข้อใดถูกหรือดีกว่า?

เรียงลำดับตามโอกาสในการถูกประหารชีวิต เงื่อนไขที่พบได้บ่อยที่สุดและเป็นไปได้ ฯลฯ ควรเป็นอันดับแรก

"ความยากลำบากในการจัดการกับพวกเขา" ควรได้รับการจัดการโดยโครงสร้างของรหัส, นามธรรม, เป็นต้นในตัวอย่างบล็อกอื่นสามารถ refactored เพื่อเรียกวิธีการเดียว คุณต้องการให้ifคำสั่งของคุณอยู่ในระดับนามธรรมเดียวกัน

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

พยายามปรับปรุงความสามารถในการอ่าน วิธีหนึ่งคือการวางบล็อกโค้ดที่ยาวขึ้นในส่วนอื่น

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

สามารถอ่านได้มากกว่า

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

ไม่มีกฎตายตัวเช่นฉันได้ยินเกี่ยวกับการใช้งาน แต่ฉันทำตามเช่นนี้

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

แต่ถ้าทั้งคู่มีฟังก์ชั่นเดียวกันกับคุณสมบัติที่แตกต่างกันไปก่อนดีกว่าปกติแล้วเพื่อให้คุณสามารถบันทึกคำสั่งหนึ่ง

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

สำหรับรหัสประกอบรหัสด้านบนจะเป็นดังนี้:

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:

ถ้าเงื่อนไขภายในถ้าเป็นจริงแล้วการไหลคือ 1-> 2 (4 intructions)
หากเงื่อนไขภายในถ้าเป็นเท็จแล้วการไหลเป็น 1-> 3 (3 intructions)

ดังนั้นจึงเป็นการดีที่จะใส่เหตุการณ์ที่ผิดปกติหรือไม่ค่อยเกิดขึ้นในส่วนอื่นและสภาพปกติเพื่อให้เราสามารถบันทึกคำสั่งได้ทุกครั้ง ;-)

ฉันพบว่ารูปแบบตรงข้ามที่แน่นอนนำไปสู่การอ่านรหัสได้ง่ายขึ้นและลดหรือกำจัดคำสั่งที่ซ้อนกัน ฉันหมายถึงสิ่งนี้เป็นรูปแบบ "ถุงมือ" (ในการเล่าเรื่องถุงมือจะเป็นชุดของความท้าทายที่จะต้องประสบความสำเร็จก่อนที่งานสุดท้ายจะเสร็จสมบูรณ์) โดยการจัดการกรณีขอบของคุณก่อนคุณอนุญาตให้ตัวหลักของรหัสของคุณสะอาดและรัดกุม:

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

สำหรับฉันมันเกี่ยวกับเงื่อนไขมากกว่าความซับซ้อนของรหัสที่พวกเขาเรียกใช้ คุณจำเป็นต้องสั่งเงื่อนไขเพื่อดักจับเงื่อนไขที่ผิดปกติก่อน นอกจากนี้หากรหัสอื่นยาวและซับซ้อนจริง ๆ ฉันอาจทำย่อยเพื่อให้ส่วนที่มีเงื่อนไขชัดเจนสำหรับผู้อ่าน

ฉันไม่มีกฎตายตัว แต่โดยทั่วไปฉันทำตามสิ่งนี้ - ก่อนอื่นให้พิจารณาว่ามีรูปแบบการออกแบบที่ไม่ได้รับซึ่งจะคำนึงถึงเรื่องนี้อย่างไร ถ้าไม่ฉันเขียนมันเพื่อให้เงื่อนไขในถ้าเป็นที่ชัดเจนที่สุด นั่นคือหลีกเลี่ยงเชิงลบสองเท่าและไม่ชอบ

สำหรับสถานการณ์ดังกล่าวไม่มีกฎง่ายๆสำหรับมัน แต่อาจเป็นไปได้ว่าเราสามารถติดตามเพื่อเขียนโค้ดเชิงบวกหรือเป็นไปได้มากที่สุดใน if block และปล่อยไว้ในบล็อกอื่นหรือกรณีเริ่มต้น