ArgumentException o ArgumentNullException para los parámetros de cadena?

Question

Lejano como las mejores prácticas están preocupados, que es mejor:

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

    // do other stuff 
} 

o

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 
} 

La segunda versión parece más preciso, pero también más complicado que el primero. Usualmente voy con el n. ° 1, pero pensé que verificaría si hay un argumento para el n. ° 2.

Answer 1

Diría que la segunda manera es más precisa, sí, es más engorrosa, pero siempre puede envolverla en un método para evitar tener que hacerlo todo el tiempo. Incluso podría ser un método de extensión:

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); 
    } 
} 

Otra forma que es potencialmente útil es uno que devuelve la cadena original si todo está bien. Se podría escribir algo como esto:

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

Otra opción a considerar es el uso de Code Contracts como una alternativa a lanzar sus propias excepciones.

Answer 2

Sugeriría utilizar el primero. Si su método no espera una cadena nula o vacía, realmente no importa si se pasó nulo o vacío: importante informar y error, y eso es lo que hace la primera variante.

Answer 3

Otra posibilidad es ArgumentOutOfRange excepción:

La excepción que se produce cuando el valor de un argumento está fuera del rango permisible de valores tal como se define por el método invocado.