매개 변수의 유효성을 검사하는 올바른 순서는 무엇입니까?

Question

참고로 에 대해 this question을 고려해야합니다.

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이 아니 었는지 테스트 한 다음 각 매개 변수를 열어 내가 원하는 것을 얻었는지 확인합니다. 사전이 키를 포함하는지 여부를 테스트하는 것보다 null 참조 검사를 수행하는 것이 더 빠릅니다 (약간).

물론이 방법이 초당 수백 번 호출되도록 예정되어 있고 null 매개 변수가 전달 될 가능성이있는 경우에만 걱정할 수 있습니다. 제한된 영향을 미치는 메소드의 경우 호출자가 먼저 알아야 할 필요가있는 부분을 고수 할 것입니다.