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

[SilentDepth]

比如说做一件事分几个动作： 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 的理解有问题？

[leojoy710]

then 可以 return 一个 promise...然后就可以继续 then 了...

[SilentDepth]

@leojoy710 是可以链式调用，但不能让 step2 的异步完成后再执行 finish

[spritevan]

http://www.html5rocks.com/en/tutorials/es6/promises/

[leojoy710]

@SilentDepth step1.then((d) => step2(d)).then(() => finish())

[napsterwu]

链式调用的时候，你可以用.catch ，任何一步出错都会跳到最后的 reject ，就和 try catch 一样。然后只要你写的每一个 Promise 都有 return Promise ，就可以链式调用。只是我不知道怎么让第一个 Promise 的数据传递到最后，我就用了 block variable 。

[napsterwu]

http://jsbin.com/wuxamogaza/1/edit?js,console

这个应该很直观了

[markmx]

建议还是使用匿名方法吧..写的太长了...还是不够整齐的.
Promise 你当做一个流程的管理.不要把逻辑代码过多的放进去

[bdbai]

试试 async/await

[SilentDepth]

@leojoy710
按这个思路试了一下，结果不符预期：
<script src="https://gist.github.com/SilentDepth/bd558eb503b11f733f97be3322d83b51.js"></script>
step2 也是一个异步动作， finish 需要 step2 完成后才能执行

[xqin]

```

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);
});
});

```

[magicdawn]

https://cnodejs.org/topic/5709cb2f94b38dcb3c09a7ac
https://cnodejs.org/topic/573b2d64fcf698421d20359d

[vghdjgh]

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]

格式全乱了，看来这里不适合讨论代码。。。

[vghdjgh]

<script src="https://gist.github.com/plantain-00/ec1a8c6a8ea857ce53d3c066949d60b2.js"></script>

[xjp]

我一般不喜欢使用 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]

这里的陷阱在于你的 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]

赞同 @mcfog . promise 用的爽前提是你的异步函数已经包成 promise 了，而不是每次都在 then()里生成 promise.

[otakustay]

核心是你先把 setTimeout 改成返回 Promise 的方法，然后就爽了

[colatin]

Promise.delay

[jerray]

支持 generator 吗？如果支持可以参考下这篇 https://www.promisejs.org/generators/
还有 co https://github.com/tj/co

[leojoy710]

@SilentDepth 原来是我说的太简略...请看 @mcfog 的回复...

[hging]

async 大法好

[SilentDepth]

感谢 @napsterwu @xjp @mcfog
我发现我的问题在于不知道`then()`的参数返回一个 Promise 会并入外部的 Promise composition （ MDN 没有写啊……），看来还是我对 Promise 的理解不到位。

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

to @xjp : 另外，我说的是 ES2015 里的 Promise ，构造函数的参数必须要有，这个怪我一开始没说清。

[napsterwu]

@SilentDepth Promise.all()

[hantsy]

用 RxJS 代替。

[SilentDepth]

@napsterwu `Promise.all()`就不是分步了呀

[napsterwu]

@SilentDepth 我上面不是还发了个链接演示怎么分步么

[napsterwu]

@SilentDepth http://jsbin.com/wuxamogaza/edit?js,console

[napsterwu]

@SilentDepth 抱歉没搞好，左上角要选择 ES6

[SilentDepth]

@napsterwu 我是说「 step2 的异步成功后执行 finish 」这个需求用`Promise.all()`不能实现

[SilentDepth]

通过空 Promise 实现了一个中止过程功能：
https://gist.github.com/SilentDepth/67866e565946e84b1409d16d294b6181
但感觉这样不优雅。

Any idea?

[yolio2003]

各种大神回复了，楼主还是没明白。。。

[yolio2003]

所以我觉得其实是 promise 本身的问题 （逃

[jimliang]

楼主对 Promise 根本不熟练，其次 step1 和 step2 的逻辑就是一样的，可以封装在一起。两行搞定
```
const step = data=>new Promise(resolve=>setTimeout(resolve, 1000, data));
step('1').then(step).then(data=> console.log(`完成: ${data}`));
```

[SilentDepth]

@jimliang 刚学。如果是需要各个动作可以独立工作呢？

[jimliang]

@SilentDepth 如果是独立的就另外处理。 Promise 的重点是形成 Promise 链， then 的函数要返回具体的对象或者 Promise 作为下一个传递。我看你把一个 Promise then 了两次，也就是说 resolve 后两个 then 都会执行并不会顺着一条链执行了。