C'è un buon modo per gestire le eccezioni in Python?

Question

Ho un sacco di codice che è simile a questo:

   try: 
        auth = page.ItemAttributes.Author 
       except: 
         try: 
          auth = page.ItemAttributes.Creator 
         except: 
           auth = None 

C'è un modo migliore per scrivere questa logica? Questo rende il mio codice davvero doloroso da leggere. Ho pensato try..finally avrebbe funzionato, ma ho assunto sbagliato

Answer 1

È possibile utilizzare hasattr per evitare il try/tranne blocchi:

auth = None 
for attrname in ['Author', 'Creator']: 
    if hasattr(page.ItemAttributes, attrname): 
     auth = getattr(page.ItemAttributes, attrname) 
     break 

Un modo alternativo di scrivere quanto sopra è quello di utilizzare la clausola di else un Python for ciclo:

for attrname in ['Author', 'Creator']: 
    if hasattr(page.ItemAttributes, attrname): 
     auth = getattr(page.ItemAttributes, attrname) 
     break 
else: 
    auth = None 

Answer 2

Questo rende il mio codice davvero doloroso per leggere

Qualsiasi cosa tu faccia, non prendere i caratteri jolly. except: è il modo per dire pazzesco: Hey, all exceptions are equal, I want every single error in my try block to end up here, I don't care if I catch an AttributeError or a WorldGotFuckedUpException. Nel tuo caso, except AttributeError è molto, molto meglio E più facile da leggere.

Questa è solo una nota a margine. La risposta di Mark mostra il modo migliore per farlo, IMHO.

Answer 3

risposta di @ Mark Byers è più flessibile, ma se si voleva un one-liner

auth = getattr(page.ItemAttributes, 'Author', None) or getattr(page.ItemAttributes, 'Creator', None)