C＃でヌルチェックの必要性を取り除く方法はありますか？

Question

私はいくつかの異なる組み合わせを試してみましたが、これは、私は比較として仕事を得ることができる唯一の組み合わせだった=「」ヌルのために問題を与えるように見えた！

 if (options.English != null && options.English != "") 
     { 
      if (options.English.StartsWith("^")) 
      { 
       query = query.Where(w => w.English.StartsWith(options.English.Trim().Substring(1))); 
      } 
      else 
      { 
       query = query.Where(w => w.English.Contains(options.English.Trim())); 
      } 
     } 

が、私はこれを最適化する可能性がどのような方法があります1つのif文、または2つのif文であっても、外側と内側のif文ではなく、互いに続く2つのif文に簡単にすることができます。

Answer 1

場合は、ネストされたif Sを削除されてやろうとしているすべてのは、その後、あなたは確かにこれを取得することができます?:と一緒String.IsNullOrEmpty()を使用することによって、今Dictionary

var lookup = new Dictionary<bool, Func<Options, bool>>() { 
    { true, w => w.English.StartsWith(options.English.Trim().Substring(1)) }, 
    { false, w => w.English.Contains(options.English.Trim())} }; 

if (options.English != null && options.English != "") 
{ 
    query = query.Where(lookup[options.English.StartsWith("^")]); 
} 

にラムダを保存することができます... 1つの文にダウンし、それも恐ろしいではありません。

query = String.IsNullOrEmpty(options.English) ? 
    query : query.Where(lookup[options.English.StartsWith("^")]); 

しかし、あなたは2つだけのオプションを持っていることを考えると、DictionaryはレアではありませんLLY必要に応じ、わずか数地元の人々を使用します。

Func<Options, bool> startsWithCaret = w => w.English.StartsWith(options.English.Trim().Substring(1)); 
Func<Options, bool> doesNotStartWithCaret = w => w.English.Contains(options.English.Trim()); 
var lambda = options.English.StartsWith("^") ? startsWithCaret : doesNotStartWithCaret; 

あなたしている文は、今発見される

query = String.IsNullOrEmpty(options.English) ? query : query.Where(lambda); 

ないifです！:-)

Answer 2

あなたはstring.IsNullOrEmpty()を探しています。ネストを削除するには

、あなたは（C＃6）を行うことができます

if (options.English?.StartsWith("^") == true) { 
    ... 
} else if (!string.IsNullOrEmpty(options.English) { 
    ... 
} 

Answer 3

これは、入れ子にすることなく、より長い鎖の中にいない他で外ifを変換するために些細です。

if (A) { 
    if (B) { ... } 
} 

は、あなたの場合は

if (!A) 
{ /* do nothing */ } 
else if (B) { ... } 

となり、それは言語の規則に従って、if (B) ... elseが実際に他の最初の内側にネストされていますが、人間のプログラマが「ドン

if (string.IsNullOrEmpty(options.English)) 
    { /* do nothing */ } 
else if (options.English.StartsWith("^")) 
    { 
     query = query.Where(w => w.English.StartsWith(options.English.Trim().Substring(1))); 
    } 
else 
    { 
     query = query.Where(w => w.English.Contains(options.English.Trim())); 
    } 

注意ですそれをネスティングとして表現するか、チェーンというよりはむしろネスティングと考えることができます。あなたは、独自のメソッドに出てコードをリファクタリングした場合

Answer 4

そして、あなたはそれらを入れ子にすることなく、別のif文を持っているreturn文を活用することができます：あなたは、あなたが示されてきた方法にquery = AddOptions(query, options)を書き込むことができ

public static IEnumerable<Item> AddOption(IEnumerable<Item> query, Option options) 
{ 
    if (string.IsNullOrEmpty(options.English)) 
     return query; 
    if (options.English.StartsWith("^")) 
     return query.Where(w => w.English.StartsWith(options.English.Trim().Substring(1))); 
    return query = query.Where(w => w.English.Contains(options.English.Trim())); 
} 

。

Answer 5

最もパフォーマンスが、nullが

if (!options.English.IsNullOrEmpty()) 
    query = query.Where(w => 
     w.English.Contains(string.Concat("^", options.English) 
     .Replace("^^","^").Substring(1).Trim())); 

ちょうど常にあなたの文字列の前に^を入れられるだけのための場合とそう最短の方法ではありません、（多分）に取って代わると^^を作成^と入力して、いつでもサブストリング（1）を実行できます。

Answer 6

あなたのロジックは、このように単純化することができます。

 if (!string.IsNullOrEmpty(options.English)) 
     { 
      var s = options.English.Trim(); 
      query = query.Where(w => { return s.StartsWith("^") ? 
       w.English.StartsWith(s.Substring(1)) : w.English.Contains(s); }); 
     }