应该在同步函数中避免 promise 吗？ [复制]

Question

我目前正在尝试美化一些 NodeJS 遗留代码，并且在这样做时提出了我找不到任何最佳实践或入门指南的问题（如果您知道，请与我分享）。

我们的目的不是简单地看到可以通过 Promises 组合同步和异步代码（这很清楚），而是为什么应该例如如果函数的同步处理是明确的，那就去同步代码。这意味着在同步代码中简单地添加一个 Promise 会有什么缺点？有吗？例如。在运行时，或者更重要的是，在异步流程中？

例如以下面的异步函数为例：

Foo.prototype.addNode = function addNode() {
  var self = this;
  return new Promise(function (resolve, reject) {
    var node = new Node(self.getNodes().getLastId() + 1);
    resolve (self.getNodes().add(node));
  });
}

在我们的业务逻辑中，这个函数确实应该是异步的，因为这是在承诺链中递归调用的。

但是，add 函数是同步处理的，因为这是一个简单的 Array.push() 通用元素，并经过了一些验证。

GenericElementList.prototype.add = function add(T) {
  if (this.T.length === 0) {
    this.T.push(T);
    this.typeOfT = T.constructor;
    return this;
  }
  else {
    if (!(T.constructor === this.typeOfT)){
      this.T.push(T);
      return this;
    }
    else {
      throw new IllegalArgumentError('Invalid type of element to push into array! Was: ' + T.constructor.name + ' but should be: ' + this.typeOfT.name + '!');
    }
  }
};

（为了避免混淆：self.getNodes() 返回一个元素 Node，它是一个 GenericElement，GenericElementList 是 GenericElements 的列表数据结构。）

现在我的问题是：如果我们将 Promise 添加到 add-function 中，在这些方法的异步处理中是否存在运行时或流程方面的缺点？有什么理由避免这种情况吗？或者唯一的缺点是一些样板代码？

Answer 1

在 javascript/node 同步执行中，无论你如何表达它都保持同步。举个例子

console.log("stage 1")
findIndex(data, 2, (err, index)=>{
    console.log(err, index)
})
console.log("stage 3")

function findIndex (arr, val, callback){
    console.log("stage 2")
    var idx =arr.indexOf(val)
    if(idx==-1) return callback(new Error("value not found"))
    return callback(null, idx)
}

虽然异步或可以说 I/O 绑定执行始终是异步的。不管怎么表达（有人认为 es6 async/await 让异步执行同步）

console.log("stage 1")
setTimeout(()=>{
    console.log("stage 2")
}, 0)
console.log("stage 3")

打开你的 js 控制台，找出不同之处和执行流程

Promise 也一样

现在循环内的异步函数

1. 当异步任务依赖时，顺序迭代很有用。 （一个操作的结果需要反馈另一个（/s））

(function loop(i) {
if(i==4) return
setTimeout(function() {
  console.log('Working on index', i);
  loop(++i)
 }, 1000);
})(0);

1. 当异步任务相互独立时，并行执行很有用。

var data = [1,2,3,4]
data.map(val=>{
    return new Promise((resolve, reject) => {
        setTimeout(function() {
         console.log('Working on index', val);
         // pass data in reslove() and get when promise resloved
        })
    });
})
Promise.all(data)
.then(done=>{
    // get passed data
    console.log(done);
})
.catch(err=>{
    console.log(err);
})

从中获得更多 https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Promise