Mengapa .NET foreach loop melempar NullRefException saat koleksi bernilai nol?

Jadi saya sering mengalami situasi ini ... di mana Do.Something(...)mengembalikan koleksi nol, seperti:

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

Lalu, saya mencoba menggunakan koleksi ini seperti ini:

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

Saya hanya ingin tahu, mengapa loop foreach tidak dapat beroperasi pada koleksi nol? Tampaknya logis bagi saya bahwa 0 iterasi akan dieksekusi dengan koleksi nol ... alih-alih melempar a NullReferenceException. Adakah yang tahu mengapa ini bisa terjadi?

Ini menjengkelkan karena saya bekerja dengan API yang tidak jelas persis apa yang mereka kembalikan, jadi saya berakhir di if (someCollection != null)mana - mana ...

Sunting: Terima kasih semua karena menjelaskan foreachkegunaan itu GetEnumeratordan jika tidak ada enumerator yang mendapatkan, foreach akan gagal. Saya kira saya bertanya mengapa bahasa / runtime tidak bisa atau tidak akan melakukan pemeriksaan nol sebelum meraih enumerator. Tampak bagi saya bahwa perilaku itu masih akan didefinisikan dengan baik.

Baiklah, jawaban singkatnya adalah "karena memang itulah yang dirancang oleh perancang kompiler." Namun, secara realistis, objek koleksi Anda adalah nol, jadi tidak ada cara bagi kompiler untuk membuat enumerator untuk mengulangi koleksi.

Jika Anda benar-benar perlu melakukan sesuatu seperti ini, coba operator penggabungan nol:

int[] array = null;

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

Sebuah foreachloop memanggil GetEnumeratormetode.
Jika pengumpulannya adalah null, pemanggilan metode ini menghasilkan a NullReferenceException.

Merupakan praktik yang buruk untuk mengembalikan nullkoleksi; metode Anda harus mengembalikan koleksi kosong.

Ada perbedaan besar antara koleksi kosong dan referensi nol untuk koleksi.

Ketika Anda menggunakan foreach, secara internal, ini memanggil metode GetEnumerator () dari IEnumerable . Ketika referensi nol, ini akan memunculkan pengecualian ini.

Namun, itu sangat sah untuk memiliki IEnumerableatau IEnumerable<T>. Dalam hal ini, foreach tidak akan "beralih" pada apa pun (karena koleksinya kosong), tetapi juga tidak akan melempar, karena ini adalah skenario yang benar-benar valid.

Edit:

Secara pribadi, jika Anda perlu mengatasinya, saya akan merekomendasikan metode ekstensi:

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

Anda kemudian dapat menghubungi:

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

Ini sedang dijawab lama tetapi saya telah mencoba melakukan ini dengan cara berikut untuk hanya menghindari pengecualian null pointer dan mungkin berguna bagi seseorang yang menggunakan operator cek C # null?

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

Metode ekstensi lain untuk mengatasi ini:

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

Konsumsi dalam beberapa cara:

(1) dengan metode yang menerima T:

returnArray.ForEach(Console.WriteLine);

(2) dengan ekspresi:

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

(3) dengan metode anonim multiline

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

Cukup tulis metode ekstensi untuk membantu Anda:

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);
      }
   }
}

Karena koleksi nol tidak sama dengan koleksi kosong. Koleksi kosong adalah objek koleksi tanpa elemen; koleksi nol adalah objek yang tidak ada.

Ini ada sesuatu yang bisa dicoba: Nyatakan dua koleksi apa pun. Inisialisasi satu secara normal sehingga kosong, dan tetapkan nilainya yang lain null. Kemudian coba tambahkan objek ke kedua koleksi dan lihat apa yang terjadi.

Itu adalah kesalahan Do.Something(). Praktik terbaik di sini adalah mengembalikan array ukuran 0 (itu mungkin) daripada nol.

Karena di belakang layar yang foreachmemperoleh enumerator, setara dengan ini:

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

Saya pikir penjelasan mengapa pengecualian dilemparkan sangat jelas dengan jawaban yang diberikan di sini. Saya hanya ingin melengkapi dengan cara saya biasanya bekerja dengan koleksi ini. Karena, beberapa kali, saya menggunakan koleksi lebih dari sekali dan harus menguji apakah nol setiap kali. Untuk menghindarinya, saya melakukan hal berikut:

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

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

Dengan cara ini kita dapat menggunakan koleksi sebanyak yang kita inginkan tanpa takut pengecualian dan kita tidak mencemari kode dengan pernyataan kondisional berlebihan.

Menggunakan operator cek nol ?.juga merupakan pendekatan yang bagus. Tetapi, dalam hal array (seperti contoh dalam pertanyaan), harus diubah menjadi Daftar sebelumnya:

    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;