¿Debo tirar ArgumentNullException si una cadena está en blanco?

Question

Estoy trabajando en un método que hace algo dado un parámetro de cadena. Un valor válido para el parámetro de cadena es cualquier cosa que no sea nulo o cadena. Vacío. Entonces mi código se ve así.

 
private void SomeMethod(string someArgument) 
{ 
    if(string.IsNullOrEmpty(someArgument)) 
     throw new ArgumentNullException("someArgument"); 

    // do some work 
} 

Nada demasiado emocionante. Mi pregunta es, ¿está bien tirar una ArgumentNullException incluso si la cadena es igual a string.Empty? Porque técnicamente no es nulo. Si crees que no debería lanzar ArgumentNullException, ¿qué excepción debería arrojarse?

Answer 1

ArgumentException se debe lanzar para el caso String.Empty. Esto indicaría un problema que no sea nulo. Para evitar un NullReferenceException compruebo primero por nulo, luego recorte y verifico la caja vacía para evitar que pase espacio en blanco.

private void SomeMethod(string someArgument) 
{ 
    if(someArgument == null) 
     throw new ArgumentNullException("someArgument"); 

    if (someArgument.Trim() == String.Empty) 
     throw new ArgumentException("Input cannot be empty", "someArgument"); 

    // do some work 
} 

A partir de .NET 4.0 se puede utilizar el método String.IsNullOrWhiteSpace para llevar a cabo estos controles de una sola vez. Al hacerlo, renuncia a la posibilidad de especificar un tipo de excepción granular, por lo que optaría por el ArgumentException y actualizar el mensaje en consecuencia.

Answer 2

Debe arrojar un ArgumentException si una cadena vacía no es una entrada aceptada para su método. Puede ser muy confuso para los clientes si lanza un ArgumentNullException mientras que no proporcionaron un argumento null.

Es simplemente otro caso de uso. También puede tener métodos que no acepten valores de entrada nulos pero que acepten cadenas vacías. Es importante ser coherente en toda su aplicación.

Answer 3

ArgumentNullException a veces se utiliza en .NET Framework para el caso String.IsNullOrEmpty - un ejemplo es System.Windows.Forms.Clipboard.SetText.

Así que creo que es razonable hacer lo mismo en su código, a menos que haya un valor real para distinguir los dos casos.

Tenga en cuenta que esta y otras excepciones derivadas de ArgumentException generalmente indican un error de programación y, por lo tanto, deben proporcionar la información necesaria para ayudar al desarrollador a diagnosticar el problema. Personalmente, creo que es poco probable que un desarrollador lo encuentre confuso si usa ArgumentNullException para un argumento de cadena vacío, especialmente si documenta este comportamiento como en el siguiente ejemplo.

/// <summary> 
/// ... description of method ... 
/// </summary> 
/// <param name="someArgument">... description ...</param> 
/// <exception cref="ArgumentNullException">someArgument is a null reference or Empty.</exception> 
public void SomeMethod(string someArgument) 
{ 
    ... 
} 

Answer 4

Tomando todas las cosas en cuenta que se han dicho (Joe/Ahmad Mageed), me gustaría crear una excepción para ese caso entonces.

class ArgumentNullOrEmptyException : ArgumentNullException 

Answer 5

Esto depende de las circunstancias realmente.

La pregunta se reduce a, ¿es realmente un error? Con eso quiero decir, ¿usted siempre espera un valor? Si lo hace, entonces probablemente su mejor apuesta aquí es la creación de su propio Exception, quizá de este modo:

class StringEmptyOrNullException : Exception 
{ 
} 

donde también podrán añadir sus propios constructores y ha añadido información, etc.

Si embargo, no es una "excepcional" en su programa, si probablemente sería una mejor idea devolver nulo el método y manejarlo desde allí.Solo recuerda, Exception's son para condiciones excepcionales.

Espero que esto ayude,

Kyle

Answer 6

¿Por qué no utilizar este código?

private void SomeMethod(string someArgument) 
{ 
//chek only NULL 
if(ReferenceEquals(someArgument,null)) 
    throw new ArgumentNullException("someArgument"); 

// and after trim and check 
if (someArgument.Trim() == String.Empty) 
    throw new ArgumentException("Input cannot be empty", "someArgument"); 

// do some work 
}