如何将此函数转换为不使用 Promise 构造函数反模式

Question

我问过这个question 并被告知要避免使用 Promise 构造函数反模式。我通读了链接的文章，但我不确定如何不使用反模式。这是函数：

db.getUserCount = function () {
    return new Promise (function (resolve, reject) {
    db.users.count().then(function (result) {
        resolve (result);
    }, function(e) {
        reject (e);
    });
});

我的所有数据库查询都按照上面的代码进行了编码。这是我不使用反模式的尝试，但由于我不完全理解它，我认为这种方式也不正确：

db.getUserCount = function () {
    return new Promise (function (resolve, reject) {
        db.users.count().then(function (result) {
            return (result);
        }).then(function(result) {
            resolve(result);
        }).catch(function(err) {
            reject(err);
        });
    });
};

由于我想编写好的代码，如果有人可以花一点时间向我展示一个不使用反模式而使用上述功能的示例，那就太好了。

Answer 1

由于db.users.count() 似乎已经返回了一个承诺，你可以这样做：

db.getUserCount = function () {
    return db.users.count();
});

无需将其包装在额外的承诺中（使用new Promise(...)）。

Answer 2

如果db.users.count() 函数是异步操作，那么你写的不是反模式。 “反模式”在方法同步时出现，并且为了使用 Promise 模式而返回一个 Promise。

这里的关键来自另一个Stack Overflow question 的引用，链接到您引用的问题：

...承诺是关于使异步代码保留同步代码的大部分丢失属性，例如扁平缩进和一个异常通道。

（强调，我的）

如果代码不是异步的，那么返回一个promise就是一种反模式。

至于返回new Promise(...) 是一种反模式？如果是这种情况，那么返回 new Anything() 是一种反模式。使用 Promise 构造函数很好。