リファクタリングガード欄

Question

あなたのクラスで爆発を管理するために人々が（もしあれば）どのようなアプローチを取っていますか？例：

public void SomeMethod<T>(string var1, IEnumerable<T> items, int count) 
{ 
    if (string.IsNullOrEmpty(var1)) 
    { 
     throw new ArgumentNullException("var1"); 
    } 

    if (items == null) 
    { 
     throw new ArgumentNullException("items"); 
    } 

    if (count < 1) 
    { 
     throw new ArgumentOutOfRangeException("count"); 
    } 

    ... etc .... 
} 

私が現在取り組んでいるプロジェクトでは、パブリックメソッドに類似のガード句セットを持つ多くのクラスがあります。

私は.NET 4.0コード契約を認識していますが、現時点ではこのオプションの選択肢ではありません。 、あなたはコードの契約をシミュレートすることができ、いくつかの方法があるがそれよりも

public void SomeMethod<T>(string var1, IEnumerable<T> items, int count) 
{ 
    if (string.IsNullOrEmpty(var1)) 
     throw new ArgumentNullException("var1"); 

    if (items == null) 
     throw new ArgumentNullException("items"); 

    if (count < 1) 
     throw new ArgumentOutOfRangeException("count"); 

    ... etc .... 
} 

その他：

Answer 1

私は静的Guardクラスを使用して見てきた多くのプロジェクト。

public static class Guard { 
    public static void ArgumentIsNotNull(object value, string argument) { 
     if (value == null) 
      throw new ArgumentNullException(argument); 
    } 
} 

私の意見では、コードをもっとクリーンにしています。

Guard.ArgumentIsNotNull(arg1, "arg1"); 

Answer 2

あなたはコードコントラクトルートを下るしたくない場合は、それを単純化するための1つの方法は、中括弧を削除することです、あなたの異議は、.NET 4.0がまだプライムタイムではないということである場合：

http://geekswithblogs.net/Podwysocki/archive/2008/01/22/118770.aspx

Answer 3

リファクタリングはIntroduce a Null Objectと考えられます。

Answer 4

一方、ここでこれについての優れた記事があります：http://haacked.com/archive/2013/01/05/mitigate-the-billion-dollar-mistake-with-aspects.aspx/

私は定型的なコードを減らすために、私はFodys能力について興奮しているようNullGuard.Fodyを使用するために検討する

Answer 5

（完全に除去しない）の数を減らすための一つのアプローチガード句は、その存在の原因を理解することです。多くの場合、引数の型に対して有効な値に対しては保護しますが、受け入れられるメソッドに対しては有効ではありません。つまり、メソッドは引数型で定義されたドメインのサブセットで定義されます。

このカテゴリのケースに対する解決策は、サブタイプ（より限定的なインターフェースなど）を定義し、そのタイプを引数として受け入れることです。この記事では、具体的な例を見つけることができます。Why do We Need Guard Clauses?

もちろん、この手法はすべての場合に当てはまるわけではありません。すべての参照型は、少なくともnull参照を許可します。その結果、ほとんどのメソッドはドメインの一部で定義され、nullへの保護条項が必要になります。

しかし、この手法は、望ましいよりも一般的な引数を受け取るメソッドの認識を高めるのに役立ちます。この穴は一般的にデザインを改善するのに役立ちます。