try-finallyを使ってメソッドが終了したら何かが実行されることを確認するのは良いことですか？

Question

私のメソッドは多くの点で返します。実行中にnewDataも多くの点で構築します。どこに戻っても、私は構築された結果を保存して保存する必要があります。 "return"を見逃すことはありません。try-finallyブロックでコードを囲んだので、今度はnewDataが格納されることを確信しています。

List<X> newData = new List<X>(); 
try 
{ 
    .... 
    update newData 
    ..... 
    return; 
    ..... 
    .... 
    update newData 
    .... 
    update newData 
    return; 
    ..... 
    return; 
} finally 
{ 
    // copy newData to data  
} 

しかし、私は例外を捕まえていません。このコードは例外を処理するためのものではありません。一般的に受け入れることができますか、別のより良いアプローチを提案することはできますか？

Answer 1

私は新しい方法にtryブロック内のコードをリファクタリングを示唆している：

data = CreateList(); 

... 

private List<X> CreateList() 
{ 
    List<X> list = new List<X>(); 
    // It's fine to return list from any point here... 
} 

Answer 2

何かが失敗した場合、最終的には（フェイルセーフメカニズムだと考える）のバックアップとして意図されているの使用を、あなたのメソッドの全体構造が間違っていて、リファクタリングすることができます。これらのリターンを得ることは、通常、別のアプローチを取ることを意味します（例えば、コメント内の誰かとしてスイッチを使用するなど） 。