有没有更好的方法来处理 NodeJS 中的嵌套承诺？

Question

我的代码看起来类似于：

const MyClass {
  checkExists: function(db_client) {
    return new Promise(fulfill, reject) {
      var sql = 'select * from table';
      db_client.connect().then(c => {
      }).then(res => {
        client.release();
        fulfill(res.rows[0].col1 === 1 ? true : false);
      }).catch(error => {
        reject(error);
      });
    });
  }
  doSomething: function(db_client) {
    return new Promise(fulfill, reject) {
      var sql = 'delete from table where x=1';
      db_client.connect().then(c => {     
      }).then(res => {
        fulfill();
      }).catch(error => {
        reject(error);
      });
    });
  }
};

module.exports = MyClass;

var myc = require('./MyClass.js');

myc.checkExists(db_client).then(resp => {
  if(resp === true) {
    myc.doSomething(db_client).then(resp => {
    console.log('success.');
  } else {
    console.log('we are done.');
  }
  }).catch(error => {
    console.log(error);
  });
}).catch(error => {
  console.log(error);
});

根据上面的示例，我必须运行一个依赖于另一个查询结果的查询。 （是伪代码，如有错误请见谅）

但是，我注意到它开始导致嵌套的 Promise 或函数调用，其中包含来自另一个 Promise 的实现中的 Promise。

我可以看到这种情况越来越糟。这是犹太洁食吗？有没有更好的方法来思考/处理我正在尝试做的事情？

编辑：

不知道为什么它被标记为一个问题的重复，其中发布者似乎明确意识到反模式并询问如何避免它与反模式/解决方案不知道的问题张贴者，但认识到编程风格中的问题并正在寻求帮助（其中的讨论可能会产生相同类型的解决方案）。

Answer 1

如果 .then 中的函数返回一个值，那么 .then() 的结果将成为解析为该值的 Promise。使用这个原理，我们可以重写你的一个方法。

function doSomething(db_client) {
  var sql = 'delete from table where x=1';
  return db_client.connect().then(() => undefined);
}

如果你想返回连接的值，那就更简单了

function doSomething(db_client) {
  var sql = 'delete from table where x=1';
  return db_client.connect();
}

Answer 2

有没有更好的方法来处理 NodeJS 中的嵌套承诺？

是的，有。您可以进行很多改进。您的伪代码似乎缺少一些部分，但我将展示最简单的版本，以捕捉您所做工作的精神：

const MyClass {
    checkExists: function(db_client) {
        var sql = 'select * from table';
        return db_client.connect().then(c => {
            // missing code here in your pseudo code
            // do something with c here to create the variable res
            client.release();
            // make this be the resolved value of the returned promise
            return res.rows[0].col1 === 1;
        });
    }
    doSomething: function(db_client) {
        var sql = 'delete from table where x=1';
        return db_client.connect().then(c => {
            // do something with c here
            // return value or another promise
        });
    }
  };

  module.exports = MyClass;

  var myc = require('./MyClass.js');

  myc.checkExists(db_client).then(resp => {
    if(resp === true) {
        return myc.doSomething(db_client).then(resp => {
            console.log('success.');
            // return something here to be the resolved value of the promise
            return resp;
        });
    } else {
        console.log('we are done.');
        // return something here to be the resolved value of the promise
        return resp;
    }
  }).catch(error => {
    console.log(error);
  });

以下是基本概念：

1. 不要用手动创建的 Promise 包装现有的 Promise。这被称为反模式，它没有提供任何好处，使您的代码膨胀并且是错误处理错误的机会。只是回报你已经拥有的承诺。

2. 通过从.then() 处理程序返回承诺，从.then() 处理程序中链接嵌入承诺。

3. 您不需要在每个级别都.catch()，除非您明确需要在较低级别执行某些操作。被拒绝的承诺会自动向上传播到下一个.catch()。这可以大大简化错误处理。

4. 如果你需要一个较低级别的.catch()，即使只是为了记录目的，那么你必须throw一个来自.catch()的错误来保持承诺被拒绝，否则拒绝将被视为“已处理”，链将切换到已解决。

5. 在.then() 处理程序中，您有三个选项。您可以返回另一个承诺，它将被添加到链中。您可以返回一个值，它将成为父 Promise 链的已实现值。您可以抛出异常，当前链将被拒绝。如果什么都不返回，那么父 Promise 链的已实现值将变为 undefined（就像一个什么都不返回的函数一样）。

仅供参考，这里有几篇关于 Promise 反模式的文章，值得研究和实践：

Six Promise Anti-Patterns

Deferred and .then(success, fail) anti-patterns

关于您的代码的其他说明：

1. 在某些地方，您似乎缺少释放数据库连接的代码。

2. 在设计时，您应该非常清楚仅处理自己的操作（包括错误）的函数，并且不期望调用者会处理错误而不是其工作的函数执行操作并返回结果或错误，以便调用者可以决定如何处理它们。这决定了您是否只是让被拒绝的承诺返回（供调用者处理），或者您是否必须在这里.catch() 他们并对其进行处理。

1234563 .

Answer 3

如果你从 then() 中返回一个 Promise，根 Promise 对象的下一个 then() 将获得之前的 Promise 的结果。示例

Promise.resolve(42).then(function(answer) {
  return new Promise(function(resolve) {
    setTimeout(function() {
      resolve('Answer to life = ' + answer);
    }, 3000);
  })
}).then(function(str){
    document.write(str);
});