本机 javascript 承诺循环

Question

这样做的目的是在继续之前循环请求几次。 github.repos.getBranch 应该先构建一个包含 10 个元素的对象，然后我才能继续使用结果。

github api 请求按预期工作，但结果记录在：console.log('++', res) 返回以下内容：++ [ undefined, undefined, undefined, undefined, undefined ]。 如果我在循环结束后登录allBranches，数据就在那里。

在 github 请求之后，我显然错过了一步。我已经尽我所能地重构了它，但没有成功。

getAllData: () => {
  let allBranches = []
  let startGetData = config.repositories.map(repo => {
    config.branches.map(branch => {
      return allBranches.push(
        github.repos.getBranch({
          owner: config.organisation,
          repo,
          branch
        })
      )
    })
  })

  return Promise.all(startGetData)
  .then(res => {
    console.log('++', res)
  })
}

Answer 1

您的config.repositories.map 调用没有返回任何内容（这是一个没有return 的详细箭头函数）。所以你最终得到了一个undefined 的数组，这就是你传递给Promise.all 你需要一个return。

但这不是唯一的问题。您将startGetData 传递给Promise.all，但这不是存储github 承诺的地方。您将它们存储在allBranches 中。所以你需要等待allBranches，而不是startGetData。

我怀疑您的目标是获取包含一系列分支的存储库数组。如果是这样，您将需要多个 Promise.all 调用（在存储库的分支上等待）和一个整体的 Promise.all 调用（等待所有存储库完成）。

如果这是您的目标，那么它的外观是这样的：

getAllData: () => Promise.all(                    // Gather up all results
  config.repositories.map(repo =>                 // Map repos to promises
    Promise.all(config.branches.map(branch =>     // Map branches to promises
      github.repos.getBranch({                    // Promise for branch
        owner: config.organisation,
        repo,
        branch
      })
    )).then(branches => ({repo, branches}))       // Wrap up branch results in...
  )                                               // ...an object identifying...
)                                                 // ...the repo

这给了你一个这样的对象数组的承诺：

[
    {
        repo: /*...repo...*/,
        branches: [
            /*...branch...*/,
            /*...branch...*/,
            /*...branch...*/
        ]
    },
    {
        repo: /*...repo...*/,
        branches: [
            /*...branch...*/,
            /*...branch...*/,
            /*...branch...*/
        ]
    },
    {
        repo: /*...repo...*/,
        branches: [
            /*...branch...*/,
            /*...branch...*/,
            /*...branch...*/
        ]
    }
]

如果你只想要一个纯数组：

[
    [
        /*...branch...*/,
        /*...branch...*/,
        /*...branch...*/
    ]
    [
        /*...branch...*/,
        /*...branch...*/,
        /*...branch...*/
    ],
    [
        /*...branch...*/,
        /*...branch...*/,
        /*...branch...*/
    ]
]

...然后删除最后的then 子句（.then(branches => ({repo, branches}))）。