V2EX = way to explore
V2EX 是一个关于分享和探索的地方
Sign Up Now
For Existing Member  Sign In
推荐关注
Meteor
JSLint - a JavaScript code quality tool
jsFiddle
D3.js
WebStorm
推荐书目
JavaScript 权威指南第 5 版
Closure: The Definitive Guide
SilentDepth

用 Promise 如何把多个连续的异步过程写得优雅一点?

  •  1
     
  •   SilentDepth · May 18, 2016 · 14744 views
    This topic created in 3637 days ago, the information mentioned may be changed or developed.

    比如说做一件事分几个动作: step1 、 step2 、 finish ,前两个是异步的, step1 完成后才能执行 step2 (然后 finish )。我想到用 Promise 这么写:

    let prom = new Promise((resolve, reject) => {
      // step1 ,用 setTimeout 模拟一下异步
      setTimeout(() => resolve('1'), 1000);
      // 先忽略 reject 相关动作
    });
    prom.then(data => {
      // step2
      setTimeout(() => {
        prom.then(data => {
          // finish
          console.log(`完成: ${data}`)
        });
      }, 1000);
    });
    

    可这样不就成 callback hell 了吗?而且从 step2 往后传数据也不方便。

    由此引发的还有 step1 出错执行reject()如何中止整个过程的问题。

    还是我对 Promise 的理解有问题?

    36 replies    2016-05-20 09:35:30 +08:00
    leojoy710
        1
    leojoy710  
       May 18, 2016   ❤️ 1
    then 可以 return 一个 promise...然后就可以继续 then 了...
    SilentDepth
        2
    SilentDepth  
    OP
       May 18, 2016
    @leojoy710 是可以链式调用,但不能让 step2 的异步完成后再执行 finish
    leojoy710
        4
    leojoy710  
       May 18, 2016
    @SilentDepth step1.then((d) => step2(d)).then(() => finish())
    orcusfox
        5
    orcusfox  
       May 18, 2016
    链式调用的时候,你可以用.catch ,任何一步出错都会跳到最后的 reject ,就和 try catch 一样。然后只要你写的每一个 Promise 都有 return Promise ,就可以链式调用。只是我不知道怎么让第一个 Promise 的数据传递到最后,我就用了 block variable 。
    orcusfox
        6
    orcusfox  
       May 18, 2016
    markmx
        7
    markmx  
       May 18, 2016
    建议还是使用匿名方法吧..写的太长了...还是不够整齐的.
    Promise 你当做一个流程的管理.不要把逻辑代码过多的放进去
    bdbai
        8
    bdbai  
       May 18, 2016 via Android
    试试 async/await
    SilentDepth
        9
    SilentDepth  
    OP
       May 18, 2016
    @leojoy710
    按这个思路试了一下,结果不符预期:
    <script src="https://gist.github.com/SilentDepth/bd558eb503b11f733f97be3322d83b51.js"></script>
    step2 也是一个异步动作, finish 需要 step2 完成后才能执行
    O3YwA1ENkb7i35XJ
        10
    O3YwA1ENkb7i35XJ  
       May 18, 2016
    ```

    new Promise((resolve, reject) => { //step 1
    setTimeout(() => resolve('1'), 1000);
    }).then(data => {
    return new Promise((resolve, reject) => { //step 2
    setTimeout(() => resolve([data, '2']), 1000);
    }).then(data => { //finish
    console.log('finish', data);
    });
    });

    ```
    vghdjgh
        12
    vghdjgh  
       May 18, 2016
    function step1Async() {
    return new Promise((resolve, reject) => {
    step1((error, result) => {
    if (error) {
    reject(error);
    } else {
    resolve(result);
    }
    });
    });
    }
    ...
    try{
    const result = await step1Async();
    await step2Async();
    await finishAsync();
    } catch(error){
    // error
    }
    vghdjgh
        13
    vghdjgh  
       May 18, 2016
    格式全乱了,看来这里不适合讨论代码。。。
    vghdjgh
        14
    vghdjgh  
       May 18, 2016
    xjp
        15
    xjp  
       May 18, 2016
    我一般不喜欢使用 Promise 构造函数 我一般这么用的


    ```
    let prom = new Promise();

    prom.then(data => {
    // step 2
    let prom1 = new Promise();
    setTimeout(() => prom1.resolve('2'), 1000);
    return prom1;
    }).then(data =>{
    // step 3
    return "3";
    }).then(data =>{
    // finish
    console.log('finish', data);
    });

    // step 1
    setTimeout(() => prom.resolve('1'), 1000);


    ```

    如果 then 里 return 的值是 promise 则将 resolve 的结果传入下一个 then

    如果 then 里 return 返回的不是 promise 则将结果直接传入下一个 then
    mcfog
        16
    mcfog  
       May 18, 2016   ❤️ 1
    这里的陷阱在于你的 setTimeout 是 callback 风格的,不是 promise 风格的

    一定要那 setTimeout 打比方(或者比如外部库是 callback 风格)的话,一定要先转换成 promise 风格

    ```

    function step1() {
    console.log('step1 start');
    return new Promise(resolve => {setTimeout( ()=> {resolve('step1 result');}, 1000)});
    }
    function step2(data) {
    console.log('step2, param is', data);
    return new Promise(resolve => {setTimeout( ()=> {resolve('step2 result');}, 1000)});
    }


    //正常的 promise 串行代码如下
    step1()
    .then(step2)
    .then(result => {
    console.log('finish, param is', result)
    });
    ```

    bluebird 等三方 promise 一般都提供标准的 promisify 方法做这件事
    FlowMEMO
        17
    FlowMEMO  
       May 18, 2016
    赞同 @mcfog . promise 用的爽前提是你的异步函数已经包成 promise 了,而不是每次都在 then()里生成 promise.
    otakustay
        18
    otakustay  
       May 18, 2016 via iPad
    核心是你先把 setTimeout 改成返回 Promise 的方法,然后就爽了
    colatin
        19
    colatin  
       May 18, 2016
    Promise.delay
    jerray
        20
    jerray  
       May 18, 2016
    支持 generator 吗?如果支持可以参考下这篇 https://www.promisejs.org/generators/
    还有 co https://github.com/tj/co
    leojoy710
        21
    leojoy710  
       May 19, 2016
    @SilentDepth 原来是我说的太简略...请看 @mcfog 的回复...
    hging
        22
    hging  
       May 19, 2016 via iPhone
    async 大法好
    SilentDepth
        23
    SilentDepth  
    OP
       May 19, 2016
    感谢 @napsterwu @xjp @mcfog
    我发现我的问题在于不知道`then()`的参数返回一个 Promise 会并入外部的 Promise composition ( MDN 没有写啊……),看来还是我对 Promise 的理解不到位。

    现在的问题就是中间的`catch()`如何优雅地中止掉整个 composition 。调用`reject()`后会直接跳到后面最近的 rejection handler ,然后就继续走正常路线了。不是应该有个`Promise.prototype.cancel()`之类的方法吗?

    to @xjp : 另外,我说的是 ES2015 里的 Promise ,构造函数的参数必须要有,这个怪我一开始没说清。
    orcusfox
        24
    orcusfox  
       May 19, 2016
    @SilentDepth Promise.all()
    hantsy
        25
    hantsy  
       May 19, 2016
    用 RxJS 代替。
    SilentDepth
        26
    SilentDepth  
    OP
       May 19, 2016
    @napsterwu `Promise.all()`就不是分步了呀
    orcusfox
        27
    orcusfox  
       May 19, 2016
    @SilentDepth 我上面不是还发了个链接演示怎么分步么
    orcusfox
        29
    orcusfox  
       May 19, 2016
    @SilentDepth 抱歉没搞好,左上角要选择 ES6
    SilentDepth
        30
    SilentDepth  
    OP
       May 19, 2016
    @napsterwu 我是说「 step2 的异步成功后执行 finish 」这个需求用`Promise.all()`不能实现
    SilentDepth
        31
    SilentDepth  
    OP
       May 19, 2016
    通过空 Promise 实现了一个中止过程功能:
    https://gist.github.com/SilentDepth/67866e565946e84b1409d16d294b6181
    但感觉这样不优雅。

    Any idea?
    yolio2003
        32
    yolio2003  
       May 19, 2016
    各种大神回复了,楼主还是没明白。。。
    yolio2003
        33
    yolio2003  
       May 19, 2016
    所以我觉得其实是 promise 本身的问题 (逃
    jimliang
        34
    jimliang  
       May 20, 2016
    楼主对 Promise 根本不熟练,其次 step1 和 step2 的逻辑就是一样的,可以封装在一起。两行搞定
    ```
    const step = data=>new Promise(resolve=>setTimeout(resolve, 1000, data));
    step('1').then(step).then(data=> console.log(`完成: ${data}`));
    ```
    SilentDepth
        35
    SilentDepth  
    OP
       May 20, 2016
    @jimliang 刚学。如果是需要各个动作可以独立工作呢?
    jimliang
        36
    jimliang  
       May 20, 2016
    @SilentDepth 如果是独立的就另外处理。 Promise 的重点是形成 Promise 链, then 的函数要返回具体的对象或者 Promise 作为下一个传递。我看你把一个 Promise then 了两次,也就是说 resolve 后两个 then 都会执行并不会顺着一条链执行了。
    About   ·   Help   ·   Advertise   ·   Blog   ·   API   ·   FAQ   ·   Solana   ·   2447 Online   Highest 6679   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 68ms · UTC 07:11 · PVG 15:11 · LAX 00:11 · JFK 03:11
    ♥ Do have faith in what you're doing.