Come designare codice irraggiungibile python

Question

Qual è il modo divinatorio per designare codice irraggiungibile in Python come in:

gender = readFromDB(...) # either 'm' or 'f' 
if gender == 'm': 
    greeting = 'Mr.' 
elif gender == 'f': 
    greeting = 'Ms.' 
else: 
    # What should this line say? 

Answer 1

raise ValueError('invalid gender %r' % gender) 

Answer 2

Questo dipende da quanto sicuro siete del genere essere o 'm' o 'f'.

Se si è assolutamente certi, utilizzare if...else anziché if...elif...else. Semplicemente rende più facile per tutti.

Se ci sono possibilità di dati non corretti, tuttavia, dovresti probabilmente sollevare un'eccezione per rendere più facile il test e la risoluzione dei problemi. potresti utilizzare un saluto di genere neutro in questo caso, ma per qualcosa di più grande, i valori speciali rendono difficile trovare gli errori più difficili da trovare.

Answer 3

Si potrebbe sollevare un'eccezione:

raise ValueError("Unexpected gender; expected 'm' or 'f', got %s" % gender) 

o utilizzare un'asserzione falsa se si prevede che il database per restituire solo 'm' o 'f':

assert False, "Unexpected gender; expected 'm' or 'f', got %s" % gender 

Answer 4

Io in realtà penso che ci sia un posto per questo.

class SeriousDesignError(Exception): 
    pass 

Così si può fare questo

if number % 2 == 0: 
    result = "Even" 
elif number % 2 == 1: 
    result = "Odd" 
else: 
    raise SeriousDesignError() 

Penso che questo sia il messaggio di errore più significativo. Questo genere di cose può consistere solo gli errori di progettazione (o cattiva manutenzione, che è la stessa cosa.)

Answer 5

volte faccio:

if gender == 'm': 
    greeting = 'Mr.' 
else: 
    assert gender == 'f' 
    greeting = 'Ms.' 

penso che questo fa un buon lavoro di raccontare un lettore del codice che ci sono solo (in questo caso) due possibilità, e quali sono. Sebbene sia possibile sollevare un errore più descrittivo di AssertionError.

Answer 6

Dipende esattamente quello che vuoi l'errore da segnalare, ma vorrei usare un dizionario, in questo caso:

greetings={'m':'Mr.', 'f':'Ms.'} 
gender = readFromDB(...) # either 'm' or 'f' 
greeting=greetings[gender] 

Se genere non è né m né f, tale da far salire un KeyError contenente il valore inaspettato:

greetings={'m':'Mr.', 'f':'Ms.'} 

>>> greetings['W'] 

Traceback (most recent call last): 
    File "<pyshell#4>", line 1, in <module> 
    greetings['W'] 
KeyError: 'W' 

Se volete maggiori dettagli nel messaggio, si può prendere & controrilancio è:

try: 
    greeting = greetings[gender] 
except KeyError,e: 
    raise ValueError('Unrecognized gender %s'%gender) 

Answer 7

Fino ad ora, ho di solito usato una variazione sul risposta di John Fouhy - ma questo non è esattamente corretto, come sottolinea Ethan:

assert gender in ('m', 'f') 
if gender == 'm': 
    greeting = 'Mr.' 
else: 
    greeting = 'Ms.' 

Il problema principale con l'utilizzo di un'asserzione è che se qualcuno corre il vostro codice con i flag -O o -OO, i richiami vengono ottimizzati. Come Ethan indica di seguito, ciò significa che ora non hai alcun controllo dei dati. Le asserzioni sono un aiuto allo sviluppo e non dovrebbero essere utilizzate per la logica di produzione.Vado a prendere l'abitudine di usare un controllo funzionale() invece - questo permette per la sintassi chiamata pulito come un'asserzione:

def check(condition, msg=None): 
    if not condition: 
     raise ValueError(msg or '') 

check(gender in ('m', 'f')) 
if gender == 'm': 
    greeting = 'Mr.' 
else: 
    greeting = 'Ms.' 

Tornando alla domanda iniziale, mi sostengo che l'utilizzo di un assert() o controllare() prima del caso/logica il resto è più facile da leggere, più sicuro e più esplicito:

• mette alla prova la qualità dei dati prima di iniziare ad agire su di esso - questo potrebbe essere importante se ci sono operatori diversi da '==' nella catena if/else
• separa il test di asserzione dalla logica di branching, invece di interlacciarli - th rende la lettura e il refactoring più semplici