对由Promise解析而未嵌套的数组中的每个结果应用操作

时间:2018-10-14 13:41:05

标签: javascript promise

ESlint和世界其他地方都告诉我,避免嵌套Promises。但是,如何将一个函数分别应用于先前的promise返回数组的多个项目?例如,我想在上一个Promise返回的每个元素上运行doSomethingWithIt,并对其结果进行一些处理。

function getSomeThings() {
   return new Promise((resolve, reject) => {
     resolve(['one', 'two', 'three'])
   })
}

function doSomethingWithIt(theThing) {
  return new Promise((resolve, reject) => {
     console.log(theThing)
     resolve("Did a thing")
  })
}

getSomeThings().then((things) => {
   things.forEach((thing) => {
      doSomethingWithIt(thing)
       .then((result) => {
          console.log(`Result was: ${result}`)
       })
   })
})

实际上,该函数可能会返回ID或一些有用的信息,以便在操作完成后(例如,提交到数据库)而不是仅打印返回值。

如何在不嵌套承诺的情况下实现这一目标?我是否需要添加一个使用Promise.all汇总结果的步骤?还是有一种更整洁的方式...

2 个答案:

答案 0 :(得分:1)

您将在循环中创建的那些承诺上使用Promise.all。不仅使ESLint高兴,而且还使您维持了完整的成功/失败链:

getSomeThings()
    .then((things) => Promise.all(things.map(doSomethingWithIt)))
    .then((results) => {
        for (const result of results) {
            console.log(`Result was: ${result}`);
        }
    })
    .catch((error) => {
        // Handle/report errors
    });

实时示例:

function getSomeThings() {
   return new Promise((resolve, reject) => {
     resolve(['one', 'two', 'three'])
   })
}

function doSomethingWithIt(theThing) {
  return new Promise((resolve, reject) => {
     console.log(theThing)
     resolve("Did a thing")
  })
}

getSomeThings()
    .then((things) => Promise.all(things.map(doSomethingWithIt)))
    .then((results) => {
        for (const result of results) {
            console.log(`Result was: ${result}`);
        }
    })
    .catch((error) => {
        // Handle/report errors
    });

是的,这的确意味着您在所有console.log的承诺都得到解决之前,对其中的任何{em} {}都没有。通常这是一件好事。但是,如果不是在这种情况下,那么打破ESLint规则可能是合理的(正如bergi在a comment中所说的那样,它不是通用的)

答案 1 :(得分:1)

  

ESlint和世界其他地方告诉我,避免嵌套Promises。

是的,您应该避免使用when you can。但是,在很多情况下,嵌套是完全合适的-magic of promises是它们可以嵌套,同时仍然产生相同的最终结果。

ESlin的建议类似于“避免缩进”。是的,不需要的时候不应编写嵌套块。但是,您的用例是“对每个结果应用一个操作”,这需要一个循环,该循环可以证明嵌套/缩进是正确的。

您当然应该avoid forEach,并为您的循环的所有操作完成返回一个适当的承诺:

doSomethingWith