问题在于您使用上下文管理器的方式。调用doquery
只是创建了一个上下文管理器对象——你需要在一个with
语句中使用它,它会根据需要调用它的__enter__
和__exit__
方法。例如,尝试以下操作:
from contextlib import contextmanager
@contextmanager
def enter_exit(text):
print('entering')
yield text
print('exiting')
print(enter_exit('attempt 1'))
with enter_exit('attempt 2') as t:
print(t)
我得到的输出是:
<contextlib._GeneratorContextManager object at 0xcf3e90>
entering
attempt 2
exiting
您可能需要重新阅读有关with 语句和contextlib的文档。
您的代码的另一个问题是,如果c.execute
或conn.commit
引发异常,c.close
将不会被调用 - 我不知道这是否真的有必要,但大概这就是您想要使用上下文管理器而不是函数的原因。地方。以下更改应该可以解决这两个问题:
import sqlite3
from contextlib import contextmanager
@contextmanager
def doquery(conn, q, params=()):
c = conn.cursor()
try:
c.execute(q, params)
conn.commit()
yield c
finally:
c.close()
with sqlite3.connect(':memory:') as db:
with doquery(db,'''create table stocks
(date text, trans text, symbol text,
qty real, price real)'''):
pass
with doquery(db,"""insert into stocks
values ('2006-01-05','BUY','RHAT',100,35.14)"""):
pass
with doquery(db, 'select * from stocks') as r:
for row in r:
print(row)
但是,我认为这不是最干净的方法。据我所知,没有理由创建三个单独cursor
的对象 - 您可以为每个查询使用相同的对象。我认为调用conn.commit
实际上也不是必需的 - 使用数据库连接作为上下文管理器将自动提交事务,或者如果引发异常则回滚它们(参见sqlite3 模块文档)。
编辑:这是一个更干净的版本,它仍然有效。我真的不知道关闭光标实际上做了什么——它可能没有必要(Cursor.close
甚至似乎没有记录)。
import sqlite3
from contextlib import closing
with sqlite3.connect(':memory:') as db:
with closing(db.cursor()) as c:
c.execute('''create table stocks
(date text, trans text, symbol text,
qty real, price real)''')
c.execute("""insert into stocks
values ('2006-01-05','BUY','RHAT',100,35.14)""")
c.execute('select * from stocks')
for row in c:
print(row)