2つのIListsを安全に組み合わせて、ヌルを考慮に入れて

Question

IListのうちの1つ（どちらか一方または両方）にnullを含めることができます。両方がnullの場合、結果はnullになります。

私はこれを持っていますが、私はもっとエレガントですが、読んでも簡単です。

private IList<Filter> CombineFilters(IList<Filter> first, IList<Filter> second) 
{ 
    IList<Filter> result = null; 

    if(first != null) 
     if (second != null) 
      result = first.Concat(second) as IList<Filter>; 
     else 
      result = first; 
    else if (second != null) 
     result = second; 

    return result; 
} 

Answer 1

私は二つのリスト（1 nullの可能性がある）を使用してCombineFiltersを呼び出し、そのリストを取り戻す場合、それは奇妙な行動の少しだと思うだろうではなく、です。その結果、次のような動作が発生します。

List<Filter> filters1 = new List<Filter>() { new Filter() }; 
List<Filter> filters2 = null; 

Console.WriteLine(filters1.Count); // 1 

var filters = CombineFilters(filters1, filters2); 
filters.Add(new Filter()); 

Console.WriteLine(filters.Count); // 2 
Console.WriteLine(filters1.Count); // also 2 

実際にこの動作が期待されるかどうかはわかりません。

最後に新しいリストがあることを常に確認することをお勧めします。

private IList<Filter> CombineFilters(IList<Filter> first, IList<Filter> second) 
{ 
    return (first ?? Array.Empty<Filter>()).Concat(second ?? Array.Empty<Filter>()).ToList(); 
} 

Answer 2

private IList<Filter> CombineFilters(IList<Filter> first, IList<Filter> second) 
{ 
    if (first == null && second == null) return null; 
    first = first ?? new List<Filter>(); 
    second = second ?? new List<Filter>(); 
    return first.Concat(second).ToList(); 
} 

Answer 3

一緒??オペレータと利用Enumerable.Concate：

if(first == null && second == null) 
    return null; 
return Enumerable.Concat(first ?? Enumerable.Empty<IFilter>(), 
         second ?? Enumerable.Empty<IFilter>()).ToList(); 

いずれかfirstまたはsecondは、連結のために代わりに使用される新しいリストnullしている場合。

---

私は場合には、両方のリストがnullその後、まだ初期化コレクションが返されるべきだと思います。その場合はif文だけを削除してください。また、多分代わりIListのIEnumerableを返す：代わりにIListのIEnumerableを返す

private IEnumerable<Filter> CombineFilters(IEnumerable<Filter> first, IEnumerable<Filter> second) 
{ 
    return Enumerable.Concat(first ?? Enumerable.Empty<IFilter>(), 
          second ?? Enumerable.Empty<IFilter>()); 
} 

はあなたのシナリオに依存します。詳細はこちらを読む：

• Should I always return IEnumerable<T> instead of IList<T>?
• Should I return an IEnumerable or IList?

Answer 4

あなたはこのような何かを行うことができます：これは、あなたが、これは本当に短い行うことができます

、それは完全にヌルコレクションを避けることをお勧めだと述べて

private IList<Filter> CombineFilters(IList<Filter> first, IList<Filter> second) 
{ 

    if (first == null && second == null) 
    { 
     return null; 
    } 

    return (first ?? Enumerable.Empty<Filter>()) 
      .Concat(second ?? Enumerable.Empty<Filter>()) 
      .ToList(); 
} 

。要素がない場合は、空のコレクションを返します。この方法では、常にNULLをチェックする必要はありません。これはあなたのケースでは可能ではないかもしれませんが、そうであれば、コードをきれいにするでしょう。それは、より一般的な抽象化であるとして

また、IList<T>いうしIEnumerable<T>を使用することを検討してください（ともIList<T>が壊れ、漏れやすい抽象化ですので、 - 実装は（例外を投げる）、彼らはサポートしていないメソッドのために！）。一方、これはプライベートメソッドであることがわかります。実際には常にList<Filter>を渡している場合は、パラメータをList<Filter>と宣言する方がよいでしょう。

EDIT：
私の答えは、私が以前見たことがなかったGilad Greenによって投稿されたoneと非常に似ています。