C#: ¿Debo arrojar una ArgumentException o una DirectoryNotFoundException?

Question

Tengo un método que toma una ruta de directorio como una cadena. Al comienzo del método, comprueba si existe esta ruta y, si no, debería lanzar una excepción. Estoy pensando que debería lanzar un DirectoryNotFoundException o algo así en lugar de un ArgumentException menos específico.

He leído la documentación de MSDN de ese DirectoryNotFoundException y se dice que

DirectoryNotFoundException utiliza el HRESULT COR_E_DIRECTORYNOTFOUND que tiene el valor 0x80070003.

No sé lo que eso significa exactamente, y se ve un poco de miedo ... debería todavía tirar esa excepción, o debería atenerse a un habitual ArgumentException? ¿O debería seguir el ArgumentException simplemente porque es un argumento del que me quejo? ¿O?

public void MakeFunOf(string path) 
{ 
    if(!Directory.Exists(path)) 
     throw new WhatException(); 
    TellJokeAbout(path); 
    PointAndLaughAt(path); 
} 

Answer 1

Si espera que el desarrollador compruebe la existencia del directorio antes de llamar a su método, use ArgumentException. Si desea que el desarrollador tenga la opción de manejar el directorio que falta, use la excepción DirectoryNotFound.

En otras palabras, "¿Es un error que el desarrollador me dijo que accediera a un directorio que no existía?"

Personalmente, utilizaría la excepción DirectoryNotFound.

Answer 2

Es sólo mi opinión (como no tengo nada concreto que una copia de seguridad), pero aquí están mis razones para lanzando DirectoryNotFoundException en lugar de ArgumentException:

• que debe desechar el tipo más específico/precisa de Excepción que puede permitir que el consumidor de su código comprenda la razón por la cual se lanzó la excepción.
• Dado que los métodos del Marco se lanzar una DirectoryNotFoundException al intentar y hacer algo con un directorio que no está presente, en lugar de un ArgumentException, siga el bejaviour del Marco

Answer 3

suena como usted debe ser tirar un DirectoryNotFoundException como ArgumentException se utiliza mejor si no se proporciona el parámetro especificado ... es decir, es nulo.

Otra opción es crear su propia excepción y un tiro que en lugar decir

[Serializable] 
public class InvalidConfigurationException: Exception 
{ 
    public InvalidConfigurationException() : base() 
    { 
    } 

    public InvalidConfigurationException(string message) 
     : base(message) 
    { 
    } 

    public InvalidConfigurationException(string message, Exception innerException) 
     : base(message, innerException) 
    { 
    } 

    protected InvalidConfigurationException(SerializationInfo info, StreamingContext context) 
     : base(info, context) 
    { 
    } 
} 

entonces se podría hacer:

public void MakeFunOf(string path) 
{  
    if(!Directory.Exists(path))   
     throw new InvalidConfigurationException('Directory entered was invalid or does not exist'); 
    TellJokeAbout(path);  
    PointAndLaughAt(path); 
} 

Answer 4

En referencia a la documentación de 'ArgumentException':

ArgumentException se lanza cuando se invoca un método y al menos uno de los argumentos pasados ​​no lo hace cumplir con la especificación del parámetro del método llamado. Todas las instancias de ArgumentException deben llevar un mensaje de error significativo que describa el argumento no válido, así como el rango esperado de valores para el argumento.

Al pie de la letra esto significa que la elección de la excepción depende de la especificación/documentación de su método.

Si el parámetro de ruta se documenta como algo así como 'ruta a un archivo/directorio existente', entonces estaría justificado en el lanzamiento de un 'ArgumentException' (o derivado) porque en el fondo, en virtud de la documentación que haya tomado la llamante responsable de garantizar que el archivo esté realmente allí.

Si el parámetro de ruta se documenta de manera más general como 'Ruta a un archivo a broma alrededor y se ríen de', entonces yo diría 'DirectoryNotFoundException' es más apropiado.

Answer 5

En mi opinión, debería verificar las correcciones del argumento y lanzar una ArgumentException luego, después de la comprobación, arrojar una excepción DirectoryNotFoundException. Es una gran diferencia si no se dio ningún argumento o solo se especificó una ruta incorrecta.

void CheckDir(string path) 
{ 
    if(String.IsNullOrEmpty(path)) 
    { 
    throw new ArgumentException("Path not specified."); 
    } 
    if(!Directory.Exists(path)) 
    { 
    throw new DirectoryNotFoundException(); 
    } 
}