同步迭代在 JavaScript 中不起作用

Question

我正在尝试遍历一个数组并为每个被迭代者创建记录。这就是我正在做的事情，就像在另一个问题中提到的那样：

async.each(data, (datum, callback) => {
  console.log('Iterated')

  Datum.create({
    row: datum,
  }).exec((error) => {
    if (error) return res.serverError(error)
    console.log('Created')
    callback()
  })
})

不幸的是，结果如下：

Iterated
Iterated
Iterated
Created
Created
Created

这不是我们想要的：

Iterated
Created
Iterated
Created
Iterated
Created

我做错了什么？

Answer 1

async.eachSeries() 将一次运行一次迭代，并等待每次迭代终止，然后再进行下一步。

我在每次创建之前创建一个唯一的用户友好标识符（如 1、2、3 等）。为此，我必须查询数据库以找到最新的标识符并增加它，这是不可用的，因为记录几乎是同时创建的。

这听起来像是瓶颈。我不喜欢串行运行异步代码，因为这通常会减慢进程。这种方法怎么样：

• 由于data，您知道需要多少标识符。
• 在后端实现一个函数，该函数不会一次创建单个但n 这样的标识符（包括必要的递增等）并将该数组返回到前​​端。现在，您可以使用/映射预先计算的 ID 数组到数据数组来并行运行常规请求。

这应该会将运行时间从(createAnId + request) * data.length 减少到几乎是单次迭代的运行时间。由于所有这些请求都可以并行运行，因此大多是重叠的。

Answer 2

看起来 Datum.create 是一个异步函数。 forEach 遍历数组的三个元素中的每一个，依次记录它们。并且由于 JavaScript 在异步事件返回之前不会阻塞，因此您将依次获取每个 console.logs。 然后在 一些 时间后，结果进来，“创建”记录到控制台。

Answer 3

您似乎正在使用异步数据处理库。对于您想要得到的结果，您需要同步处理数据。你可以这样做：

data.forEach(function(datum) {
  console.log('Iterated')

  Datum.create({
    row: datum,
  }).exec((error) => {
    if (error) return res.serverError(error)
    console.log('Created')
    callback()
  })
})

您现在可能还想完全删除 callback 函数，因为数据是同步处理的。