なぜresharperは '単一の' throw文を持つcatch節は冗長です 'と言っていますか？

Question

私は例外をスローすることは、UIにバブルアップするか、または例外をログに記録し、そのことをユーザーに通知するようにするのがよい方法であると考えました。

なぜresharperは重複していると言いますか？

try 
{ 
    File.Open("FileNotFound.txt", FileMode.Open); 
} 
catch 
{ 
    throw; 
} 

Answer 1

ため

try { 
    File.Open("FileNotFound.txt", FileMode.Open); 
} catch { 
    throw; 
} 

は

File.Open("FileNotFound.txt", FileMode.Open); 

よりも違いはありませんUIに渡します。

上記のcatchの節では、ロギング、トランザクションのロールバック、追加情報の追加のためのラップなど、何もせずに例外をキャッチして再スローするだけです。

しかし、

try { 
    File.Open("FileNotFound.txt", FileMode.Open); 
} catch(Exception ex) { 
    GetLogger().LogException(ex); 
    throw; 
} 

は任意の冗長性を含んでいないだろうとReSharperのは、文句を言うべきではありません。同様に、

try { 
    File.Open("FileNotFound.txt", FileMode.Open); 
} catch(Exception ex) { 
    throw new MyApplicationException(
     "I'm sorry, but your preferences file could not be found.", ex); 
} 

は冗長ではありません。

Answer 2

上記のステートメントは、そこになかった場合と同じ動作をするためです。書き込みと同じ：File.Open(string, FileMode)への呼び出しは、その方法を見つけるでしょう、まったく同じ例外をサンプリングのいずれかで、その後、失敗した場合

File.Open("FileNotFound.txt", FileMode.Open); 

Answer 3

tryのコードがすでに例外をスローしているためです。

例外を再スローするだけでなく、catchブロックで何か他のことを行う場合は、例外をキャッチして再スローする必要があります。

Answer 4

catchブロックで処理を実行しておらず、例外を再度スローしただけです。

そこにtry ... catchブロックがあるという点はないので警告します。

また、「スローex」はスタックトレースを保存せず、スローします。

Answer 5

これは冗長なためです。

Answer 6

それはしばらく注目に値します...

try 
{ 
    DoSomething(); 
} 
catch 
{ 
    throw; 
} 

は...

try 
{ 
    DoSomething(); 
} 
catch (Exception ex) 
{ 
    // Generally a very bad idea! 
    throw ex; 
} 

この2番目のコードスニペットは、私が継承されたコードベースを通じてはびこっていた...次はないが、reduntantですいくつかのプロジェクトが前にあり、元の例外のスタックトレースを隠すという厄介な効果があります。このようにキャッチした例外をスローすると、スタックトレースの先頭がthrowレベルにあり、DoSomethingという言及なしに、またはネストされたメソッド呼び出しが実際に例外を引き起こしたことを意味します。

これを行う幸運のデバッグコード！