节点。从数据库中获取数据并使用结果发出发布请求[重复]

Question

我有一个数据库中的用户列表。在注册 db 时，他们最初获得了 access_token，但随着它过期，我需要通过 refresh_token 创建一个新的 access_token，它也已保存在 db 中针对用户。

我需要向端点发出 POST 请求以获取新的访问令牌。

我正在尝试遍历数据库中的每一行并以这种方式获取新的 access_token。当有一个用户时，它工作得很好，但是当users 数组中有多个用户时，每个user 都是相同的，即数据库表中的最后一个用户。我猜该请求尚未完成，它正在尝试发送新请求或类似的东西。

有谁知道如何解决这个问题？

app.get('/get-users', function (req, res) {
    connection.query('SELECT * from users', function(err, rows, fields) {
        if(err) console.log(err);

        for(var i = 0; i < rows.length; i++) {
            name = rows[i].name;
            userId = rows[i].userId;
            console.log(userId);// this gives the correct userid
            var authOptions = {
                url: url,
                headers: {
                    'Authorization': 'Basic ' + (new Buffer(client_id + ':' + client_secret).toString('base64'))
                },
                form: {
                    grant_type: 'refresh_token',
                    refresh_token: refresh_token
                },
                json: true
            };

            request.post(authOptions, function(error, response, body) {
                if (!error && response.statusCode === 200) {
                    var access_token = body.access_token;
                    users.push({name: name, userId: userId, accessToken: access_token});
                    console.log(userId)//gives last entry in db
                }
            });
        setTimeout(function () {
           console.log(users)
         }, 3000)
        }
    });
   res.send('getting here');
})

Answer 1

你的问题是这样的：

    for(var i = 0; i < rows.length; i++) {
        name = rows[i].name;
        userId = rows[i].userId;
        // ...
        request.post(authOptions, function(error, response, body) {
          // you're using name and userId here
        });
        // and also you use users here:
        console.log(users)
    }

现在，问题是在所有循环迭代完成之前不会调用任何回调（即使是第一次迭代）。

首先，console.log(users) 将始终获得 users 的值，无论它在任何users.push() 发生之前是什么。

此外，对于每个回调，name 和 userId 的值将是相同的 - 无论在您的 for 循环的最后一次迭代之后是什么。

您的name 和userId 似乎是全局变量，或者它们是在某个外部范围内定义的。您需要做的是使用let 在本地定义它们，以便它们在每次循环迭代时获得新的绑定。

看一个简单的例子来演示这里发生了什么：

for (var i = 0; i < 4; i++) {
  x = 'Number ' + i;
  setTimeout(function () {
    console.log(x);
  }, 500*i);
}

这不好，因为x 是一个全局变量。现在var:

for (var i = 0; i < 4; i++) {
  var x = 'Number ' + i;
  setTimeout(function () {
    console.log(x);
  }, 500*i);
}

仍然不好，因为每次迭代都共享一个 x 绑定。现在let:

for (var i = 0; i < 4; i++) {
  let x = 'Number ' + i;
  setTimeout(function () {
    console.log(x);
  }, 500*i);
}

之所以有效，是因为每次迭代都会获得一个新的绑定。

更新

另一件事是确保您知道每个request.post 何时结束，以便您可以在正确的时间发送回复。

您可以使用async 来帮助您。您可以使用诸如 Bluebird 或 Q 之类的 Promise 和库来帮助您。但你也可以做一个简单的把戏。如果您想知道您在我上面展示的示例的最后一个回调中，您可以像这样计算调用：

let current = 0, last = 0;
for (let i = 0; i < 4; i++) {
  let x = 'Number ' + i;
  setTimeout(function () {
    if (++current === last) console.log('Last callback:');
    console.log(x);
  }, 500*i);
  last++;
}

这可能不是处理复杂情况的最佳方式，但它展示了正在发生的事情：last 变量在每次迭代时递增，并且回调仅在那时注册。然后回调开始一个接一个地触发，您可以在每次触发 ne 回调时递增 current 变量，并比较是否是最后一个。

现在，一般建议：每次都使用let（或const）而不是var。它的工作方式更加直观。当然，如果您知道自己在做什么，您可以使用var，但如果有疑问，请使用let。