Stile Python preferito (o qualsiasi lingua, in realtà): se si desidera utilizzare diversamente quando restituisce?

Question

domanda molto semplice:

In particolare in Python (dal Python ha in realtà linee guida dello stile "fortemente raccomandato" specificati in PEP 8, ma in realtà questo vale per qualsiasi lingua), devono una funzione con una clausola if che restituisce sempre avere l'alternativa codice in una clausola else o no? In altre parole, func_style_one() e func_style_two() nel seguente pezzo di codice sono (ovviamente) esattamente 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() 

Ovviamente, lo stile migliore e più leggibile dipende dalla situazione, e opinioni varia notevolmente sul quale è meglio, ma sto chiedendo quale sia specificamente preferito dalla core community Python. (Ad esempio, che viene utilizzato più spesso nella libreria standard, a parità di altre condizioni?)

Answer 1

Come regola generale, è sempre consigliabile evitare l'aggiunta di complessità non necessaria al codice, qualunque sia la lingua. Spesso è anche una buona idea provare a suddividere il tuo codice in sottosezioni semanticamente banali.

Data questa euristica, non esiste una risposta definitiva. Si riduce davvero a ciò che stai cercando di ottenere.

Lo dimostrerò con esempi.

Se abbiamo una funzione che verifica la presenza di diverse condizioni di errore prima di procedere, che potrebbe dare un senso a scrivere senza else:

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

Questo è meglio, come spesso si finisce per controllare diversi errori in modo simile a una sequenza:

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

Tuttavia, se la funzione invece rami a due rami uguali, potrebbe semanticamente più senso a voi il resto:

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

Questo perché spesso si finisce per aggiungere diverse altre opzioni con elif:

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

Per riassumere: pensate la semantica di codice e scegliere la sintassi di conseguenza.