문자열 매개 변수에 대한 ArgumentException 또는 ArgumentNullException?

Question

는 지금까지와 같은 모범 사례가 더있는 우려하고 있습니다 :

public void SomeMethod(string str) 
{ 
    if(string.IsNullOrEmpty(str)) 
    { 
     throw new ArgumentException("str cannot be null or empty."); 
    } 

    // do other stuff 
} 

또는

public void SomeMethod(string str) 
{ 
    if(str == null) 
    { 
     throw new ArgumentNullException("str"); 
    } 

    if(str == string.Empty) 
    { 
     throw new ArgumentException("str cannot be empty."); 
    } 

    // do other stuff 
} 

두 번째 버전은보다 정확한뿐만 아니라, 처음보다 더 성가신 것 같다. 나는 보통 # 1로 가고 있지만, # 2에 대한 논쟁이 있는지 확인해 볼 것입니다.

Answer 1

두 번째 방법은 실제로 더 정확하다고 말할 수 있습니다. 그렇습니다. 더 귀찮은 방법이지만 항상 모든 방법을 사용하지 않아도되는 방법으로 줄 바꿈 할 수 있습니다. 확장 메서드 일 수도 있습니다 :

str.ThrowIfNullOrEmpty("str"); 


public static void ThrowIfNullOrEmpty(this string value, string name) 
{ 
    if (value == null) 
    { 
     throw new ArgumentNullException(name); 
    } 
    if (value == "") 
    { 
     throw new ArgumentException("Argument must not be the empty string.", 
            name); 
    } 
} 

잠재적으로 유용한 또 다른 형식은 모든 것이 괜찮 으면 원본 문자열을 반환하는 것입니다.

public Person(string name) 
{ 
    this.name = name.CheckNotEmpty(); 
} 

자신의 예외를 던지는에 대한 대안으로 Code Contracts을 사용하고 고려해야 할 또 다른 옵션 : 당신이 뭔가를 작성할 수 있습니다.

Answer 2

첫 번째 것을 사용하는 것이 좋습니다. 귀하의 방법이 null 또는 빈 문자열을 기대하지 않는다면, null 또는 empty가 전달되었는지는 중요하지 않습니다.보고 및 오류에 중요하며, 첫 번째 변종이하는 일입니다.

Answer 3

다른 가능성 ArgumentOutOfRange 예외 :

호출 된 방법에 의해 정의되는 인자의 값이 허용 값의 범위 밖에있는 경우에 발생되는 예외.