PromiseAll的实现确实是有问题的

[lincycode]

No description provided.

[lincycode]

并不符合预期, 使用以下p1,p2作为例子就知道问题所在了, p1没有resolve的时候就到p2执行res[1] = 1导致最终结果变为[empty,1]
const p1 = new Promise((resolve, reject) => {
setTimeout(() => {
resolve('hello');
}, 500);
});
let p2 = Promise.resolve(1);

[lincycode]

解决方案为使用计数器而不是用res的数组长度判断, 如果使用数组长度判断应该用push操作更合理, 直接操作下标会导致数组长度"虚假"的增加, 但是push不能用在这里因为是异步请求可能导致最终res的结果不按原顺序呈现, 因此请用计数器判断是否可以最终resolve

[lincycode]

这是我的代码

function PromiseAll(arr) {
  return new Promise((resolve, reject) => {
    const len = arr.length;
    const result = [];
    let succeed = 0;
    for (let i = 0; i < len; ++i) {
      const cur = arr[i];
      if (isPromise(cur)) {
        arr[i].then((res) => {
          process(i, cur);
          // 这里只是简化放在第二个参数,catch行为同
        }, reject);
      } else {
        process(i, cur);
      }
    }
    function process(index, value) {
      result[index] = value;
      if (++succeed === len) {
        resolve(result);
      }
    }
  });
}

[Mayandev]

👏 欢迎提 PR