V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
liufei123
V2EX  ›  Node.js

promise.all 里异步比 for of 循环做异步操作时间更快吗?

  •  
  •   liufei123 · 2019-06-07 00:27:18 +08:00 · 5954 次点击
    这是一个创建于 1998 天前的主题,其中的信息可能已经有所发展或是发生改变。

    按理说 promise.all 里异步是并行的 for of 循环做异步操作是串行的 在 promise.all 方法里运行时间应该更短,但是遇到了问题? 为什么在 promise.all()方法中做 mongoDB 的查询操作 要比在 for of 里的做同样的操作时间更久

    async function test(){
    	  let userItem=await User.find({},{uid:1,_id:0}).limit(10)
    	  console.time('程序耗时 1')
    	  for(var item of userItem){
    	       await User.findOne({
    	           uid:item.uid
    	       })
    	   }
              console.timeEnd('程序耗时 1')
              console.time('程序耗时 2')
    	  await Promise.all(userItem.map(async item=>{
    		  let user=await User.findOne({
    			  uid:item.uid
    		  })
    		  return user
    	  }))
        console.timeEnd('程序耗时 2')
    }
    程序耗时 1: 32.129ms
    程序耗时 2: 3025.852s
    

    感觉 promise.all 方法查询完之后卡了好久才返回结果 猜测是数据库的连接问题吗?? 求大佬解答!

    14 条回复    2019-06-08 16:45:19 +08:00
    geelaw
        1
    geelaw  
       2019-06-07 02:13:36 +08:00 via iPhone
    是“并发”而不是“并行”。

    Promise.all 等价于下面这个写法(不考虑异常情况):

    var awaitables = [];
    for (var item of userItem)
    awaitables.push(User.findOne({ uid: item.uid });
    for (var awtb of awaitables)
    await awtb;

    你可以试试这样做的情况。

    另外你的计时方法不好,应该掐头(抹去 JIT 编译)并重复。
    WittBulter
        2
    WittBulter  
       2019-06-07 02:55:08 +08:00
    你的 `for of` 中的 await 会等待上一个执行完才会执行下一个。
    Promise.all 并不会保证顺序,也不等待上一个异步任务执行完,但是可以保证在所有异步全部执行完后才结束。
    zjsxwc
        3
    zjsxwc  
       2019-06-07 07:02:33 +08:00 via Android   ❤️ 1
    试试直接返回 promise 呗,

    await Promise.all(userItem.map(item=>{return User.findOne({ uid:item.uid }) }))

    楼主原先的写法类似于一群已经拉完屎的人挤在厕所等茅坑。
    liufei123
        4
    liufei123  
    OP
       2019-06-07 13:25:18 +08:00
    @zjsxwc 对我之后就是直接返回的 promise 但是时间感觉也是差不多 promise.all 还是会卡好久
    zjsxwc
        5
    zjsxwc  
       2019-06-07 14:00:08 +08:00 via Android
    @liufei123

    不负责任猜测,可能是你的 mongodb 服务器性能太差,n 个请求顺序阻塞处理居然比并发同时处理 n 个请求还快,2333333
    liufei123
        6
    liufei123  
    OP
       2019-06-07 16:00:46 +08:00
    @zjsxwc 我是开的本地的 localhost 的 有什么影响吗 刚学 node 不久。。
    SoloCompany
        7
    SoloCompany  
       2019-06-07 17:50:36 +08:00 via iPad
    怀疑你用了 babel 或者 nodejs 使用了出乎意料的优化导致
    建议你修改一下程序 1,把循环 await 返回值保存到一个数组里面而不是直接丢弃
    iugo
        8
    iugo  
       2019-06-07 18:32:24 +08:00
    不应该是 User.find 比 User.findOne 快的问题吗?

    和 Promise 没有关系.
    liufei123
        9
    liufei123  
    OP
       2019-06-07 19:14:58 +08:00 via iPhone
    @iugo 下面那个 promise all 也是 findOne 啊哈哈
    jiejiss
        10
    jiejiss  
       2019-06-07 19:52:52 +08:00
    我感觉是你的 Promise.all 用的不对
    把 userItem.map(后面的函数的 async 和 await 去掉试试
    iugo
        11
    iugo  
       2019-06-07 21:12:27 +08:00
    @liufei123 是.

    但我的意思是批量获取的时候就不要用 findOne 这种.
    liufei123
        12
    liufei123  
    OP
       2019-06-07 23:09:13 +08:00 via iPhone
    @jiejiss 后面去掉了 其实结果也是差不多的速度没差多少
    jiejiss
        13
    jiejiss  
       2019-06-08 12:51:09 +08:00
    那只能是你的 mongodb 的问题了
    liufei123
        14
    liufei123  
    OP
       2019-06-08 16:45:19 +08:00
    @geelaw 试了下你说的 promise.all 的那种写法 确实是快一些
    关于   ·   帮助文档   ·   博客   ·   API   ·   FAQ   ·   实用小工具   ·   1071 人在线   最高记录 6679   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 37ms · UTC 18:34 · PVG 02:34 · LAX 10:34 · JFK 13:34
    Developed with CodeLauncher
    ♥ Do have faith in what you're doing.