Preferowany styl w Pythonie (lub w jakimkolwiek innym języku): Powinien być użyty, gdy zwraca?

Question

Bardzo proste pytanie:

szczególności w Pythonie (od Pythona faktycznie ma „zaleca” wytyczne styl określony w PEP 8, ale tak naprawdę to ma zastosowanie do każdego języka), należy funkcję z klauzulą ​​if że zawsze zwraca mają alternatywę kod w klauzuli else czy nie? Innymi słowy, func_style_one() i func_style_two() w następujący kawałek kodu są (oczywiście) dokładnie równoważne:

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() 

Oczywiście, najlepszym i najbardziej czytelny styl zależy od sytuacji i opinie będą się znacznie różnić, w którym jest lepiej, ale pytam, który jest szczególnie preferowany przez rdzenną społeczność Pythona. (np. Które jest używane częściej w standardowej bibliotece, wszystkie inne rzeczy są równe?)

Answer 1

Jako ogólną zasadę należy zawsze unikać niepotrzebnego skomplikowania kodu, niezależnie od języka. Często dobrym pomysłem jest również próba podzielenia kodu na semantycznie podłe podsekcje.

Biorąc pod uwagę te heurystyki, nie ma jednoznacznej odpowiedzi. To naprawdę sprowadza się do tego, co próbujesz osiągnąć.

Pokażę to na przykładach.

Jeśli mamy funkcję, która sprawdza różne warunki błędach przed postępowaniem, może to sensu pisać je bez else:

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

To jest lepsze jak często kończy się sprawdzanie kilka błędów w podobny sposób w sekwencja:

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

jeśli jednak funkcja zamiast oddziałów na dwie równe oddziałów, może to więcej sensu semantycznie wam innego:

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

To dlatego, że często kończy się dodając kilka innych opcji z elif:

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

Podsumowując: myślę o semantyce kodzie i wybierz odpowiednio składnię.