Sqlite3，用户已经存在但不存在

Question

app.post('/register', function(req,res){
  console.log("register post got", req.body)
  if (req.body.username && req.body.password) {
    db.get("SELECT * FROM Users WHERE username = '" + req.body.username + "' LIMIT 1");
    res.end("Account already exists");
    var stmt = db.prepare("INSERT INTO user VALUES (?,?)");
    stmt.run(req.body.username, req.body.password);
    res.end("OK");
  } else {
    res.end("username and password are requesiraehri");
  }
});

这里出了什么问题，因为如果我试图注册用户，它说它已经存在，而数据库完全是空的。 有什么帮助吗？

Answer 1

在接下来的所有内容中，我假设您使用的是这个 sqlite 库：https://www.npmjs.com/package/sqlite3

res.send("Account already exists"); 被无条件调用。您忘记检查查询结果。

但是您的代码中还有其他问题。首先，您没有使用带有回调的异步函数。 db.get 是一个异步函数，将回调作为第二个参数，它接收结果集的第一行或错误 (see documentation)。

app.post('/register', function(req,res){
  console.log("got register post", req.body)
  if (req.body.username && req.body.password) {
    db.get("SELECT * FROM Users WHERE username = ? LIMIT 1", req.body.username, function(err, row){
      if(row){
        res.end("Account already exists");
      } else {
        var stmt = db.prepare("INSERT INTO user VALUES (?,?)");
        stmt.run(req.body.username, req.body.password);
        res.end("OK");
      }
    }
  } else {
    res.end("username and password are required");
  }
});

stmt.run 也是异步的。所以请注意，res.end('OK') 是在在数据插入数据库之前执行的。在发送响应之前检查插入语句是否成功运行可能会更好。

另一个重要问题是您在数据库中以明文形式插入密码。你应该考虑加密它。 切勿以明文形式存储密码！