Чи є (items! = Null) зайвим перед foreach (T item in items)?

Я часто стикаюся з таким кодом:

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

В основному, ifумова гарантує, що foreachблок буде виконуватися лише в тому випадку, якщо itemsвін не є нульовим. Мені цікаво, чи ifсправді потрібна умова, чи foreachвирішуватиметься випадок, якщо items == null.

Я маю на увазі, чи можу я просто писати

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

не турбуючись про те, чи itemsє нульовим чи ні? Чи ifумова зайва? Або це залежить від типу від itemsабо , може бути , на Tа?

Вам все одно потрібно перевірити, чи (items! = Null) в іншому випадку ви отримаєте NullReferenceException. Однак ви можете зробити щось подібне:

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

але ви можете перевірити його ефективність. Тому я все-таки вважаю за краще спочатку мати if (items! = Null).

На основі пропозиції Еріка Ліпперта я змінив код на:

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

Використовуючи C # 6, ви можете використовувати новий нульовий умовний оператор разом із List<T>.ForEach(Action<T>)(або власним IEnumerable<T>.ForEachметодом розширення).

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

Справжній винос тут повинен бути послідовністю, майже ніколи не повинен бути нульовим в першу чергу . Просто зробіть його інваріантом у всіх ваших програмах: якщо у вас є послідовність, вона ніколи не буде нульовою. Він завжди ініціалізується як порожня послідовність або якась інша справжня послідовність.

Якщо послідовність ніколи не є нульовою, очевидно, її не потрібно перевіряти.

Насправді на цьому @Connect є запит на функцію: http://connect.microsoft.com/VisualStudio/feedback/details/93497/foreach-should-check-for-null

І відповідь цілком логічна:

Я думаю, що більшість циклів foreach написані з наміром повторити ненульову колекцію. Якщо ви спробуєте ітерацію через null, вам слід отримати виняток, щоб ви могли виправити свій код.

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

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

Якщо вираз має значення null, викидається System.NullReferenceException.

Це не надмірно. Під час виконання елементи будуть передаватися в IEnumerable, і його метод GetEnumerator буде викликаний. Це призведе до зняття посилань на елементи, які не зможуть

Ви можете інкапсулювати нульову перевірку в методі розширення та використовувати лямбда:

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

Код стає:

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

Якщо може бути ще більш стислим, якщо ви хочете просто викликати метод, який бере елемент і повертає void:

items.ForEach(MethodThatTakesAnItem);

Вам це потрібно. Ви отримаєте виняток, коли отримаєте foreachдоступ до контейнера, щоб налаштувати ітерацію в іншому випадку.

Під обкладинками foreachвикористовується інтерфейс, реалізований у класі колекції для виконання ітерації. Загальний еквівалентний інтерфейс тут .

Заява foreach мови C # (для кожного в Visual Basic) приховує складність перелічувачів. Тому використовувати foreach рекомендується замість прямого маніпулювання перерахувачем.

Тест необхідний, тому що якщо колекція нульова, foreach видасть NullReferenceException. Насправді це досить просто спробувати.

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

другий кине а NullReferenceExceptionз повідомленнямObject reference not set to an instance of an object.

Як зазначалося тут, вам потрібно перевірити, чи не є він нульовим.

Не використовуйте вираз, який обчислює нуль.

У C # 6 ви можете написати sth так:

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

В основному це рішення Влада Бездена, але використання ?? вираз, щоб завжди генерувати масив, який не є нульовим і, отже, переживає foreach, а не має цю перевірку всередині скобки foreach.