Foreach döngüsünde null olup olmadığını kontrol edin

Aşağıdakileri yapmanın daha güzel bir yolu var mı:
Dosyada boş olması için bir kontrol yapmam gerekiyor. Döngüye devam etmeden önce başlıklar

if (file.Headers != null)
{
  foreach (var h in file.Headers)
  {
   //set lots of properties & some other stuff
  }
}

Kısacası, kodumdaki girinti seviyesinden dolayı foreach'i if'in içine yazmak biraz çirkin görünüyor.

Değerlenecek bir şey

foreach(var h in (file.Headers != null))
{
  //do stuff
}

mümkün?

Tıpkı Rune'un önerisine küçük bir kozmetik ek olarak, kendi uzatma yönteminizi oluşturabilirsiniz:

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

O zaman yazabilirsiniz:

foreach (var header in file.Headers.OrEmptyIfNull())
{
}

İsmi zevkinize göre değiştirin :)

Dosyadaki öğelerin türünün Başlıklar T olduğunu varsayarsak, bunu yapabilirsiniz

foreach(var header in file.Headers ?? Enumerable.Empty<T>()){
  //do stuff
}

bu boş bir T if dosya numaralandırması oluşturur. Başlıklar null. Dosya türü sahip olduğunuz bir türse, Headersbunun yerine alıcıyı değiştirmeyi düşünün . nullbilinmeyen değeridir, bu nedenle mümkünse null kullanmak yerine "Öğelerin olmadığını biliyorum" yerine null (/ orijinal) "herhangi bir öğe olup olmadığını bilmiyorum" olarak yorumlanmalıdır, göstermek için boş bir küme kullanın sette hiç öğe olmadığını bildiğinizi. Bu aynı zamanda DRY'er olurdu, çünkü sıfır kontrolünü sık sık yapmanız gerekmeyecek.

Jons önerisinin devamı olarak DÜZENLE , ayrıca yukarıdaki kodu değiştirerek bir uzantı yöntemi de oluşturabilirsiniz.

foreach(var header in file.Headers.OrEmptyIfNull()){
  //do stuff
}

Alıcıyı değiştiremediğiniz durumda, operasyona bir isim vererek (OrEmptyIfNull) niyeti daha net ifade ettiği için bu benim tercihim olur.

Yukarıda bahsedilen uzantı yöntemi, optimize edicinin algılamasını imkansız kılabilir. Özellikle, yöntem aşırı yükleme kullanan IList ile ilgili olanlar ortadan kaldırılabilir.

public static IList<T> OrEmptyIfNull<T>(this IList<T> source)
{
    return source ?? Array.Empty<T>();
}

Açıkçası ben şunu tavsiye ederim: nulltesti tamamlayın . Bir nulltest yalnızca bir brfalseveya brfalse.s; her şey (gereksiz testler, ödevler ekstra yöntem çağrıları, çok fazla iş dahil etmek için gidiyor GetEnumerator(), MoveNext(), Dispose()yineleyici üzerinde, vs).

Bir iftest basit, açık ve etkilidir.

yineleme iyi olmadan önceki "eğer", bu "güzel" anlambilimlerin birkaçı kodunuzu daha az okunabilir hale getirebilir.

her neyse, girinti sizi rahatsız ediyorsa, kontrol edilip edilmeyeceğini değiştirebilirsiniz:

if(file.Headers == null)  
   return;

ve foreach döngüsüne yalnızca headers özelliğinde gerçek bir değer olduğunda ulaşırsınız.

Düşünebileceğim başka bir seçenek de foreach döngünüzde sıfır birleştirme operatörünü kullanmak ve sıfır denetiminden tamamen kaçınmaktır. örneklem:

List<int> collection = new List<int>();
collection = null;
foreach (var i in collection ?? Enumerable.Empty<int>())
{
    //your code here
}

(koleksiyonu gerçek nesneniz / türünüzle değiştirin)

Standart foreach döngüsünden daha hızlı çalışan Null-conditional Operator ve ForEach () kullanma .
Yine de koleksiyonu Listeye atmanız gerekiyor.

   listOfItems?.ForEach(item => // ... );

Bu senaryolar için güzel bir küçük genişletme yöntemi kullanıyorum:

  public static class Extensions
  {
    public static IList<T> EnsureNotNull<T>(this IList<T> list)
    {
      return list ?? new List<T>();
    }
  }

Üstbilgilerin liste türünde olduğu göz önüne alındığında, şunları yapabilirsiniz:

foreach(var h in (file.Headers.EnsureNotNull()))
{
  //do stuff
}

Bazı durumlarda, kural olarak, varsayılan koleksiyon oluşturucularının boş örnekler döndürdüğünü varsayarak, biraz başka, genel bir varyantı tercih ederim.

Bu yöntemi adlandırmak daha iyi olacaktır NewIfDefault. Yalnızca koleksiyonlar için faydalı olmayabilir, bu nedenle tür kısıtlaması IEnumerable<T>gereksiz olabilir.

public static TCollection EmptyIfDefault<TCollection, T>(this TCollection collection)
        where TCollection: class, IEnumerable<T>, new()
    {
        return collection ?? new TCollection();
    }