Qual è il modo pitone di inizializzazione della variabile condizionale?

Question

A causa delle regole di scoping di Python, tutte le variabili una volta inizializzate in un ambito sono disponibili successivamente. Poiché i condizionali non introducono un nuovo ambito, i costrutti in altri linguaggi (come l'inizializzazione di una variabile prima di tale condizione) non sono necessariamente necessari. Per esempio, potremmo avere:

def foo(optionalvar = None): 
    # some processing, resulting in... 
    message = get_message() 
    if optionalvar is not None: 
     # some other processing, resulting in... 
     message = get_other_message() 
    # ... rest of function that uses message 

o, potremmo avere invece:

def foo(optionalvar = None): 
    if optionalvar is None: 
     # processing, resulting in... 
     message = get_message() 
    else: 
     # other processing, resulting in... 
     message = get_other_message() 
    # ... rest of function that uses message 

Naturalmente, i get_message e get_other_message funzioni potrebbero essere molte righe di codice e sono sostanzialmente irrilevante (si può assumere che lo stato del programma dopo ogni percorso è lo stesso); l'obiettivo qui è rendere message pronto per l'uso oltre questa sezione della funzione.

Ho visto il secondo costrutto utilizzato più volte in altre questioni, come ad esempio:

• https://stackoverflow.com/a/6402327/18097
• https://stackoverflow.com/a/7382688/18097

Quale costrutto sarebbe più accettabile?

Answer 1

Python ha anche una molto utile se modello sintassi che è possibile utilizzare qui

message = get_other_message() if optional_var else get_message() 

Oppure, se si vuole confrontare rigorosamente Nessuno

message = get_other_message() if optional_var is not None else get_message() 

differenza con l'esempio 1) hai postato questo doesn chiamiamo get_message() inutilmente.

Answer 2

Penso che sia più sdolcinato non impostare una regola esplicita su questo, e invece basti pensare che funzioni piccole sono migliori (in parte perché è possibile tenere a mente proprio quando vengono introdotti nuovi nomi).

Suppongo però che se i test condizionali ottenere molto più complicato di un if/else si può correre il rischio di tutti loro in mancanza e in seguito utilizzando un nome definito, con conseguente possibilità di errore di runtime, a meno che non siete molto attenti. Questo potrebbe essere un argomento per il primo stile, quando è possibile.

Answer 3

La risposta dipende da se ci sono effetti collaterali di get_message() che si vogliono.

Nella maggior parte dei casi in modo chiaro la seconda vince, perché il codice che produce il risultato indesiderato non viene eseguito. Ma se hai bisogno degli effetti collaterali, dovresti scegliere la prima versione.

Answer 4

Potrebbe essere meglio (leggi: più sicuro) per inizializzare la variabile di fuori delle condizioni. Se è necessario definire altre condizioni o addirittura rimuoverne alcune, in seguito l'utente di message potrebbe ricevere un'eccezione variabile non inizializzata.

Answer 5

Al secondo approccio generale è migliore e più generico in quanto non comporta chiamando get_message incondizionatamente.Che può essere ok se tale funzione non è di risorse incentivo ma si consideri una funzione di ricerca

def search(engine): 
    results = get_from_google() 
    if engine == 'bing': 
     results = get_from_bing() 

, ovviamente, questo non è buono, non riesco a pensare a tale scenario male per secondo caso, in modo sostanzialmente un approccio che passa attraverso tutti opzioni e, infine, l'impostazione predefinita è meglio eg

def search(engine): 
    if engine == 'bing': 
     results = get_from_bing() 
    else: 
     results = get_from_google()