Nodejs 的 Promise 实现是不是有问题？

[haozhang]

我看zakas的understandinges6上面的例子实际运行出来的结果跟zakas讲的结果不一样，比如下面这个例子：

"use strict";

var p1 = new Promise((resolve, reject) => {
    resolve("this is p1");
});

var p2 = Promise.reject(2);

var p3 = new Promise((resolve, reject) => {
    resolve("this is p3");
});


var p4 = Promise.race([p1, p2, p3]);

p4
    .then(
        (value) => {
            console.log(value);
        },
        (error) => {
            console.log(error);
        }
    );

结果是：

this is p1

也就是说p4的fulfillment handler被调用了。

按照zakas讲的（我觉得zakas讲的没错啊），应该是p2初始就是rejected状态的，所以导致p4立即变成rejected，然后调用p4的rejection handler.

但现在却是p1抢先一步变成了fulfilled，然后导致p4变成fulfilled了。

[haozhang]

@otakustay 我的理解是：Promise.race()和Promise.all()方法看的是，谁先调用race或者all添加的回调函数，也就是race/all产生的promise对象的状态是根据谁先调用回调函数来判断的（这个回调函数指的是race/all添加的回调函数），根据这个理解，我仿写了一个race函数，你看看我的理解对不对：

var race = (array) => {
return new Promise((resolve, reject) => {
array.map((cur) => {
cur
.then(
(value) => {
resolve(value);
},
(error) => {
reject(error);
}
);
});
});
};

[haozhang]

<script src="https://gist.github.com/Summerlve/fa021540aabf887bd241.js"></script>

[haozhang]

我受不了了....你看下面的代码吧，上面贴的代码格式太差了。

[haozhang]

https://gist.github.com/Summerlve/98d4b41768e868416a38.js

[otakustay]

没错，race就是这么搞的，.map可以改成.each，更简单的写法：

Promise.race = (array) => new Promise((resolve, reject) => array.forEach((promise) => promise.then(resolve, reject)));

all要更麻烦些，有参数收集等

[haozhang]

all方法我也实现了：
https://gist.github.com/Summerlve/914ac09376833c2935e3

[ysmood]

@otakustay race 和 all 的实现都不完全正确，或者不符合 ES6 的规范。数组的每个 item 可以不是 promise 的，应该用 resolve wrap 起来，比如 race 的例子：

Promise.race = (arr) => new Promise((resolve, reject) => arr.forEach((v) => Promise.resolve(v).then(resolve, reject)));

另外不要忘记判断 arr 是否是 iterable 类型。虽然有点吹毛求疵了 XD

[haozhang]

@ysmood →_→ 我只是大致实现了一下

[otakustay]

@ysmood 我懂，但是我懒得在v2上敲这么多代码啊- -