MySQL使用nodejs将新用户插入数据库

Question

我正在尝试使用 nodejs 和 mysql 注册一个新用户。我对 mysql 语法不是很熟悉，所以我可能做错了什么，但这就是我所拥有的：

app.post('/createNewUser', (req, res) => {
  let newUsername = req.body.newUsername;
  let newPassword = req.body.newPassword;
  let newData = 'INSERT INTO users (username, password) VALUES (newUsername, newPassword)';
  db.query(newData, function(err, result) {
    if (err) throw err;
    console.log('New user registered');
  });
});

它给我的错误是“字段列表中的未知列'newUsername'，它说该变量从未使用过，所以我假设它正在将它作为字符串读取？但我不确定如何通过它作为那个变量而不是在引号内。

Answer 1

根据the documentation for the module you referred to，应该是这样的：

let newData = 'INSERT INTO users (username, password) VALUES (?, ?)';
//                                              Placeholders −^−−^
db.query(newData, [newUsername, newPassword], function(err, result) {
//                ^^^^^^^^^^^^^^^^^^^^^^^^^^^−− Values for them
  if (err) throw err;
  console.log('New user registered');
});

或者像这样（你可以选择）：

let newData = 'INSERT INTO users (username, password) VALUES (?, ?)';
//                                              Placeholders −^−−^
db.query({
    sql: newData,
    values: [newUsername, newPassword]
//  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^−−−−−−−−− Values for them
}, function(err, result) {
  if (err) throw err;
  console.log('New user registered');
});

这是一种相当常见的模式，使用 ? 将值放在其中，然后在数组中单独提供值或使用某种索引。 （在我看来，这不是最好的方法，但如果这是您正在使用的模块...）

我更喜欢占位符有名字，我可能会使用这样的函数将名字转换为?：

function doQuery(options, callback) {
    let {sql, parameters, ...rest} = options;
    const values = [];
    for (const name in parameters) {
        sql = sql.replace(":" + name, "?");
        values.push(parameters[value]);
    }
    db.query({sql, values, ...rest}, callback);
}

然后你会这样使用它：

doQuery({
    sql: 'INSERT INTO users (username, password) VALUES (:newUsername, :newPassword)',
    parameters: {newUsername, newPassword}
}, function(err, result) {
    if (err) {
        // Handle/report error
    } else {
        console.log('New user registered');
    }
});

---

旁注：if (err) throw err; 通常是带有 Node.js 回调的反模式。您在调用回调的代码中抛出错误，而不是在您的 自己的 代码中。最好有一种处理/报告错误的方法。