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

所以您可以创建一个类似的函数并使用它