これら2つのステートメントは同じですか？

Question

次の2つのコードスニペットは同じことを達成していますか？

私の元のコード：私は上記のコードは、それを変更する任意の説得力のある理由読みする方がはるかに簡単だと思う

this.SafeFileNames = safeFileNames != null && 
        Convert.ToBoolean(safeFileNames.Value); 

：

if (safeFileNames != null) 
{ 
    this.SafeFileNames = Convert.ToBoolean(safeFileNames.Value); 
} 
else 
{ 
    this.SafeFileNames = false; 
} 

は良いアイデアは何ReSharperの考えでしたか？
これは速く実行されますか、そして最も重要なのは、コードはまったく同じことでしょうか？

また、：Convert.ToBoolean(safeFileNames.Value);セクションを見ると、確かにnull参照例外が発生する可能性がありますか？

this.SafeFileNames = bool 

ローカルsafeFileNamesは、ここでは、強く型付けされたカスタムオブジェクトであるクラスです：

public class Configuration 
    { 
     public string Name 
     { 
      get; 
      set; 
     } 
     public string Value 
     { 
      get; 
      set; 
     } 
    } 

Answer 1

あなたが質問したという事実は、前者が優先されることを私に示唆しています。つまり、あなたの質問は、最初のコードが理解しやすいと考えていることを意味し、2番目のコードが同等かどうかはわかりません。ソフトウェア設計の第一の目標は、複雑さを管理することです。それがあなたに今混乱している場合は、後であなたにやって来るかもしれません。

Answer 2

両方の文がまったく同じことを行います。どちらを使うのが好みの問題ですが、私はResharperのバージョンを好む。より簡潔で動きの少ない部品。コードの意図を見やすくする。

Answer 3

これらは同じです。 & &は短絡演算子であるため、safeFileNamesがnullの場合、式の後半は評価されません。

Answer 4

これらは同じです。 1ライナーでは、最初の条件が失敗した場合、2番目の条件は評価されません。だからあなたはヌル参照を取得しません。

どちらの場合もILは同じです。

私は2番目のバージョンが好きです。

Answer 5

resharperの提案を使用すると、コードの循環的複雑さが軽減されます。

読みやすいかどうかは個人的な意見ですが、私はresharperがあなたに与えた提案を好む。

それらが同一である、とあなたは可読性をしたい場合、私はまた、次のことを提案することができます：

if (safeFileNames != null) 
    this.SafeFileNames = Convert.ToBoolean(safeFileNames.Value); 
else 
    this.SafeFileNames = false; 

または

this.SafeFileNames = safeFileNames != null ? Convert.ToBoolean(safeFileNames.Value) : false 

Answer 6

うん、両方のこれらのステートメントは、あなたがいない、同じことを行います再福音をゴスペルとして取らなければならない、一人の人間が読めるコードと考えるのは、もう一つの混乱である。

あなたのやりたいことをもっと読みやすくするために、いくつかの方法があります。どのタイプの値がsafeFileNamesですか？それはnull可能なブールかもしれないようですね？もしあなたが単に書くことができれば、

this.SafeFileNames = safeFileNames.GetValueOrDefault(); 

Answer 7

論理的には同じです。パフォーマンスの差異は無視できる可能性があります。 2番目の形式は条件付きでなくなるため、2番目の形式は一部のプラットフォームでより効率的なバイナリコードに変換される可能性があります。コンディショナル（間違った投機実行）は、CPUを大量に使用する作業でCPUの命令パイプラインを台無しにする可能性があります。しかし、ILとJITterの両方は、これに大きな差をつけるために適切な品質のコードを発行する必要があります。

私はあなたの読みやすさに同意しますが、誰もがそれを共有しているとは限りません。

Answer 8

両方のコードでILがあります。私はあなたのコードを取って、ILを見てコンソールアプリケーションを作成しました。結果のILから見ることができるように、1つのメソッド（method2）は4バイト短くなりますが、両方のために実行されるILはほぼ同じであるため、パフォーマンスが上がるまでは...心配する必要はありません。どちらも同じパフォーマンスを持ちます。どちらが読みやすく、あなたの意図をよりよく示すかに焦点を当てます。

マイコード：

class Program 
{ 
    static void Main(string[] args) 
    { 


    } 
    public void method1() 
    { 
     bool? safeFileNames = null; 

     if (safeFileNames != null) 
     { 
      SafeFileNames = Convert.ToBoolean(safeFileNames.Value); 
     } 
     else 
     { 
      SafeFileNames = false; 
     } 
    } 
    public void method2() 
    { 
     bool? safeFileNames = null; 
     SafeFileNames = safeFileNames != null && Convert.ToBoolean(safeFileNames.Value); 
    } 
    public static bool SafeFileNames { get; set; } 
} 

IL方法1：

.method public hidebysig instance void method1() cil managed 
{ 
    // Code size  42 (0x2a) 
    .maxstack 1 
    .locals init ([0] valuetype [mscorlib]System.Nullable`1<bool> safeFileNames) 
    IL_0000: ldloca.s safeFileNames 
    IL_0002: initobj valuetype [mscorlib]System.Nullable`1<bool> 
    IL_0008: ldloca.s safeFileNames 
    IL_000a: call  instance bool valuetype [mscorlib]System.Nullable`1<bool>::get_HasValue() 
    IL_000f: brfalse.s IL_0023 
    IL_0011: ldloca.s safeFileNames 
    IL_0013: call  instance !0 valuetype [mscorlib]System.Nullable`1<bool>::get_Value() 
    IL_0018: call  bool [mscorlib]System.Convert::ToBoolean(bool) 
    IL_001d: call  void ConsoleApplication5.Program::set_SafeFileNames(bool) 
    IL_0022: ret 
    IL_0023: ldc.i4.0 
    IL_0024: call  void ConsoleApplication5.Program::set_SafeFileNames(bool) 
    IL_0029: ret 
} // end of method Program::method1 

IL方法2について：

.method public hidebysig instance void method2() cil managed 
{ 
    // Code size  38 (0x26) 
    .maxstack 1 
    .locals init ([0] valuetype [mscorlib]System.Nullable`1<bool> safeFileNames) 
    IL_0000: ldloca.s safeFileNames 
    IL_0002: initobj valuetype [mscorlib]System.Nullable`1<bool> 
    IL_0008: ldloca.s safeFileNames 
    IL_000a: call  instance bool valuetype [mscorlib]System.Nullable`1<bool>::get_HasValue() 
    IL_000f: brfalse.s IL_001f 
    IL_0011: ldloca.s safeFileNames 
    IL_0013: call  instance !0 valuetype [mscorlib]System.Nullable`1<bool>::get_Value() 
    IL_0018: call  bool [mscorlib]System.Convert::ToBoolean(bool) 
    IL_001d: br.s  IL_0020 
    IL_001f: ldc.i4.0 
    IL_0020: call  void ConsoleApplication5.Program::set_SafeFileNames(bool) 
    IL_0025: ret 
} // end of method Program::method2