【踩坑记录】数据库连接未关闭连接，释放资源的坑

最新推荐文章于 2023-05-09 17:39:58 发布

原创最新推荐文章于 2023-05-09 17:39:58 发布

· 1.1k 阅读

1 ·

版权

文章标签：

#sql #数据库

Python 专栏收录该内容

21 篇文章

订阅专栏

博客讨论了在执行数据库查询时，不正确的异常处理可能导致的资源泄露问题。原始代码将关闭数据库连接和游标的操作放在try块内，如果查询异常，连接和游标可能不会被正确关闭。改进后的版本在finally块中处理关闭操作，确保资源始终会被释放。作者提醒开发者，即使使用ORM框架，也应该注意基础的错误处理和资源管理。

摘要生成于 C知道，由 DeepSeek-R1 满血版支持，前往体验 >

今天不聊需求，在代码review时，看到这么一段代码。大家来看一下，这段数据库代码可能引发什么问题？

def execute(self, sql):
    """执行SQL语句方法"""
    try:
        conn = self.client.connection()
        cursor = conn.cursor()
        cursor.execute(sql)
        result = cur.fetchall()
        cur.close()
        conn.close()
        return result
    except Exception as e:
        log.error(e)
        raise e

把关闭数据库连接写在try里面，要是在查询的时候异常了，连接和游标能释放么？

是否该连接会长期占用系统资源直到被超时回收？

在限定连接池大小的条件下，如果这种查询SQL异常过多，超过了连接池限定大小，是否会导致其他正常查询无法执行？

相信这是大部分开发都容易踩的坑，为什么这么说呢？现在有很多完善的框架，帮我们做好了ORM，大部分开发长期依赖框架，业务写的很优秀但却把基础丢了。

下面是个人认为没什么问题的版本，欢迎指教。

try:
    conn = self.client.connection()
    cur = conn.cursor()
    cur.execute(sql)
    result = cur.fetchall()
except Exception as e:
    log.error(e)
    raise e
else:
    return result
finally:
    if cur:
        cur.close()
    if conn:
        conn.close()