Estilo preferido de Python (o cualquier idioma, realmente): ¿Debería usarse de otra forma si se devuelve?

Question

pregunta muy simple:

Específicamente en Python (ya que Python tiene en realidad "muy recomendable" guías de estilo especificado en PEP 8, pero en realidad esto se aplica a cualquier idioma), en caso de una función con una cláusula de if que vuelve siempre tienen la alternativa código en una cláusula else o no? En otras palabras, func_style_one() y func_style_two() en el siguiente fragmento de código son (obviamente) exactamente equivalente:

def func_style_one(): 
    if some_conditional_function(): 
     do_something() 
     return something() 
    else: 
     do_something_else() 
     return something_else() 

def func_style_two(): 
    if some_conditional_function(): 
     do_something() 
     return something() 
    do_something_else() 
    return something_else() 

Obviamente, la mejor y más legible estilo depende de la situación, y opiniones variará en gran medida de que es mejor, pero estoy preguntando cuál es específicamente preferido por la comunidad principal de Python. (Por ejemplo, ¿cuál se usa con más frecuencia en la biblioteca estándar, siendo todo lo demás igual?)

Answer 1

Como regla general, siempre debe evitar agregar complejidad innecesaria a su código, sea cual sea el idioma. También a menudo es una buena idea tratar de dividir su código en subsecciones semánticamente malas.

Dadas estas heurísticas, no hay una respuesta definitiva. Realmente se reduce a lo que estás tratando de lograr.

Lo demostraré con ejemplos.

Si tenemos una función que comprueba si hay varias condiciones de error antes de continuar, podría tener sentido para escribir sin else:

def do_stuff(): 
    if error1(): 
     return cleanup_and_fail() 
    return ok() 

Esto es mejor, ya que a menudo terminan haciendo el registro de varios errores de forma similar en una secuencia:

def do_stuff(): 
    if error1(): 
     return cleanup_and_fail() 
    if error2(): 
     return do_different_cleanup_and_fail() 
    return ok() 

Sin embargo, si su función en lugar de dos ramas ramas iguales, es semánticamente podría tener más sentido que otra cosa:

def do_stuff(): 
    if option1(): 
     return do_option1() 
    else: 
     return do_option2() 

Esto se debe a que a menudo terminan añadiendo varias otras opciones con elif:

def do_stuff(): 
    if option1(): 
     return do_option1() 
    elif: 
     return do_option2() 
    else: 
     return do_option3() 

Para resumir: piensa en la semántica de su código y elige la sintaxis en consecuencia.