python try/finally para control de flujo

Question

Estoy seguro de que este concepto ha surgido antes pero no puedo encontrar una respuesta buena y simple. ¿Está utilizando try/finally una mala forma de manejar funciones con retornos múltiples? Por ejemplo, tengo

 

try: 
    if x: 
     return update(1) 
    else: 
     return update(2) 
finally: 
    notifyUpdated() 
 

Esto sólo parece mejor que el almacenamiento de actualización() comandos en una variable temporal y regresar eso.

Answer 1

yo no lo recomendaría. Primero porque se llamará notifyUpdated() incluso si el código en cualquiera de las ramas arroja una excepción. Lo que se necesita algo como esto para conseguir realmente el comportamiento previsto:

try: 
    if x: 
     return update(1) 
    else: 
     return update(2) 
except: 
    raise 
else: 
    notifyUpdated() 

En segundo lugar, porque try bloques generalmente indican que estás haciendo algún tipo de control de excepciones y no está, sólo estás utilizarlos para conveniencia. Entonces esta construcción confundirá a las personas.

Por ejemplo, no creo que ninguna de las dos primeras personas (al menos una de las cuales eliminó su respuesta) para responder a su pregunta se dio cuenta de lo que realmente estaba tratando de hacer. El código confuso es malo, no importa cuán conveniente e inteligente parezca.

Answer 2

de http://docs.python.org/library/contextlib.html:

from contextlib import closing 
import urllib 

with closing(urllib.urlopen('http://www.python.org')) as page: 
    for line in page: 
     print line 

para que pueda crear una función similar y usarlo

Answer 3

Creo que significa que desea usar try/finally como alternativa a esto:

if x: 
    result = update(1) 
else: 
    result = update(2) 
notifyUpdated() 
return result 

supongo que esto es una cuestión de estilo. Para mí, me gusta reservar try para el manejo excepcional condicional. No lo usaré como una declaración de control de flujo.

Answer 4

No utilizaría try/finally para flujo que no implique excepciones. Es demasiado complicado por su propio bien.

Esto es mejor:

if x: 
    ret = update(1) 
else: 
    ret = update(2) 
notifyUpdated() 
return ret 

Answer 5

creo que esto es buscar problemas. ¿Qué pasa después, cuando cambias tu código a lo siguiente?

try: 
    if x: 
     return update(1) 
    elif y: 
     return update(2) 
    else: 
     return noUpdateHere() 
finally: 
    notifyUpdated() # even if noUpdateHere()! 

A lo sumo, es un punto de tropiezo para la mayoría de los lectores del código (probablemente incluso en seis meses), porque está utilizando try/finally para un propósito que se diferencia de los patrones normales de uso. Y el tipeo que guarda es mínimo, de todos modos.

Answer 6

creo que un decorador es una mejor idea aquí

def notifyupdateddecorator(f): 
    def inner(*args, **kw): 
     retval = f(*args, **kw) 
     notifyUpdated() 
     return retval 
    return inner 

@notifyupdateddecorator 
def f(x): 
    if x: 
     return update(1) 
    else: 
     return update(2) 

@notifyupdateddecorator 
def g(x): 
    return update(1 if x else 2)