Является ли if (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, вы можете использовать новый условный оператор null вместе с 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 вместо непосредственного управления перечислителем.

Тест необходим, потому что, если коллекция имеет значение NULL, 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.