這兩個陳述是否相同？

Question

以下2個代碼片段是否實現了相同的功能？

我的原代碼：

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

什麼ReSharper的想法是一個更好的主意：

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

我覺得上面的代碼更易於閱讀，任何令人信服的理由去改變它？
它會執行得更快嗎，而且最重要的是，代碼能夠完成同樣的事情嗎？

此外，如果你看看：Convert.ToBoolean(safeFileNames.Value);節，那麼肯定這可能會導致空引用異常？

this.SafeFileNames = bool 

本地safeFileNames是一個強類型的自定義對象，這裏是類：

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

Answer 1

你問這個問題的事實告訴我，前者是首選。也就是說，在我看來，你的問題意味着你相信第一個代碼更容易理解，而你不確定第二個代碼是否相同。軟件設計的主要目標是管理複雜性。如果您現在對此感到困惑，那麼稍後您可能會對此感到困惑，或者對於任何支持您的代碼的人來說，這可能都會對您造成困擾。

Answer 2

兩個語句做同樣的事情。哪個使用是一個偏好問題，但我更喜歡Resharper的版本。更簡潔，更少移動的部分。更容易看到代碼的意圖。

Answer 3

它們是一樣的。 & &是一個短路運算符，因此表達式的後半部分不會評估safeFileNames是否爲空。

Answer 4

它們是相同的。在單行中，如果第一個條件失敗，則第二個條件不被評估。所以你不會得到一個空引用。

我敢打賭，IL在兩種情況下都是一樣的。

我更喜歡第二個版本。

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？它看起來可能是一個可空的布爾？如果是這樣，你可以簡單地寫，

this.SafeFileNames = safeFileNames.GetValueOrDefault(); 

Answer 7

從邏輯上說，它們是相同的。任何性能差異都可能會被忽略。第二種形式可能會轉化爲更有效的二進制代碼，因爲第二種形式會取消條件式。條件（不正確的推測執行）會在CPU密集型工作中混淆CPU的指令流水線。然而，IL和JITter都需要發出足夠的質量代碼才能產生很大的差異。

我同意你的可讀性，但我不認爲每個人都會這樣認爲。

Answer 8

這是兩個代碼段的IL。我把你的代碼和創建一個控制檯應用程序來看看IL。正如你從結果IL中看到的，一個方法（方法2）縮短了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