try＆catchで戻って、最後に戻る？

Question

これらのいずれかが危険ですか？ 1つ良いですか？それとも、あなたが決めるためにダーツを投げ捨てるものの1つですか？

私は今、私がどのように機能するかを最終的に理解することがこれをしたい：

try { 
    stuff that changes something... 
} 
catch (System.Exception ex) { 
    something.worked = false; 
    something.err = ex.Message; 
} 
finally { 
    stuff.close(); 
    return something; 
} 

をしかし、私が見てきた：

try { 
    stuff that changes something... 
    return something; 
} 
catch (System.Exception ex) { 
    something.worked = false; 
    something.err = ex.Message; 
    return something; 
} 
finally { 
    stuff.close(); 
} 

Answer 1

あなたがすることができませんreturnfinallyから。あなたは、コンパイラエラーが発生します。

Control cannot leave the body of a finally clause

---

ターゲットクラスがIDisposableを実装する場合、私は次のだろう。その意志場合

using (stuff s = new stuff()) 
{ 
    return stuff; 
} 

または

using (stuff s = new stuff()) 
{ 
    try 
    { 
     // do stuff 
     return stuff; 
    } 
    catch (Exception ex) 
    { 
     // do logging or another stuff 
     return something; 
    } 
} 

はあなたのためにDispose()を呼び出します。必要/可能であること。

Answer 2

個人的に私はどちらを行うだろうし、あなたは彼らが失敗した場合に設定/開かれていなかったかもしれないとして、オブジェクトの/処分を閉鎖する必要があることを確認し、finally声明の中でも

try { 
    stuff that changes something... 
} 
catch (System.Exception ex) { 
    something.worked = false; 
    something.err = ex.Message; 
} 
finally { 
    stuff.close();  
} 
return something; 

を使用します。

はまた、第二のアプローチにはリスクはありませんここIs it bad practice to return from within a try catch finally block?

Answer 3

参照してください。しかし、例外の場合には異なる値を返すことができます。