İf (öğeler! = Null) foreach'ten önce gereksiz mi (öğelerdeki T öğesi)?

Sık sık aşağıdaki gibi kodlarla karşılaşıyorum:

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

Temel olarak ifkoşul, foreachbloğun yalnızca itemsboş değilse yürütülmesini sağlar . Durumun ifgerçekten gerekli olup olmadığını merak ediyorum , yoksa foreachdavayı halledecek mi items == null?

Demek istediğim, basitçe yazabilir miyim

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

itemsboş olup olmadığı konusunda endişelenmeden ? Mı ifkoşul gereksiz? Veya buna bağlıdır tip ait itemsbelki veya Tyanı?

Yine de (items! = Null) olup olmadığını kontrol etmeniz gerekir, aksi takdirde NullReferenceException alırsınız. Ancak bunun gibi bir şey yapabilirsiniz:

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

ancak performansını kontrol edebilirsiniz. Bu yüzden hala önce if (items! = Null) olmasını tercih ediyorum.

Eric'in Lippert önerisine dayanarak kodu şu şekilde değiştirdim:

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

C # 6'yı kullanarak, yeni boş koşullu operatörü List<T>.ForEach(Action<T>)(veya kendi IEnumerable<T>.ForEachgenişletme yönteminizle) birlikte kullanabilirsiniz.

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

Buradaki gerçek paket servis , ilk etapta neredeyse hiçbir zaman boş olmamalı bir dizi olmalıdır . Basitçe onu tüm programlarınızda bir değişmez yapın, eğer bir sıranız varsa, asla boş olmaz. Her zaman boş sıra veya başka bir gerçek sıra olarak başlatılır.

Bir dizi hiçbir zaman boş değilse, açıkça kontrol etmenize gerek yoktur.

Aslında @Connect'te bir özellik isteği var: http://connect.microsoft.com/VisualStudio/feedback/details/93497/foreach-should-check-for-null

Ve yanıt oldukça mantıklı:

Çoğu foreach döngüsünün boş olmayan bir koleksiyonu yineleme amacıyla yazıldığını düşünüyorum. Null ile yinelemeyi denerseniz, istisnanızı almalısınız, böylece kodunuzu düzeltebilirsiniz.

Her zaman boş bir liste ile test edebilirsiniz ... ama msdn web sitesinde bulduğum şey bu

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

İfade null değerine sahipse, bir System.NullReferenceException oluşturulur.

Gereksiz değil. Çalışma zamanında öğeler bir IEnumerable'a dönüştürülecek ve GetEnumerator yöntemi çağrılacaktır. Bu, başarısız olacak öğelerin referans alınmasına neden olur

Boş denetimi bir uzantı yönteminde kapsülleyebilir ve bir lambda kullanabilirsiniz:

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

Kod şu hale gelir:

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

Sadece bir öğeyi alıp geri döndüren bir yöntemi çağırmak istiyorsanız daha da kısa olabilir void:

items.ForEach(MethodThatTakesAnItem);

Buna ihtiyacın var. foreachAksi takdirde yinelemeyi ayarlamak için konteynere eriştiğinizde bir istisna alırsınız .

Kapaklar altında, yinelemeyi gerçekleştirmek için koleksiyon sınıfında uygulanan bir arabirimforeach kullanır . Jenerik eşdeğer arayüz burada .

C # dilinin foreach ifadesi (Visual Basic'teki her biri için) numaralandırıcıların karmaşıklığını gizler. Bu nedenle, numaralandırıcıyı doğrudan manipüle etmek yerine foreach kullanılması önerilir.

Test gereklidir, çünkü koleksiyon boşsa, foreach bir NullReferenceException oluşturur. Denemek aslında oldukça basit.

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

ikincisi NullReferenceExceptionmesajla birlikte bir atacakObject reference not set to an instance of an object.

Belirtildiği gibi burada bunu boş değil kontrol etmeniz gerekir.

Null olarak değerlendirilen bir ifade kullanmayın.

C # 6'da şu şekilde sth yazabilirsiniz:

// 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)
{
    //..
}

Temelde Vlad Bezden'in çözümü ama ?? Her zaman boş olmayan ve bu nedenle foreach köşeli parantez içinde bu denetimi yapmak yerine foreach'den sağ kalan bir dizi oluşturmak için ifade.