Tại sao vòng lặp .NET foreach ném NullRefException khi bộ sưu tập là null?

Vì vậy, tôi thường xuyên gặp phải tình huống này ... nơi Do.Something(...)trả về một bộ sưu tập null, như vậy:

int[] returnArray = Do.Something(...);

Sau đó, tôi cố gắng sử dụng bộ sưu tập này như vậy:

foreach (int i in returnArray)
{
    // do some more stuff
}

Tôi chỉ tò mò, tại sao một vòng lặp foreach không thể hoạt động trên một bộ sưu tập null? Nó có vẻ hợp lý với tôi rằng 0 lần lặp sẽ được thực hiện với một bộ sưu tập null ... thay vào đó nó ném a NullReferenceException. Bất cứ ai cũng biết tại sao điều này có thể được?

Điều này thật khó chịu vì tôi đang làm việc với các API không rõ ràng về chính xác những gì họ trả lại, vì vậy tôi kết thúc với if (someCollection != null)mọi nơi ...

Chỉnh sửa: Cảm ơn tất cả các bạn đã giải thích về việc foreachsử dụng GetEnumeratorvà nếu không có điều tra viên để nhận, foreach sẽ thất bại. Tôi đoán tôi đang hỏi tại sao ngôn ngữ / thời gian chạy không thể hoặc sẽ không thực hiện kiểm tra null trước khi lấy liệt kê. Dường như với tôi rằng hành vi vẫn sẽ được xác định rõ.

Vâng, câu trả lời ngắn gọn là "bởi vì đó là cách các nhà thiết kế trình biên dịch đã thiết kế nó." Trên thực tế, mặc dù, đối tượng bộ sưu tập của bạn là null, vì vậy không có cách nào để trình biên dịch có thể đưa trình liệt kê lặp qua bộ sưu tập.

Nếu bạn thực sự cần phải làm một cái gì đó như thế này, hãy thử toán tử hợp nhất null:

int[] array = null;

foreach (int i in array ?? Enumerable.Empty<int>())
{
   System.Console.WriteLine(string.Format("{0}", i));
}

Một foreachvòng lặp gọi GetEnumeratorphương thức.
Nếu bộ sưu tập là null, phương thức này gọi kết quả là a NullReferenceException.

Đó là thực tế xấu để trả lại một nullbộ sưu tập; phương pháp của bạn sẽ trả về một bộ sưu tập trống thay thế.

Có một sự khác biệt lớn giữa một bộ sưu tập trống và một tài liệu tham khảo null cho một bộ sưu tập.

Khi bạn sử dụng foreach, trong nội bộ, đây là cách gọi phương thức GetEnumerator () của IEnumerable . Khi tham chiếu là null, điều này sẽ đưa ra ngoại lệ này.

Tuy nhiên, nó hoàn toàn hợp lệ để có một sản phẩm nào IEnumerablehoặc IEnumerable<T>. Trong trường hợp này, foreach sẽ không "lặp lại" bất cứ điều gì (vì bộ sưu tập trống), nhưng nó cũng sẽ không ném, vì đây là một kịch bản hoàn toàn hợp lệ.

Biên tập:

Cá nhân, nếu bạn cần giải quyết vấn đề này, tôi khuyên bạn nên sử dụng một phương pháp mở rộng:

public static IEnumerable<T> AsNotNull<T>(this IEnumerable<T> original)
{
     return original ?? Enumerable.Empty<T>();
}

Sau đó bạn có thể gọi:

foreach (int i in returnArray.AsNotNull())
{
    // do some more stuff
}

Nó đang được trả lời từ lâu nhưng tôi đã cố gắng làm điều này theo cách sau để tránh ngoại lệ con trỏ null và có thể hữu ích cho ai đó sử dụng toán tử kiểm tra C # null ?.

     //fragments is a list which can be null
     fragments?.ForEach((obj) =>
        {
            //do something with obj
        });

Một phương pháp mở rộng khác để khắc phục điều này:

public static void ForEach<T>(this IEnumerable<T> items, Action<T> action)
{
    if(items == null) return;
    foreach (var item in items) action(item);
}

Tiêu thụ theo nhiều cách:

(1) với phương thức chấp nhận T:

returnArray.ForEach(Console.WriteLine);

(2) với một biểu thức:

returnArray.ForEach(i => UpdateStatus(string.Format("{0}% complete", i)));

(3) với phương pháp ẩn danh nhiều dòng

int toCompare = 10;
returnArray.ForEach(i =>
{
    var thisInt = i;
    var next = i++;
    if(next > 10) Console.WriteLine("Match: {0}", i);
});

Chỉ cần viết một phương thức mở rộng để giúp bạn hiểu:

public static class Extensions
{
   public static void ForEachWithNull<T>(this IEnumerable<T> source, Action<T> action)
   {
      if(source == null)
      {
         return;
      }

      foreach(var item in source)
      {
         action(item);
      }
   }
}

Bởi vì bộ sưu tập null không giống với bộ sưu tập trống. Một bộ sưu tập trống là một đối tượng bộ sưu tập không có yếu tố; một bộ sưu tập null là một đối tượng không tồn tại.

Đây là một cái gì đó để thử: Khai báo hai bộ sưu tập thuộc bất kỳ loại nào. Khởi tạo một cái bình thường để nó trống và gán giá trị kia null. Sau đó thử thêm một đối tượng vào cả hai bộ sưu tập và xem điều gì sẽ xảy ra.

Đó là lỗi của Do.Something(). Cách thực hành tốt nhất ở đây sẽ là trả về một mảng có kích thước 0 (điều đó là có thể) thay vì null.

Bởi vì đằng sau hậu trường, foreachmột người điều tra, tương đương với điều này:

using (IEnumerator<int> enumerator = returnArray.getEnumerator()) {
    while (enumerator.MoveNext()) {
        int i = enumerator.Current;
        // do some more stuff
    }
}

Tôi nghĩ rằng lời giải thích tại sao ngoại lệ được ném là rất rõ ràng với các câu trả lời được cung cấp ở đây. Tôi chỉ muốn bổ sung với cách tôi thường làm việc với các bộ sưu tập của người Đức. Bởi vì, đôi khi, tôi sử dụng bộ sưu tập nhiều hơn một lần và phải kiểm tra nếu null mỗi lần. Để tránh điều đó, tôi làm như sau:

    var returnArray = DoSomething() ?? Enumerable.Empty<int>();

    foreach (int i in returnArray)
    {
        // do some more stuff
    }

Bằng cách này, chúng tôi có thể sử dụng bộ sưu tập nhiều như chúng tôi muốn mà không sợ ngoại lệ và chúng tôi không xử lý mã với các tuyên bố có điều kiện quá mức.

Sử dụng toán tử kiểm tra null ?.cũng là một cách tiếp cận tuyệt vời. Nhưng, trong trường hợp mảng (như ví dụ trong câu hỏi), nó nên được chuyển thành Danh sách trước:

    int[] returnArray = DoSomething();

    returnArray?.ToList().ForEach((i) =>
    {
        // do some more stuff
    });

SPListItem item;
DataRow dr = datatable.NewRow();

dr["ID"] = (!Object.Equals(item["ID"], null)) ? item["ID"].ToString() : string.Empty;