蓝鸟警告：在处理程序中创建了一个承诺，但没有从它返回

Question

我有一个文件名数组，我使用这个节点的异步模块进行迭代。

       async.eachSeries(imageStore, function(imageDetails,callback){
            mongoMan.updateCollection('imageStore',imageDetails,{_id: imageDetails.fileName}).then(function(res){
                  return callback(null, res);
                }).catch(function(err){
                  logger.error(err);
                  return callback(err);
                });
            },function(err){
              callback(null);
            });

updateCollection()函数是这样的：

exports.updateCollection = function(collection, values, condArr){

  var result = imageStores.updateAsync(condArr, values, { upsert: true }).then(function(res){
    return new Promise.resolve(res);
  }).catch(function(err){
    logger.error(err);
  });

  return new Promise.resolve(result);
}

此代码运行良好，可更新数据库和所有内容。但我仍然无法解决蓝鸟抛出的警告：

Warning: a promise was created in a handler but was not returned from it
at Object.exports.updateCollection (/home/swateek/Documents/codebase/poc/apps/webapp/server/components/mongodb/mongoConn.js:46:22)
at /home/swateek/Documents/codebase/poc/apps/webapp/server/components/imageStore.js:72:24
at /home/swateek/Documents/codebase/poc/apps/webapp/node_modules/async/lib/async.js:181:20
at iterate (/home/swateek/Documents/codebase/poc/apps/webapp/node_modules/async/lib/async.js:262:13)

已查找解决方案here，但在我的情况下这还不够令人信服。至少有办法关闭警告吗？

更新

请检查下面的正确答案，这是我的调用函数的样子：

function(callback){// post imageStore data to DB
          async.eachSeries(imageStore, function(imageDetails,callback){
            mongoMan.updateCollection('imageStore',imageDetails,{_id: imageDetails.fileName}).catch(function(err){
                  logger.error(err);
                  return callback(err);
                });
            return callback(null);
            },function(err){
              callback(null);
            });
        }

Answer 1

如果您尝试将异步库回调与 Promise 混合使用，您就是在自找麻烦并放弃编程优势。选择一种或另一种结构并在任何地方使用它。就个人而言，我建议你转向承诺，只是“承诺”目前与回调一起工作的函数。如果您使用的是 Bluebird Promise 库，那么它有 Promise.promisify() 和 Promise.promisifyAll() 可以很容易地承诺使用标准 node.js 异步回调的东西，因此您可以使用 Promise 逻辑控制一切。

现在，谈谈您的具体问题。该错误意味着您正在 .then() 处理程序中创建承诺，但这些承诺不会返回或链接到任何先前的承诺。因此，它们完全独立于您的其他链。这通常是一个错误（因此是警告的原因）。唯一可能真正想要这样做的是你有一些你不希望外部承诺等待的火灾和忘记操作，并且你没有跟踪不是你在这里所拥有的错误。

改成这样：

exports.updateCollection = function(collection, values, condArr){
  return imageStores.updateAsync(condArr, values, { upsert: true}).catch(function(err){
    logger.error(err);
    // rethrow error so the caller will see the rejection
    throw err;
  });
}

变化：

1. 返回主承诺，而不是创建新承诺。
2. 不需要return new Promise.resolve(res);，因为res 已经从该承诺中返回，因此您可以删除整个.then() 处理程序，因为它没有做任何事情。
3. 不需要return new Promise.resolve(result);，因为您可以直接返回之前的承诺。

仅供参考，虽然您在这里根本不需要它，但可以直接调用 Promise.resolve() 而无需 new。

Answer 2

这里有一些问题：

 var result = imageStores.updateAsync(condArr, values, { upsert: true }).then(function(res){
    return new Promise.resolve(res);
 }).catch(function(err){
    logger.error(err);
 });

 return new Promise.resolve(result);

首先：Promise.resolve 不是构造函数，因此不应与new 一起使用。其次：调用Promise.resolve( result ) 根本没有意义，只需返回已经是Promise 的result。中间的then 也毫无意义。您可以将该代码简化为：

return imageStores.updateAsync(condArr, values, { upsert: true })
  .catch( function(err){
      logger.error(err);
  } )
;