if（items！= null）がforeach（T item in items）の前に不必要ですか？

私はしばしば次のようなコードに出くわします：

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

基本的に、if条件は、がnullでないforeach場合にのみブロックが実行されることを保証しitemsます。if条件が本当に必要かどうか、または場合foreachを処理しますかitems == null。

つまり、簡単に書けますか

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

itemsnull かどうかを心配せずに？ある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 =>
{
    // ...
});

ここでの本当の要点は、最初はシーケンスがnullになることはほとんどないはずです。すべてのプログラムでそれを不変にするだけで、シーケンスがある場合はnullになりません。常に初期化されて、空のシーケンスまたはその他の正規のシーケンスになります。

シーケンスがnullになることがない場合、明らかにそれをチェックする必要はありません。

実際にはその@Connectに機能リクエストがあります：http ://connect.microsoft.com/VisualStudio/feedback/details/93497/foreach-should-check-for-null

そして、応答は非常に論理的です：

ほとんどのforeachループは、null以外のコレクションを反復する目的で記述されていると思います。nullを反復処理しようとすると、例外が発生するため、コードを修正できます。

あなたはいつもnullリストでそれをテストすることができます...しかし、これは私がmsdnウェブサイトで見つけたものです

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

式の値がnullの場合、System.NullReferenceExceptionがスローされます。

余計なものではありません。実行時に項目はIEnumerableにキャストされ、そのGetEnumeratorメソッドが呼び出されます。それは失敗するアイテムの逆参照を引き起こします

拡張メソッドでnullチェックをカプセル化し、ラムダを使用できます。

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使用するコレクションクラスに実装されたインタフェースを反復処理を実行します。一般的な同等のインターフェースはこちらです。

C＃言語のforeachステートメント（Visual Basicではそれぞれに対して）は、列挙子の複雑さを隠します。したがって、列挙子を直接操作するのではなく、foreachを使用することをお勧めします。

コレクションがnullの場合、foreachはNullReferenceExceptionをスローするため、テストが必要です。実際に試すのはとても簡単です。

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

2番目はNullReferenceExceptionメッセージをスローしますObject reference not set to an instance of an object.

前述のように、ここであなたはそれがnullでないか確認する必要があります。

nullと評価される式は使用しないでください。

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

基本的にはVlad Bezdenのソリューションですが、?? nullではない配列を常に生成する式。これにより、foreachブラケット内でこのチェックが行われるのではなく、foreachが存続します。