Node.js中MySQL查询结果处理及Session赋值后Undefined问题求助

看起来你遇到的问题主要来自三个方面：SQL查询的逻辑错误、缺少错误处理，还有异步操作的时序问题。咱们一步步来解决：

1. 修复SQL查询的逻辑优先级问题 #

你的SQL语句里，OR和AND的优先级是不一样的——AND的优先级高于OR，所以原语句实际执行的逻辑是：

SELECT * FROM users WHERE username = ? OR (email = ? AND password = ?)

这显然不是你想要的，你应该是想先匹配username=?或者email=?，再验证密码。所以需要给OR的部分加上括号，修正后的SQL应该是：

SELECT * FROM users WHERE (username = ? OR email = ?) AND password = ?

2. 必须处理数据库查询的错误 #

你的代码里完全没处理err参数，如果数据库查询出错（比如连接失败、语法错误），results就会是undefined，自然没法赋值Session。一定要加上错误判断：

connection.query('SELECT * FROM users WHERE (username = ? OR email = ?) AND password = ?', [login, login, password], function (err, results) {
  // 先处理错误
  if (err) {
    console.error('数据库查询出错:', err);
    return; // 出错就终止后续逻辑
  }
  
  if (results.length > 0) {
    req.session.username = results[0].username;
    req.session.email = results[0].email;
    req.session.premium = results[0].premium;
    req.session.created_at = results[0].created_at;
    // 注意：这里绝对不要把密码存在Session里！太不安全了！
    console.log('Session赋值成功:', req.session);
  } else {
    console.log("用户不存在或密码错误！");
  }
});

3. 注意异步操作的时序问题 #

connection.query是异步函数，如果你在这个回调函数外面直接访问req.session，那时候回调还没执行，Session肯定是undefined。比如下面这种写法就会有问题：

// 错误示例：异步操作外直接访问Session
connection.query(..., function() { ... });
console.log(req.session.username); // 这里会是undefined，因为回调还没跑完

所有依赖Session的逻辑，都必须放在回调函数里面，或者用Promise/async-await改写成同步风格的代码。

额外提醒：不要把密码存在Session里 #

把用户的密码存储在Session里是非常危险的行为！Session数据通常存在服务器内存或者Cookie里（如果用Cookie存储Session），很容易泄露。你只需要存储用户ID、用户名、邮箱这些非敏感信息就够了，密码永远不要在数据库之外的地方存储明文。

内容的提问来源于stack exchange，提问作者Timothy Myagi