今天不聊需求,在代码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()