您每次都从数据库查询中重建一个列表。
我会要求数据库不要返回被禁止的记录:
from psycopg2 import sql # safe SQL composing
# Add a NOT IN clause to filter out banned doorbots, generating a
# separate parameter placeholder per value
query = sql.SQL(query) + sql.SQL(' WHERE ding_id NOT IN ({})').format(
sql.SQL(', ').join([sql.Placeholder()] * len(BANNED_DOORBOTS)))
await cur.execute(query, BANNED_DOORBOTS)
我在这里使用psycopg.sql framework 进行合成,但您也可以使用字符串格式(使用'%s' 作为占位符)。
考虑将BANNED_DOORBOTS 集放在数据库的表中,以便您可以使用WHERE ding_id NOT IN (SELECT id from BANNED_DOORBOTS WHERE id IS NOT NULL) 子查询。这样您仍然可以获得更好的性能(数据库可以为此优化),并且您不必生成占位符。
接下来,使用列表推导来构建列表。这更快,因为它避免了重复的list.append 查找和方法调用。将您的列名定义为一个元组并将其与每一行一起压缩:
keys = ('ding_id', 'doorbot_id', 'created_at', 'address', 'latitude',
'longitude', 'ding_kind')
return [dict(zip(keys, row)) async for row in cur]
async for 列表解析语法需要 Python 3.6 或更高版本。
aiopg 驱动程序允许您配置一个替代游标工厂,即already produces dictionaries,它可能更快。然后,您根本不必使用任何列表推导:
from psycopg2.extras import RealDictCursor
# configure cursor to yield dictionaries rather than tuples
async with conn.cursor(cursor_factory=RealDictCursor) as cur:
await cur.execute(query, BANNED_DOORBOTS)
# directly return the cursor; have the caller do the async iteration
return cur
如果您不想让调用者负责循环,但有生成列表,请使用cursor.fetchall() method 生成该列表;每个元素都是一个字典:
# configure cursor to yield dictionaries rather than tuples
async with conn.cursor(cursor_factory=RealDictCursor) as cur:
await cur.execute(query, BANNED_DOORBOTS)
return await cur.fetchall()