python try/finally 用于流控制

Question

我确信这个概念之前已经提出过，但我找不到一个好的、简单的答案。使用 try/finally 是处理具有多个返回的函数的坏方法吗？例如我有

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

这似乎比将 update() 命令存储在临时变量中并返回它要好。

Answer 1

我不会推荐它。首先是因为notifyUpdated() 将被调用，即使任一分支中的代码抛出异常。您需要这样的东西才能真正获得预期的行为：

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

其次，因为try 块通常表明您正在执行某种异常处理，而您没有，您只是为了方便而使用它们。所以这个结构会让人迷惑。

例如，我认为回答您问题的前两个人（至少其中一个人删除了他们的答案）中的任何一个人都没有意识到您真正想要做什么。混乱的代码是不好的，不管它看起来多么方便和聪明。

Answer 2

对于不涉及异常的流程，我不会使用 try/finally。它本身就太棘手了。

这样更好：

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

Answer 3

我认为你的意思是你想使用 try/finally 作为替代：

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

我想这是风格问题。对我来说，我喜欢保留try 来处理异常条件。我不会将它用作流控制语句。

Answer 4

我认为这是自找麻烦。当您将代码更改为以下内容时会发生什么？

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

对于大多数阅读您的代码的读者来说，充其量是一个绊脚石（可能在六个月内甚至是您），因为它使用try/finally 的目的与正常使用模式不同。无论如何，它节省的打字量很少。

Answer 5

我认为装饰器在这里是一个更好的主意

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)

Answer 6

来自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

这样你就可以创建一个类似的函数并使用它