PyQt4应用数据库登录异常求助：任意凭证均可通过验证

嘿，我太懂你现在的糟心感了——注册功能顺顺当当，用户数据也老老实实存进MySQL了，结果登录的时候不管输啥阿猫阿狗的凭证都能直接过，这明显是登录验证的逻辑环节出了漏洞。咱们先把核心问题拆解开，一步步给你捋明白怎么修。

首先，你贴出来的数据库初始化代码是没问题的，问题肯定出在登录验证的核心逻辑里。我先给你梳理几个最容易踩的坑，再给你正确的代码示例对照着改：

1. 先排查登录逻辑的核心错误 #

最常见的问题无非这几个：

• 根本没正确执行查询，或者没检查查询结果是否真的匹配
• 不管查询有没有找到对应记录，代码都直接跳转到“登录成功”的分支
• 数据库查询出错了，但没做异常处理，导致默认“放行”了

2. 错误案例vs正确写法 #

假设你的登录代码是类似下面这种错误版本（很多人刚开始都会犯这个错）：

def login(self):
    # 获取输入的用户名和密码
    username = self.user_line.text()
    password = self.pwd_line.text()
    # 直接拼SQL（还容易被注入！），而且没检查结果
    query = QtSql.QSqlQuery()
    query.exec_("SELECT * FROM users WHERE username='%s' AND password='%s'" % (username, password))
    # 不管查没查到，直接登录成功
    self.accept()

这种写法当然会让任何凭证都能过！正确的写法应该严格检查查询结果，而且要用参数化查询（还能防SQL注入）：

from PyQt4.QtGui import QMessageBox

def login(self):
    # 先获取并清理输入（去掉前后空格）
    username = self.user_line.text().strip()
    password = self.pwd_line.text().strip()
    
    # 先校验输入是否为空
    if not username or not password:
        QMessageBox.warning(self, "提示", "用户名和密码不能为空哦！")
        return
    
    # 先确认数据库连接正常
    if not self.database1.isOpen():
        if not self.database1.open():
            QMessageBox.critical(self, "错误", "连不上数据库：%s" % self.database1.lastError().text())
            return
    
    # 使用参数化查询，安全又准确
    query = QtSql.QSqlQuery()
    query.prepare("SELECT id FROM users WHERE username = :username AND password = :password")
    query.bindValue(":username", username)
    query.bindValue(":password", password)
    
    # 执行查询并检查结果
    if query.exec_():
        # query.next()会移动到第一条结果，如果有结果说明匹配成功
        if query.next():
            QMessageBox.information(self, "成功", "登录啦！")
            self.accept()
        else:
            QMessageBox.warning(self, "错误", "用户名或密码不对哦！")
    else:
        # 查询出错了，提示错误信息
        QMessageBox.critical(self, "数据库错误", "查询失败：%s" % query.lastError().text())

3. 额外要注意的细节 #

• 密码存储的一致性：如果注册时你对密码做了哈希加密（比如MD5、SHA256），那登录时必须对用户输入的密码做同样的加密处理再和数据库里的哈希值对比，不能直接用明文比！
• 别忽略异常：数据库连接失败、查询语法错误这些情况一定要处理，不然代码可能会“静默”跳过验证环节，导致误判。

内容的提问来源于stack exchange，提问作者Elsie 44