ถ้า (items! = null) ฟุ่มเฟือยก่อน foreach (รายการ T ในรายการ)?

ฉันมักจะเจอรหัสดังต่อไปนี้:

if ( items != null)
{
   foreach(T item in items)
   {
        //...
   }
}

โดยทั่วไปifเงื่อนไขจะทำให้แน่ใจว่าforeachบล็อกจะดำเนินการต่อเมื่อitemsไม่เป็นโมฆะ ฉันสงสัยว่าifเงื่อนไขนั้นจำเป็นจริงๆหรือforeachจะจัดการกับกรณีนี้หรือitems == nullไม่

ฉันหมายความว่าฉันสามารถเขียน

foreach(T item in items)
{
    //...
}

โดยไม่ต้องกังวลว่าจะitemsเป็นโมฆะหรือไม่? เป็นifฟุ่มเฟือยสภาพ? หรือขึ้นอยู่กับประเภทของitemsหรืออาจจะTด้วย?

คุณยังต้องตรวจสอบว่า (items! = null) ไม่เช่นนั้นคุณจะได้รับ NullReferenceException อย่างไรก็ตามคุณสามารถทำสิ่งนี้ได้:

List<string> items = null;  
foreach (var item in items ?? new List<string>())
{
    item.Dump();
}

แต่คุณอาจตรวจสอบประสิทธิภาพของมัน ดังนั้นฉันยังคงชอบมี if (items! = null) ก่อน

ตามคำแนะนำของ Lippert ของ Eric ฉันเปลี่ยนรหัสเป็น:

List<string> items = null;  
foreach (var item in items ?? Enumerable.Empty<string>())
{
    item.Dump();
}

การใช้ C # 6 คุณสามารถใช้ตัวดำเนินการเงื่อนไขว่างใหม่ร่วมกับList<T>.ForEach(Action<T>)(หรือIEnumerable<T>.ForEachวิธีการขยายของคุณเอง)

List<string> items = null;
items?.ForEach(item =>
{
    // ...
});

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

หากลำดับไม่เป็นโมฆะแน่นอนคุณไม่จำเป็นต้องตรวจสอบ

จริงๆแล้วมีการร้องขอคุณสมบัติใน @Connect นั้น: http://connect.microsoft.com/VisualStudio/feedback/details/93497/foreach-should-check-for-null

และการตอบสนองค่อนข้างสมเหตุสมผล:

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

คุณสามารถทดสอบด้วยรายการว่างได้ตลอดเวลา ... แต่นี่คือสิ่งที่ฉันพบในเว็บไซต์ msdn

foreach-statement:
    foreach   (   type   identifier   in   expression   )   embedded-statement 

ถ้านิพจน์มีค่า null System.NullReferenceException จะถูกโยนทิ้ง

มันไม่ฟุ่มเฟือย ในรายการรันไทม์จะถูกแคสต์ไปยัง IEnumerable และเมธอด GetEnumerator จะถูกเรียกใช้ ซึ่งจะทำให้การอ้างอิงรายการที่จะล้มเหลว

คุณสามารถห่อหุ้มการตรวจสอบค่าว่างในวิธีการขยายและใช้แลมด้า:

public static class EnumerableExtensions {
  public static void ForEach<T>(this IEnumerable<T> self, Action<T> action) {
    if (self != null) {
      foreach (var element in self) {
        action(element);
      }
    }
  }
}

รหัสจะกลายเป็น:

items.ForEach(item => { 
  ...
});

หากสามารถกระชับได้มากขึ้นหากคุณต้องการเรียกใช้เมธอดที่รับไอเท็มและส่งคืนvoid:

items.ForEach(MethodThatTakesAnItem);

คุณต้องการสิ่งนี้ คุณจะได้รับข้อยกเว้นเมื่อforeachเข้าถึงคอนเทนเนอร์เพื่อตั้งค่าการทำซ้ำเป็นอย่างอื่น

ภายใต้ฝาครอบforeachใช้อินเทอร์เฟซที่ติดตั้งบนคลาสคอลเลกชันเพื่อดำเนินการวนซ้ำ อินเตอร์เฟซที่เทียบเท่าทั่วไปคือที่นี่

คำสั่ง foreach ของภาษา C # (สำหรับแต่ละรายการใน Visual Basic) ซ่อนความซับซ้อนของตัวนับ ดังนั้นจึงแนะนำให้ใช้ foreach แทนการจัดการกับตัวแจงนับโดยตรง

การทดสอบเป็นสิ่งที่จำเป็นเพราะถ้าคอลเล็กชันเป็นโมฆะ foreach จะส่ง NullReferenceException มันค่อนข้างง่ายที่จะทดลองใช้

List<string> items = null;
foreach(var item in items)
{
   Console.WriteLine(item);
}

ครั้งที่สองจะโยนNullReferenceExceptionข้อความObject reference not set to an instance of an object.

ดังที่กล่าวไว้ที่นี่คุณต้องตรวจสอบว่ามันไม่เป็นโมฆะหรือไม่

อย่าใช้นิพจน์ที่ประเมินว่าเป็นโมฆะ

ใน C # 6 คุณสามารถเขียน sth ดังนี้:

// some string from file or UI, i.e.:
// a) string s = "Hello, World!";
// b) string s = "";
// ...
var items = s?.Split(new char[] { ',', '!', ' ' }) ?? Enumerable.Empty<string>();  
foreach (var item in items)
{
    //..
}

โดยพื้นฐานแล้วเป็นวิธีแก้ปัญหาของ Vlad Bezden แต่ใช้ ?? นิพจน์เพื่อสร้างอาร์เรย์ที่ไม่เป็นโมฆะเสมอดังนั้นจึงอยู่รอดของ foreach แทนที่จะมีการตรวจสอบนี้ภายในวงเล็บ foreach