どちらがエラーをスローする傾向がありますか？

Question

私はアプリケーションをレビューしていますが、どちらがエラーを起こしやすいのでしょうか？

Public Function ToBool(ByVal vsValue As String) As Boolean 

     If IsNothing(vsValue) OrElse vsValue = "" Then 
      Return False 
     End If 
     If UCase(vsValue) = "TRUE" Or vsValue = "1" Or UCase(vsValue) = "Y" Or UCase(vsValue) = "YES" Or UCase(vsValue) = "T" Then 
      Return True 
     Else 
      Return False 
     End If 
    End Function 

又は

Public Function ToBool(ByVal vsValue As String) As Boolean 

      If IsNothing(vsValue) OrElse vsValue = "" Then 
       Return False 

      ElseIf UCase(vsValue) = "TRUE" Or vsValue = "1" Or UCase(vsValue) = "Y" Or UCase(vsValue) = "YES" Or UCase(vsValue) = "T" Then 
       Return True 
      Else 
       Return False 
      End If 
     End Function 

Answer 1

2つのコード例は、意味的に同一であるように、より多くのエラーが発生しやすいものであるように違いはありません。コードが最初のIfブロックの本体に入ると、どちらの場合も関数を終了します。実行するコードがでない場合ブロックの先頭にあるIfブロックの本体も同じです。

Answer 2

最初のものは、vsValueが何もない場合、 でなければなりません。そうでなければ、それが何かであり、2番目のuCase値の基準がそこになければ、最後のelseに行きます。特定の状態が不足している場合はfalseを返します。

最後にプログラムを2つ追加する必要はありません。

Answer 3

エラーが発生しやすいとは思っていません。 2つのifsの代わりにelseを使用しているので、2番目の方が少し良いかもしれません...

Answer 4

関数による観測：数値に変換されたブール値Trueを渡すと、評価されないため失敗します。次のように-1ない1.

When Visual Basic converts numeric data type values to Boolean, 0 becomes False and all other values become True

私が最初に偽側面を評価するこれに非常に類似した何かを書いた：

Public Shared Function ConvertBool(ByVal Value As String) As Boolean 
    If Value Is Nothing Then Return False 
    Select Case Value.ToUpper 
     Case "", "0", "N", "FALSE", "F" 'etc 
      Return False 
     Case Else 
      Return True 
    End Select 
End Function 

Answer 5

最終結果は、いずれの場合も同じことになるだろう。コードを少し読みやすくし、他のマッチを含むように展開するのが簡単な次のリファクタリング手法をお勧めしますか？

Public Function ToBool(ByVal vsValue As String) As Boolean 
    If Not String.IsNullOrEmpty(vsValue) Then 
    Dim value As Boolean = False 
    vsValue = vsValue.ToUpper() 
    value = value OrElse vsValue = "TRUE" 
    value = value OrElse vsValue = "1" 
    value = value OrElse vsValue = "Y" 
    value = value OrElse vsValue = "YES" 
    value = value OrElse vsValue = "T" 
    Return value 
    End If 
    Return False 
End If 

これは別の可能性です。

Public Function ToBool(ByVal vsValue As String) As Boolean 
    If Not String.IsNullOrEmpty(vsValue) Then 
    vsValue = vsValue.ToUpper() 
    Select Case vsValue 
     Case "TRUE": Return True 
     Case "1": Return True 
     Case "Y": Return True 
     Case "YES": Return True 
     Case "T": Return True 
    End Select 
    End If 
    Return False 
End If 

上記の例では、コンパイラはシーケンシャルルックアップを出力します。 case文の数が一定の閾値に達すると、Select構造全体がDictionary検索like what C# doesに変換される可能性があります。これが呼び出されると予想される関数であれば、両方の方法を試してどちらが速いのかを調べることができます。