Перевірте наявність нуля в циклі foreach

Чи є більш приємний спосіб зробити наступне:
мені потрібна перевірка на наявність null у файлі. Headers перед тим, як продовжити цикл

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

Коротше кажучи, виглядає трохи потворно писати foreach всередині if, якщо через рівень відступу, що відбувається в моєму коді.

Це те, що можна оцінити

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

можливо?

Просто як невелике косметичне доповнення до пропозиції Рун, ви можете створити власний метод розширення:

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

Тоді ви можете написати:

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

Змінюйте назву за смаком :)

Якщо припустити, що тип елементів у файлі. Headers - T, ви можете це зробити

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

це створить порожній перелік T, якщо file.Headers має значення null. Якщо тип файлу є типом, яким ви володієте, я, однак, міркував би про те, щоб замінити Headersзамість нього гетер . nullце значення невідомого, тому, якщо можливо, замість використання null як "я знаю, що немає елементів", коли null насправді (/ спочатку) слід інтерпретувати як "я не знаю, чи є якісь елементи", використовуйте порожній набір для показу що ви знаєте, що в наборі немає елементів. Це також буде DRY'er, оскільки вам не доведеться робити нульову перевірку так часто.

РЕДАКТУЙТЕ як продовження пропозиції Джонса, ви також можете створити метод розширення, змінивши наведений вище код на

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

У випадку, коли ви не можете змінити геттер, це було б моїм власним уподобанням, оскільки воно чіткіше виражає намір, даючи операції ім'я (OrEmptyIfNull)

Згаданий вище метод розширення може зробити неможливим виявлення певних оптимізацій. Зокрема, ті, які пов'язані з IList, використовуючи метод перевантаження, це можна усунути

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

Чесно кажучи, раджу: просто всмоктуй nullтест. nullТест тільки а , brfalseабо brfalse.s; все інше буде включати в себе набагато більше роботи (тести, завдання, додаткові виклики методів, немає необхідності GetEnumerator(), MoveNext(), Dispose()ітератора, і т.д.).

ifТест простий, очевидно, і ефективний.

"якщо" перед ітерацією все в порядку, мало хто з тієї "симпатичної" семантики може зробити ваш код менш читабельним.

у будь-якому випадку, якщо відступ турбує вас, ви можете змінити, якщо потрібно перевірити:

if(file.Headers == null)  
   return;

і ви потрапите до циклу foreach лише тоді, коли у властивості headers є справжнє значення.

ще один варіант, про який я можу подумати, - це використання оператора нульового злиття всередині вашого циклу foreach і для повного уникнення перевірки нуля. зразок:

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

(замініть колекцію вашим справжнім об’єктом / типом)

Використання Null-умовного оператора та ForEach (), який працює швидше, ніж стандартний цикл foreach.
Ви повинні передати колекцію до списку.

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

Я використовую хороший маленький метод розширення для цих сценаріїв:

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

Враховуючи те, що заголовки є списком типів, ви можете зробити наступне:

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

У деяких випадках я б віддав перевагу трохи іншому, загальному варіанту, припускаючи, що, як правило, конструктори колекції за замовчуванням повертають порожні екземпляри.

Краще було б назвати цей метод NewIfDefault. Це може бути корисно не тільки для колекцій, тому обмеження типу IEnumerable<T>може бути надлишковим.

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