この例外処理をきれいに書いていますか？

Question

例外処理のベストプラクティスにはいくつか質問があることがわかりましたが、私が見つけたサンプルについていくつか微調整を加えなければなりませんでした。私の目標は、 "ArgumentException"、 "FileNotFoundException"、 "ArrayIndexOutOfBoundsException"などの基本的な例外を取り除くことでした。

次のコードを階層内に配置することができます。したがって、Process（）はCSVData（）を呼び出すValidateData（）を呼び出します。私は基本的に同じ種類のものを実行しています。は私のカスタム関数を含む既知の関数をすべてラップしてから、そのメソッドの例外に入れて内部の例外として渡します。これはこの仕事をうまくやっていく方法ですか？それは私のために働いていますが、過去の私のプロジェクトが「catch（Exception e）...」と「e.Messageの解析...」でカバーされているので、非常にクリーンな方法で例外を処理する習慣を得たいと思います。 「

ここに私のコード例があります：

public class CSVData 
{ 
    DataTable data; 
    public CSVData(string file) 
    { 
     try 
     { 
      this.data = CSVReader.ToDataTable(file); //throws a few basic exceptions 
     } 
     catch (FileNotFoundException e) 
     { 
      throw new FailedLoadingCSVException(e, currentLine, file); 
     } 
     catch (IOException e) 
     { 
      throw new FailedLoadingCSVException(e, currentLine, file); 
     } 
     catch (ArgumentNullException e) 
     { 
      throw new FailedLoadingCSVException(e, currentLine, file); 
     } 
     catch (ArgumentException e) 
     { 
      throw new FailedLoadingCSVException(e, currentLine, file); 
     } 
     catch (OutOfMemoryException e) 
     { 
      throw new FailedLoadingCSVException(e, currentLine, file); 
     } 
    } 

    class FailedLoadingCSVException : Exception 
    { 
     public int failedAtLine; 

     public FailedLoadingCSVException(Exception e, int failedLine, string file) 
      :base ("The system failed at loading "+file+" at line "+failedLine, e) 
     { 
      this.failedAtLine = failedLine; 
     } 
    } 
} 

Answer 1

try 
{ 
} 
catch (Exception ex) 
{ 
    if (e is FileNotFoundException || 
     e is IOException || 
     // enumerate few 
     ) 
     throw new FailedLoadingCSVException(e, currentLine, file); 
    else 
     throw; // rethrow all other exceptions without touching them   
} 

Answer 2

各catchブロックの本体は同じです。タイプ固有のアクションを取っていないので、この場合は特定の例外タイプをキャッチする必要はありません。これは良いです：

try 
    { 
     this.data = CSVReader.ToDataTable(file); //throws a few basic exceptions 
    } 
    catch (Exception e) 
    { 
     throw new FailedLoadingCSVException(e, currentLine, file); 
    } 

Answer 3

try 
{ 
    this.data = CSVReader.ToDataTable(file); //throws a few basic exceptions 
} 
catch (Exception e) 
{ 
    Console.WriteLine(e.Message);   
} 
//unless you know what the exact Exception of the CSVReader is you can always just use Base Exception Class..