例外をスローするのは良い方法ですか？

Question

ユーザーが2つのaspxページでフォームを使用して顧客オブジェクトの値を入力する例を考えてみましょう。最初のアプローチでは、両方のaspxページは、IDが0より大きく、FirstNameがコンストラクタを呼び出す前に空でないことを検証する必要があります。 2番目のオプションでは、両方のページが検証機能を呼び出し、エラーメッセージをユーザーに表示できます。

上記の例に基づいて、私は第2の選択肢を好む。しかし、私がWeb上で研究しているときには、例外をすぐにスローし、オブジェクトに無効なデータを受けさせないようにすることが、よりオブジェクト指向であることが分かり続けます。以前の例外と同様に、このコンストラクタを呼び出すすべてのページは、入力が有効であることを検証する必要があります。私は論理を繰り返すのが好きではないので、私は第二の選択肢を好む。

ドメインドリブンデザインではどのような選択肢がありますか？ 1

public class Customer{ 

    public int ID { get; set; } 
    public string FirstName { get; set; } 

    public Customer(int ID, string FirstName){ 
     if (ID < 0) 
      throw new Exception("ID cannot be less than 0"); 
     if (string.IsNullOrEmpty(FirstName)) 
      throw new Exception("First Name cannot be empty"); 

     this.ID = ID; 
     this.FirstName = FirstName; 
    } 
} 

オプション2

public class Customer{ 

    public int ID { get; set; } 
    public string FirstName { get; set; } 

    public Customer(int ID, string FirstName){ 

     this.ID = ID; 
     this.FirstName = FirstName; 

    } 

    public List<string> Validate(){ 
     List<string> ErrorMessages = new List<string>(); 

     if (ID < 0) 
      ErrorMessages.Add("ID cannot be less than 0"); 
     if (string.IsNullOrEmpty(FirstName)) 
      ErrorMessages.Add("First Name cannot be empty"); 

     return ErrorMessages; 
    } 

} 

Answer 1

私はこの質問が他の場所で回答されたと確信しています。しかし、ここで読書のためのいくつかの他のリンクです：書籍「実践的なプログラマー」から

• Best practices for catching and re-throwing .NET exceptions
• When to throw an exception?

、例外の使用に関する大きな問題は何が例外である」であります？ "そのセクションで

、私は引用：

...例外はほとんどのプログラムの通常の流れの一部として使用することはできません。予期しないイベントに対して例外を予約する必要があります。

それはあなたのケースで例外を使用するかどうか議論の余地があるが、私は何を言わないだろう - おそらく一つのリクエストですべての可能な入力エラーをキャプチャし、修正するために、ユーザーのためのフォームに戻って反映させる必要があるため、それらの値。

ここで思い出したように、はいここでは例外を使用する必要があります。これはあなたのコードをで防衛する方法です。すでに有効な引数がCustomerクラスに渡されることを期待している場合、コードはクラスの無効な使用（たとえば、別のプログラマなど）からの保護のために例外をスローする必要があります。その場合、Customerクラスに達する前に、アプリケーションへのユーザーの入力を検証するための別の入力バリデーターを用意する必要があります。

Answer 2

オプション短い答えはNOです。

アプリケーションが不良データで実行を続行できない場合は、例外がスローされます。あなたの例では、ロジックはフロントエンドにエラーメッセージを表示し、オプション2はこの要件を達成するためのよりクリーンな方法です。

例外がスローされても（例外がキャッチされたとしても）、コストのかかる操作です。スレッドが続行される前に、例外が呼び出しスタック全体を走査しなければなりません。これにより、規模の問題が発生します。

Answer 3

適切な例外処理がある場合は、例外をスローすることをお勧めします。アプリケーションがクラッシュするまで、盲目的に例外をスローしないでください。

オプション1の例では、ArgumentExceptionがより適切です。

public Customer(int ID, string FirstName){ 
     if (ID < 0) 
      throw new ArgumentException("ID cannot be less than 0"); 
     if (string.IsNullOrEmpty(FirstName)) 
      throw new ArgumentException("First Name cannot be empty"); 

     this.ID = ID; 
     this.FirstName = FirstName; 
    } 

例：

try 
{ 
    Customer cust = new Customer(-1, string.Empty); 
} 
catch(ArgumentException aex) 
{ 
    DoSomething(aex); // Log to database, file, or alike 
    // Maybe display a message on the user 
} 
catch(Exception ex) 
{ 
    DoSomething(ex); // Log to database, file, or alike 
    // Do generic error process here 
} 

あなたのオプション2は、データの検証のためのより適切です。その動作は、DataAnnotationsを適用するときと同様に観察できます。例外はスローされませんが、エラーメッセージが返されます。