パラメータを検証する正しい順序は何ですか？参考のため

Question

は、私はthis questionがprarametersがテストされるかのために考慮されることに注意したいと思います。両方の方法は、方法の内部の仕組みに同じ保護を達成する

public void DoSomething1(Dictionary<string, string> param1, Object param2) 
{ 
    if (null == param1) 
     throw new ArgumentNullException("param1"); 
    if(!param1.Keys.Contains("somekey")) 
     throw new ArgumentException("param1 must contain key \"somekey\""); 
    if(null == param2) 
     throw new ArgumentNullException("param2"); 

    // now we do some stuff 
} 

public void DoSomething2(Dictionary<string, string> param1, Object param2) 
{ 
    if (null == param1) 
     throw new ArgumentNullException("param1"); 
    if (null == param2) 
     throw new ArgumentNullException("param2"); 
    if(!param1.Keys.Contains("somekey")) 
     throw new ArgumentException("param1 must contain key \"somekey\""); 

    // now we do some stuff 
} 

：

は、一例として次の二つのスニペットを取ることができます。

私の質問は...（この例のように）メソッドに渡されたパラメータ、なぜをテストするための正しいアプローチ/オーダー

は何ですか？

バックアップへの参照あなたの答えを引用してください。

Answer 1

には "正しい順序" はありません。

それは完全に実装固有です。言い換えれば、パラメータがヌルであることを報告することが重要ですか、それともパラメータの詳細な検査を行うことが重要ですか？

私はこれをやっていたなら、私は、「パフォーマンス」のアプローチを取るだろう。言い換えれば、まず、必要なすべてのパラメータがnullでないことをテストし、各パラメータを開いて私が望むものを得られるようにします。ヌル参照チェックは、辞書にキーが含まれているかどうかをテストするよりも速く（わずかに）です。もちろん

この方法は、何百回も二呼ばれるように運命、それがnullパラメータを渡される可能性を有していたならば、私もそれを心配したいだけです。限られた影響しか持たないメソッドについては、呼び出し元が最初に知る必要があったものに固執します。