使用async与MongoDB的findOne查询过慢且无连接反馈的问题排查

核心问题排查 #

• 箭头函数导致this绑定错误：User.prototype.login用箭头函数定义时，this不会指向当前User实例，而是继承外层作用域的this，导致this.data.username为undefined。数据库执行findOne({name: undefined})时会遍历全表匹配，数据量大时就会出现查询耗时极久的情况。
• 未触发异步逻辑：login方法内部定义了run异步函数，但从未调用它，导致整个登录逻辑根本没执行，自然没有任何反馈。
• 密码明文处理：直接比较明文密码存在严重安全风险，一旦数据库泄露，用户密码会直接暴露。
• 缺少查询索引：name作为登录查询的关键字段，没有创建索引的话，MongoDB会执行全表扫描，查询效率极低。

优化后的代码 #

const usersCollection = require('../db').collection("users");

// 改用普通函数保证this指向User实例
User.prototype.login = async function() {
    try {
        // 检查用户名是否存在
        const attemptedUser = await usersCollection.findOne({ name: this.data.username });
        
        if (!attemptedUser) {
            throw new Error("用户未注册");
        }
        
        // 注意：生产环境必须用密码哈希比对（比如bcrypt.compare），以下仅为逻辑演示
        // 正确示例：const isPasswordValid = await bcrypt.compare(this.data.password, attemptedUser.password);
        const isPasswordValid = (attemptedUser.password === this.data.password);
        
        if (isPasswordValid) {
            console.log("登录成功");
            return true; // 返回登录状态给调用方
        } else {
            throw new Error("密码错误，访问被拒绝");
        }
    } catch (err) {
        console.error(err.message);
        throw err; // 抛出错误让调用方处理
    }
};

额外优化建议 #

• 创建数据库索引：在MongoDB中为users集合的name字段创建唯一索引，既避免重复用户名，又能大幅加速查询：

  // 在数据库初始化阶段执行一次即可
  usersCollection.createIndex({ name: 1 }, { unique: true });
  

• 密码哈希存储：使用bcrypt或argon2等库对用户密码进行哈希存储，登录时比对哈希值而非明文：

  // 用户注册时哈希密码
  const hashedPassword = await bcrypt.hash(userInputPassword, 10);
  // 登录时验证密码
  const isPasswordValid = await bcrypt.compare(this.data.password, attemptedUser.password);
  

• 添加查询超时：为数据库查询设置超时时间，避免无限制等待：

  const attemptedUser = await usersCollection.findOne(
      { name: this.data.username },
      { maxTimeMS: 5000 } // 设置5秒超时
  );
  

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