为什么如果我将resolve()放入fs的回调函数中，promise不会返回答案

【问题标题】：Why if I put resolve() into the callback function of fs, the promise doesn't return为什么如果我将resolve()放入fs的回调函数中，promise不会返回
【发布时间】：2020-02-27 01:38:06
【问题描述】：

（提示：我正在使用基于koa 的nodejs 框架eggjs）

下面的代码可以成功运行。但是如果我把resolve()放到fs.renameSync()的回调函数中，promise不会返回任何东西，请求会保持pending状态。 p>

这是什么原因造成的？这和执行顺序有关系吗？


 async uploadAsset(assetName, file) {
    const { app } = this;
    const logger = this.logger;
    return new Promise(function(resolve, reject) {
      fs.renameSync(file.filepath, `${app.config.multipart.projectAssetLocalPath}${assetName}`, err => {
        if (err) {
          logger.warn(err);
          reject();
        }
      });
      resolve();
    });
  }

【问题讨论】：

我知道没有理由将同步的fs.renameSync() 包装在一个承诺中。它只是让它更难使用。
你打电话给对方的.then()？比如uploadAsset('name', file).then(function() {...});?
而且，fs.renameSync() 没有回调。

标签： javascript node.js koa

【解决方案1】：

renameSync 是rename 的同步版本。它不接受回调作为参数；它只接受oldpath and a newpath。如果您传递第三个参数，它将被忽略；你传递的回调函数永远不会被调用。

如果您希望它基于回调，请改用fs.rename，它确实接受回调。

你的resolve 也是在回调之外此刻，它应该在里面：

async uploadAsset(assetName, file) {
  const { app } = this;
  const logger = this.logger;
  return new Promise(function(resolve, reject) {
    fs.rename(file.filepath, `${app.config.multipart.projectAssetLocalPath}${assetName}`, err => {
      if (err) {
        logger.warn(err);
        reject();
      }
      resolve();
    });
  });
}

或者使用 fs.promises 代替，无需自己构建 Promise 即可完成此操作。

async uploadAsset(assetName, file) {
  const { app } = this;
  const logger = this.logger;
  return fs.promises.rename(file.filepath, `${app.config.multipart.projectAssetLocalPath}${assetName}`)
    .catch((err) => {
      logger.warn(err);
      throw new Error(err);
    });
}

【讨论】：