การกรองลูป foreach ด้วยเงื่อนไขที่ vs clause ต่อไป

ฉันเห็นโปรแกรมเมอร์บางคนใช้สิ่งนี้:

foreach (var item in items)
{
    if (item.Field != null)
        continue;

    if (item.State != ItemStates.Deleted)
        continue;

    // code
}

แทนที่จะใช้ที่ฉันปกติ:

foreach (var item in items.Where(i => i.Field != null && i.State != ItemStates.Deleted))
{
    // code
}

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

ฉันจะเชื่อว่านี่เป็นสถานที่ที่เหมาะสมที่จะใช้คำสั่งแยกแบบสอบถาม / ตัวอย่างเช่น:

// query
var validItems = items.Where(i => i.Field != null && i.State != ItemStates.Deleted);
// command
foreach (var item in validItems) {
    // do stuff
}

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

เมื่อทำการดีบั๊กคุณสามารถหยุดก่อนที่foreachจะตรวจสอบได้อย่างรวดเร็วว่าเนื้อหาของการvalidItemsแก้ไขสิ่งที่คุณคาดหวัง คุณไม่จำเป็นต้องก้าวเข้าไปในแลมบ์ดาเว้นแต่ว่าคุณต้องการ หากคุณจำเป็นต้องก้าวเข้าไปในแลมบ์ดาฉันขอแนะนำให้แยกออกเป็นฟังก์ชั่นแยกต่างหากจากนั้นค่อยก้าวผ่านสิ่งนั้นแทน

มีความแตกต่างในประสิทธิภาพหรือไม่ หากแบบสอบถามได้รับการสนับสนุนโดยฐานข้อมูลดังนั้นรุ่น LINQ มีโอกาสที่จะทำงานได้เร็วขึ้นเนื่องจากแบบสอบถาม SQL อาจมีประสิทธิภาพมากกว่า หากเป็น LINQ ไปยัง Objects คุณจะไม่เห็นความแตกต่างด้านประสิทธิภาพที่แท้จริง เช่นเคยโปรไฟล์รหัสของคุณและแก้ไขปัญหาคอขวดที่มีการรายงานจริงแทนที่จะพยายามทำนายการเพิ่มประสิทธิภาพล่วงหน้า

แน่นอนว่าประสิทธิภาพการทำงานแตกต่างกัน.Where()ส่งผลให้มีการโทรแบบมอบหมายสำหรับทุกรายการ อย่างไรก็ตามฉันไม่ต้องกังวลเลยเกี่ยวกับประสิทธิภาพ:

• รอบสัญญาณนาฬิกาที่ใช้ในการเรียกผู้แทนมีความสำคัญน้อยเมื่อเทียบกับรอบสัญญาณนาฬิกาที่ใช้โดยรหัสที่เหลือที่วนซ้ำในการรวบรวมและตรวจสอบเงื่อนไข

• บทลงโทษที่แสดงถึงการเรียกผู้ร่วมประชุมนั้นเป็นไปตามลำดับของรอบสัญญาณนาฬิกาสองสามรอบและโชคดีที่เราต้องผ่านพ้นวันเวลาที่เราต้องกังวลเกี่ยวกับรอบนาฬิกาแต่ละรอบ

^{หากมีเหตุผลบางอย่างที่ประสิทธิภาพการทำงานจริงๆสิ่งสำคัญสำหรับคุณในระดับวงจรนาฬิกาแล้วใช้List<Item>แทนIList<Item>เพื่อให้คอมไพเลอร์สามารถใช้โดยตรง (และ inlinable) เรียกแทนการโทรเสมือนจริงและเพื่อให้ iterator ของList<T>ที่เป็นจริงstructไม่ได้จะต้องมีการบรรจุกล่อง แต่นั่นเป็นเรื่องขี้ประติ๋วจริงๆ}

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

คุณจะเห็นประโยชน์ของif(x) continue;ช่วงเวลาที่คุณต้องแก้ไขข้อบกพร่องของรหัสนี้: การทำงานแบบif()s และcontinues อย่างเดียว การก้าวเข้าสู่ผู้แทนจากการกรองเป็นความเจ็บปวด