C＃の例外処理のベストプラクティス

Question

呼び出しメソッドと呼び出しメソッドで重複して例外を処理することに関する質問があります。次の例を考えてみてください。

protected void method1(string action) 
{ 
    if (string.IsNullOrWhiteSpace(action)) 
    { 
    throw new ArgumentException("action cannot be null or empty or white space."); 
    } 
    //do something 
} 

public void method2(string name, string action) 
{ 
    if (string.IsNullOrWhiteSpace(name)) 
    { 
    throw new ArgumentException("name cannot be null or empty or whitespace."); 
    } 
    method1(action); 
    //do something 
} 

上記の例では、method2でnullのアクションもチェックする必要がありますか？メソッド1で処理されるので、重複したコードが生成される可能性があるため、処理する必要はありません。

例外処理のベストプラクティスについて学ぶ上で、どんな提案も大きな助けになるでしょう。ありがとう

Answer 1

いつもチェックしています。私はコードがどのように動作するのかを知っています今度はですが、今後どのようになるか分かりません。コードが変更されます。私の手や他人の手によって。チェックしないコードは、爆発するのを待つ時爆弾です。

Answer 2

あなたmethod1がmethod2のprivate/internal実装であるだけにしてあなたは、両方のメソッドに引数ガードを配置する必要はありませんmethod2によって呼び出されている場合。