Question

我是承诺的新手，我试图回复像mongoose这样的承诺的价值但是使用mongoskin和bluebird。这适用于mongoose return User.find().then(users => users)。这将返回一个用户列表，而不是apollo-server解析器中的promise。

我尝试过承诺生成器和异步，但没有运气。从我所读到的，承诺总是返回一个承诺，所以不知道mongoose如何返回一个值。

mongodb.js

import Promise from 'bluebird';
import mongoskin from 'mongoskin';

Object.keys(mongoskin).forEach(function (key) {
  var value = mongoskin[key];
  if (typeof value === 'function') {
    Promise.promisifyAll(value);
    Promise.promisifyAll(value.prototype);
  }
});

Promise.promisifyAll(mongoskin);

export default {
  connect (uri) {
    return mongoskin.db(uri, {native_parser:true});
  }
};

users.js

import mongodb from '../../databases/mongodb';

export default class User {
  constructor () {
    this.db = mongodb.connect('mongodb://127.0.0.1:27017/test', {native_parser:true});
    this.collection = this.db.collection('users');
  }

  find (query = {}, options = {}) {
    const findAsync = () => {
      return Promise.resolve().then(() => {
        return this.collection.findAsync(query, options);
     })
     .then((xx) => {
        xx.toArray((err, items) => {
         if (err) return err;
        return items;
       });
     });

    };

    async function getData () {
      let foo = await findAsync();

      return foo;
    }

    return getData();
  }
}

const user = new User();

function bar () {
  return user.find().then(x => console.log(x));
}

console.log(bar());

Answer 1

您的代码似乎过于复杂。我想你想要的是这个（我没有宣传mongoskin，因为它不太适合那个;见下文）：

export default class User {
  constructor() {
    this.db         = mongoskin.connect(...);
    this.collection = this.db.collection('users');
  }

  find (query = {}, options = {}) {
    return new Promise((resolve, reject) => {
      this.collection.find(query, options).toArray((err, items) => {
        if (err) return reject(err);
        resolve(items);
      });
    });
  }
}

我不得不说Mongoskin感觉已经过时了。它是官方mongodb驱动程序之上的一个皮肤，现在相当不错（它支持开箱即用的承诺，一个是Mongoskin不会传播的东西）。

如果你想宣传，那么我应该说以下是一个承诺反模式：

return Promise.resolve().then(() => {
  return this.collection.findAsync(query, options);
}).then(...)

您可以将其重写为：

return this.collection.findAsync(query, options).then(...);

然而，toArray()再次使事情变得困难，因为为此你做需要创建一个新的承诺，所以代码变成这样：

return this.collection.findAsync(query, options).then((cursor) => {
  return new Promise((resolve, reject) => {
    cursor.toArray((err, items) => {
      if (err) return reject(err);
      resolve(items);
    });
  });
});

这看起来并不是很漂亮，因此我选择不进行promisify并且在这种情况下只使用回调（虽然我确信Bluebird有一些很好的工具，可能上面的代码更容易看，但仍然...）。

承诺的回报价值

1 个答案: