Generar excepción vs. retorno ¿Ninguno en funciones?

¿Cuál es una mejor práctica en una función definida por el usuario en Python: raiseuna excepción o return None? Por ejemplo, tengo una función que busca el archivo más reciente en una carpeta.

def latestpdf(folder):
    # list the files and sort them
    try:
        latest = files[-1]
    except IndexError:
        # Folder is empty.
        return None  # One possibility
        raise FileNotFoundError()  # Alternative
    else:
        return somefunc(latest)  # In my case, somefunc parses the filename

Otra opción es dejar la excepción y manejarla en el código de la persona que llama, pero creo que es más claro lidiar con FileNotFoundErrorun IndexError. ¿O es una mala forma volver a generar una excepción con un nombre diferente?

Realmente es una cuestión de semántica. ¿Qué foo = latestpdf(d) significa ?

¿Es perfectamente razonable que no haya un archivo más reciente? Entonces seguro, devuelve None.

¿Espera encontrar siempre un archivo más reciente? Plantee una excepción. Y sí, volver a plantear una excepción más apropiada está bien.

Si esta es solo una función general que se supone que se aplica a cualquier directorio, haría lo primero y devolvería Ninguno. Si, por ejemplo, el directorio está destinado a ser un directorio de datos específico que contiene el conjunto de archivos conocido de una aplicación, plantearía una excepción.

Haría un par de sugerencias antes de responder su pregunta, ya que puede responderla por usted.

• Siempre asigne un nombre descriptivo a sus funciones. latestpdfsignifica muy poco para cualquiera, pero al revisar su función se latestpdf()obtiene el último pdf. Sugeriría que lo nombre getLatestPdfFromFolder(folder).

Tan pronto como hice esto, quedó claro qué debería devolver. Si no hay un pdf, plantee una excepción. Pero espera ahí más ...

• Mantenga las funciones claramente definidas. Dado que no es evidente lo que se supone que debe hacer somefuc y no es (aparentemente) obvio cómo se relaciona con obtener el último pdf, le sugiero que lo mueva. Esto hace que el código sea mucho más legible.

for folder in folders:
   try:
       latest = getLatestPdfFromFolder(folder)
       results = somefuc(latest)
   except IOError: pass

¡Espero que esto ayude!

Por lo general, prefiero manejar las excepciones internamente (es decir, intentar / excepto dentro de la función llamada, posiblemente devolviendo None) porque Python se escribe dinámicamente. En general, lo considero un juicio de una forma u otra, pero en un lenguaje escrito dinámicamente, hay pequeños factores que inclinan la balanza a favor de no pasar la excepción a la persona que llama:

1. Cualquiera que llame a su función no es notificado de las excepciones que se pueden lanzar. Se convierte en una especie de forma de arte saber qué tipo de excepción está buscando (y los bloques de excepción genérica deben evitarse).
2. if val is Nonees un poco más fácil que except ComplicatedCustomExceptionThatHadToBeImportedFromSomeNameSpace. En serio, odio tener que recordar escribir from django.core.exceptions import ObjectDoesNotExisten la parte superior de todos mis archivos de django solo para manejar un caso de uso realmente común. En un mundo de tipos estáticos, deje que el editor lo haga por usted.

Honestamente, sin embargo, siempre es una decisión de criterio, y la situación que está describiendo, donde la función llamada recibe un error que no puede ayudar, es una excelente razón para volver a generar una excepción que sea significativa. Tiene la idea exacta correcta, pero a menos que sea una excepción, proporcionará información más significativa en un seguimiento de pila que

AttributeError: 'NoneType' object has no attribute 'foo'

que, nueve de cada diez veces, es lo que verá la persona que llama si devuelve un Ninguno sin controlar, no se moleste.

(Todo esto me hace desear que las excepciones de Python tuvieran los causeatributos por defecto, como en Java, lo que le permite pasar excepciones a nuevas excepciones para que pueda volver a lanzar todo lo que quiera y nunca perder la fuente original del problema).

con Python 3.5 escribiendo :

La función de ejemplo al devolver None será:

def latestpdf(folder: str) -> Union[str, None]

y al plantear una excepción será:

def latestpdf(folder: str) -> str 

la opción 2 parece más legible y pitónica

(+ opción para agregar un comentario a la excepción como se indicó anteriormente).

En general, diría que se debe lanzar una excepción si ha ocurrido algo catastrófico del que no se puede recuperar (es decir, su función se ocupa de algún recurso de Internet al que no se puede conectar), y debe devolver Ninguno si su función realmente debería devolver algo pero no sería apropiado devolver nada (es decir, "Ninguno" si su función intenta hacer coincidir una subcadena en una cadena, por ejemplo).