Python初学者：如何阻止'finally'执行？

Question

这是一个函数的代码：

# Connect to the DB
try:
    dbi = MySQLdb.connect(host='localhost', \
                          user='user', \
                          passwd='pass', \
                          db='dbname', \
                          port=3309)

    print "Connected to DB ..."

except MySQLdb.Error, e:
    apiErr = 2
    apiErrMsg = "Error %d: %s" % (e.args[0], e.args[1])
    return

    # To prevent try..finally bug in python2.4,
    # one has to nest the "try: except:" part. 
try:
    try:
        sql = dbi.cursor()
        sql.execute("""
        SELECT *
        FROM table
        WHERE idClient =  %s
        """, (key, ))

        access = sql.fetchall()

        # [some more code here]           

    except MySQLdb.Error, e:
        apiErr = 2
        apiErrMsg = "Error %d: %s" % (e.args[0], e.args[1])
        return

finally:
    sql.close()
    dbi.close()

我知道在 try .. except .. finally 结构中，finally 块总是会执行。在上面的代码中，如果第一个 try 块出现异常，我不希望第二个 try 块中的 finally 执行。我哪里做错了？

（注意：使用的是 Python 2.4）

补充说明：我不清楚 MySQLdb 在发生错误时是否会自动关闭连接。我在上面的代码中遇到的问题是，当建立连接时出现错误（代码的第一个 try 块），在 finally 块中调用 dbi.close() 会引发 "AttributeError: 'NoneType' object has no attribute 'close'" 的错误，指的是 dbi ...

解决方案： 这个方法达到了我想要的效果 -

# define at the start 
dbi = None
sql = None

在 finally 块中，

if sql is not None:
    sql.close()
if dbi is not None:
    dbi.close()

感谢那些回复我的人。我从你们身上学到了新东西。（下次我会尽量更清楚地表达我的问题 :）。

Answer 1

不要使用finally。如果你不想让某段代码总是被执行，那就应该找其他的控制流程结构来满足你的需求。

一种实现这种行为的方法是把你'finally'块里的语句移动到'尝试'块的底部。这样，当发生异常时，这些语句就不会被执行，而在其他语句都执行完后，它们才会被执行。

编辑：

经过进一步讨论，看来在你的情况下，你确实想使用'finally'。我建议在你尝试关闭连接之前，先检查一下连接是否已经关闭。

Answer 2

使用 else: 而不是 finally:。可以查看文档中的 异常处理 部分：

try ... except 语句有一个可选的 else 子句，如果有这个子句，它必须放在所有 except 子句之后。这个 else 子句很有用，因为它可以用来执行那些只有在 try 语句没有出现异常时才需要执行的代码。

for arg in sys.argv[1:]:
    try:
        f = open(arg, 'r')
    except IOError:
        print 'cannot open', arg
    else:
        print arg, 'has', len(f.readlines()), 'lines'
        f.close()

..基本上：

try:
    [code that might error]
except IOError:
    [This code is only ran when IOError is raised]
else:
    [This code is only ran when NO exception is raised]
finally:
    [This code is always run, both if an exception is raised or not]

Answer 3

我觉得在这种情况下，你确实需要使用finally，因为你需要关闭那些连接。

我不同意在同一个方法里使用两个try块的想法。

我认为设计上的问题在于在同一个方法里获取连接和执行查询。我建议把这两件事分开。一个服务类或者方法应该知道要做的工作。它应该获取连接，然后把连接传给另一个类去执行查询，最后在完成后关闭连接。这样，查询方法就可以抛出它遇到的任何异常，而把清理工作留给负责连接的类或方法去处理。