Python调用SQL查询商品时报错及查询结果遍历优化咨询

1. 修复SQL执行报错（OperationalError: incomplete input） #

报错根源是SQL语句不完整，且参数传递方式不符合规范：

• 原SQL语句"SELECT * FROM item_in_stock WHERE item_name = "缺少参数占位符，数据库无法识别匹配变量的位置
• cursor.execute()的第二个参数要求为元组类型，不能直接传单个字符串

修改后的代码：

#----------Storefront----------#
while login(username, password, list_of_users) == True:
    search_bar = input("Enter what item you would like to look up >>> ")
    # 使用?作为SQLite参数化查询的占位符
    sql = "SELECT * FROM item_in_stock WHERE item_name = ?"
    # 参数以元组形式传递，单个参数也要加逗号保证是元组类型
    cursor.execute(sql, (search_bar,))
    # 替换原循环的简洁写法
    for row in cursor:
        print(row)

2. 理解并替换遍历查询结果的循环 #

原循环的作用 #

for row in iter(cursor.fetchone, None):
    print(row)

iter(cursor.fetchone, None)会生成一个迭代器：

• 每次迭代自动调用cursor.fetchone()获取一行数据
• 当cursor.fetchone()返回None（无更多数据）时，迭代终止
• 本质是手动实现逐行遍历查询结果的逻辑

更直观的替代方法 #

有两种更简洁易懂的写法：

方法一：直接遍历cursor对象

cursor本身是可迭代对象，每次迭代返回一行数据：

for row in cursor:
    print(row)

方法二：先获取所有结果再遍历

用fetchall()一次性把所有结果存入列表，再遍历：

rows = cursor.fetchall()
for row in rows:
    print(row)

这种方法适合需要多次复用结果的场景，但结果集过大时会占用更多内存。

额外优化建议 #

• 登录逻辑优化：无需一次性拉取所有用户数据到内存，直接用SQL查询验证登录更高效：

  def login(email: str, pwd: str, cursor) -> bool:
      cursor.execute("SELECT 1 FROM users WHERE email = ? AND password = ?", (email, pwd))
      return cursor.fetchone() is not None
  

• 异常处理完善：原代码except块捕获所有异常，建议明确捕获sqlite3.Error，避免隐藏其他非数据库类错误。

内容的提问来源于stack exchange，提问作者Jeremy Panthier