ข้อไหนคือการบำรุงรักษามากขึ้น - การกำหนดบูลีนผ่านทาง if / else หรือการแสดงออกบูลีน?

สิ่งใดที่จะถือว่าได้รับการบำรุงรักษามากขึ้น

if (a == b) c = true; else c = false;

หรือ

 c = (a == b);

ฉันลองดูใน Code Complete แต่ไม่พบคำตอบ

ฉันคิดว่าอันแรกสามารถอ่านได้มากขึ้น (คุณสามารถอ่านออกเสียงออกมาได้ดัง ๆ ) ซึ่งฉันคิดว่าทำให้สามารถบำรุงรักษาได้มากขึ้น อันที่สองมีความเหมาะสมมากกว่าและลดรหัส แต่ฉันไม่แน่ใจว่ามันจะบำรุงรักษาได้สำหรับนักพัฒนา C # (ฉันคาดว่าจะเห็นสำนวนนี้มากขึ้นเช่น Python)

ตัวเลือกที่สองจะดีกว่า

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

อย่างไรก็ตามฉันไม่คิดว่าc = (a == b);จะเป็นตัวอย่างของเคล็ดลับที่ฉลาด มันเป็นการนำเสนอแนวคิดที่เรียบง่าย ตรงไปตรงมาเท่าที่คุณจะได้รับ

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

if (a == b)
{
    c = true; 
}
else 
{
    c = false;
}

จากประสบการณ์ของฉันการเขียนตรรกะบูลีนอย่างง่ายในลักษณะที่ละเอียดและผิดพลาดเป็นสัญญาณของรหัส "iffy" มันจะทำให้ฉันสงสัยว่าตรรกะที่ซับซ้อนมากขึ้นจะถูกจัดการในรหัสฐานนี้

ก่อนอื่นให้ตระหนักว่ารูปแบบทั้งสองของคุณนั้นไม่เทียบเท่ากัน

if (a == b) c = true;

cจะถูกตั้งค่าเป็นจริงถ้าaเท่ากับbและถ้าไม่มูลค่าของมันจะยังคงอยู่ไม่ว่ามันจะเป็นอะไรก็ตาม

c = (a == b);

cจะถูกตั้งค่าเป็นจริงถ้าaเท่ากับbและถ้าไม่มันจะถูกตั้งค่าเป็นเท็จ

หากคุณต้องการรูปแบบที่เทียบเท่าในรูปแบบของรูปแบบแรกคุณต้องเขียนดังนี้:

if (a == b) {
  c = true;
} else c = false;

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

ฉันไม่เห็นด้วยว่ารูปแบบแรกของคุณอ่านได้ง่ายกว่า - แน่นอนไม่ใช่ C # ที่มีสองประโยคในบรรทัดเดียวและไม่แนะนำให้มีifคำสั่งโดยไม่ใช้วงเล็บปีกกา

ประการที่สองฉันไม่เห็นว่ารูปแบบที่สองสามารถบำรุงรักษาได้น้อยลง - ไม่มีสิ่งใดที่ต้องดูแล มันเป็นคำสั่งที่เรียบง่ายของความสัมพันธ์ระหว่างaและbและมันอาจไม่ได้รับการแสดงใด ๆ เพิ่มเติมเพียง

อีกเหตุผลที่ชอบฟอร์มที่สองคือคุณสามารถประกาศcและกำหนดมันในคำสั่งเดียวเช่น

bool c = (a == b);

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

"บำรุงรักษาได้ดีกว่า" อาจเป็นเรื่องส่วนตัวมาก

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

การใช้ภาษาและวัฒนธรรมรอบ ๆ ภาษาเป็นลักษณะของ 'ความสามารถในการอ่าน' ในความคิดของฉัน

มีบางครั้งที่ประสิทธิภาพอาจลดรหัสเพื่อเพิ่มประสิทธิภาพโค้ดไบต์ แต่ควรทำอย่างระมัดระวังหลังจากทำโปรไฟล์

ที่สอง. มันมีการทำซ้ำน้อยกว่า (DRY) และง่ายต่อการเข้าใจสิ่งที่เกิดขึ้นซึ่งcถือเป็นมูลค่าของหรือไม่aและจะbเท่ากัน

IMHO จะดีกว่านี้

c = a == b

เช่นเดียวกับฉันจะเขียน

• 1 + 2 + 3 แทน ((1 + 2) + 3)
• 5 + 3 * 7 แทน (5 + (3 * 7))

รหัสที่ไม่จำเป็นอย่างชัดเจนและไม่สำคัญคือคุณธรรม มันรก

ผู้ลงคะแนนโปรดอธิบายสิ่งที่ผิดกับคำตอบที่แก้ไขของฉัน

ใช่c = (a == b);อาจจะยากที่จะอ่าน (แย่ลงยัง StyleCop บ่นเกี่ยวกับวงเล็บที่ไม่จำเป็น) a == bแต่ฉันยังคงชอบความเรียบง่ายของ ดังนั้นนี่คือสิ่งที่ฉันต้องการใช้เมื่อทั้งสองaและbเหมือนกัน:

private static bool NoPeriod
{
    get
    {
        return this.MyWaveLength == this.HerWaveLength;
    }
}

จากนั้นคุณสามารถทำได้: this.c = this.NoPeriodแทน:

this.c = this.MyWavelength == this.HerWaveLength;