Mongoose: どのES6がライブラリマングースの使用を約束するかを指定する機能

Promise.all（）を使用して、すべてのデータベース作業が完了した後に何かを実行できることを本当に楽しみにしています。

：+1：

https://github.com/LearnBoost/mongoose/issues/1699であまり明確ではなかったことの1つは、どの実装がデフォルトになるかということです。

逆方向のリリースではないことを考えると、mpromiseをデフォルトにする必要がありますが、オーバーライドすることはできます。

+1

：+1：

これがハッキングされているブランチはありますか？ 私はmpromiseでエラー処理フローに耐えられず、ソースをクラックして開いて、これが何をするかを確認しようとしています。

mpromise：

  query.exec()
    .then(function(ou) {
      if(!ou) {
        return next(new errors.http.NotFound('The specified OU was either not found, or your credentials lack the required permissions to view it.'));
      }

      res.send(ou);
    }, next)
    .end(next);

拒否を処理し、そこにend配置する必要があります。 endがないと、例外（たとえば、NotFoundクラスのスペルを間違えた）は黙って飲み込まれ、ストールだけを表現します。

青い鳥：

  query.exec()
    .then(function(ou) {
      if(!ou) {
        return next(new errors.http.NotFound('The specified OU was either not found, or your credentials lack the required permissions to view it.'));
      }

      res.send(ou);
    })
    .catch(next);

promisifyAll(require('mongoose'))を実行すると、これまでのところ、実際にはまだMongoose4で機能しているようです。 これをカバーする回帰テストはあまりにもターゲットにされますか？

現時点ではない。 ほとんどの作業は、＃2754およびvkarpov15 / kareem＃2に従って、 kareemモジュールで行われます。これにより、1つの石で2羽の鳥を殺し、フックと約束を作成するために作成された非常に厄介なポイントカットのナンセンスを削除できます。共に働く。 でも、気軽に試してみてください。私はPRを受け入れています。

しかし、なぜ他のPromiseライブラリのサポートを維持する必要があるのでしょうか。 ES6 Promisesの仕様は今や堅実であり、ここにとどまります。 古いバージョンのノードで使用できないときにポリフィルがロードされた、純粋なES6 Promisesを使用することはできませんか？

もしそうなら、私はそれを試すことができます。

重要なのは、ES6互換のPromiseライブラリを好きなように使用できるということです。 多くの人々は、q、rsvpなどの場合、まだブルーバードに多額の投資を行っており、これらのライブラリのそれぞれには、一般的なポリフィルでは捉えられない独自の癖があります。

私は別の提案を受け入れています-私は特に約束を好きではなく、使用していません。この機能は、「ブルーバード機能Xをmpromiseでサポートする」または「ネイティブrsvp.jsサポート」を求める人々が関与する問題が山ほどあるという事実によって動機付けられています「そして、人々に自分の約束のライブラリをパーティーに持って来させることは、これらの問題を解決する最も簡単な方法です。

意味がわかります。 私はPromisesのビッグユーザーでありサポーターです。 基準を強制することを考えるべきだと思います。
ES6の頼りになるものとしてA+プロミスが選ばれました。 ES6と互換性のないノードバージョンを確実にサポートするために、ポリフィルを提案しました（例：https：//github.com/jakearchibald/es6-promise）。

ES6のPromiseと互換性があり、混合可能であるためには、他のPromiseライブラリの手にある必要があります。

編集：
そして、現在のAPIに問題はありませんか、それとも何かが足りませんか？

Promises / A +の仕様は、ES6のPromiseの仕様とは大きく異なります。これは、前のコメントでリストしたPromiseライブラリとは異なります。 無数のPromiseライブラリがすべてES6に統合されていれば良いのですが、オープンソースの美しさは、Promiseを愛する一部の人々が追加機能を必要とし、独自のPromiseを作成することであるため、これが実現するかどうかは疑問です。ライブラリ。

現在のAPIに重大な変更はありません。私が考えているのは、 mongoose.set('Promise', require('bluebird'));などの言い方なので、オプトインになり、デフォルトはmpromiseになります。

そうそう、その間違いでごめんなさい。
現在の実装と他のpromiseライブラリを確認しました。

私はこのような作品を作ることができると思います：

mongoose.set('Promise', Promise);

mongoose.set('Promise', require('bluebird'));

mongoose.set('Promise', require('q').defer());

mongoose.set('Promise', require('when').defer());

// and so on...

したがって、基本的には、 resolve ＃$メソッドとrejectメソッドを持つpromiseofchoiceオブジェクトをMongooseに公開する必要があります。

これはあなたが考えていたものでしょうか？ もしそうなら、私はプルリクエストに取り組みます。

編集：
mongoose.set('Promise', Promise);を書くのは非常にばかげています。 ES6がデフォルトであり、選択したライブラリを使用できる可能性があると思います（ES6 Promisesが利用できない場合はいつでも妥協します）。

確かに、私はあなたの助けに感謝します。 トリッキーな点は、1）フックで動作させることです。vkarpov15/ kareem＃2を参照してください。2）mpromiseとの下位互換性を確保します。

また、re：Q、ES6仕様に最も近いQ構文であるため、 require('q').Promiseを使用します

ご入力いただきありがとうございます、それに取り組みます。 競合するプルリクエストと継続的なPRのどちらが好きですか？

完了した方が良いですが、私は常にいくつかの提案があります。 行き詰まったら教えてください

この機能の+1。 ブルーバードと一緒に使いたい

抜本的なことについては申し訳ありませんが、別のアプローチは、promiseサポートを完全に廃止することです。 それは他の図書館の支援を求める約束を使っている人を黙らせるでしょう。
bluebirdのような強力なpromiseライブラリを使用している人は、コールバックAPIを公開し、bluebirdはそれを0のコストで簡単にラップできるため、とにかくそれを使い続けることができます。とにかく手動でそれらをサポートすることはより遅くなり、よりエラーが発生しやすくなります。

@benjamingr 100％同意しません。 iojsとNodeJがNode3.0にマージされると、ES6PromisesとGeneratorsがサポートされるようになります。

サポートを削除すると、大幅に後退します。

つまり、 Promise.promisifyAll(require("mongoose"))は、Mongooseの高速な（手動での試行よりも高速な）ラッパーを作成します。これは、標準に準拠しており、何もしなくてもすべてのAPIを囲みます。 実際、エクスポートオブジェクトに対してPromise.promisifyAllを自分で実行し、bluebirdpromiseとdualpromiseメソッド(save - saveAsync)を無料で公開してから、実際の作業を行わなくてもbluebirdpromiseインターフェースを公開すると言うことができます。

したがって、私はbluebird promisesを使用していますが、このすべては、2つのインターフェイスにコーディングしてコードを複雑にするのではなく、ドキュメントのセクションで修正できると思います。

@albertorestifoは、実際には何もドロップされないことを除いて。 私はpromiseが何であるか（スタックオーバーフローで1500ポイント以上と500回答：P）をよく知っており、io.js（https://github.com/など）での動作の一部についても責任を負っています。 nodejs / io.js / issues / 256）。

@ vkarpov15はプロミスを使用しないため、ライブラリでプロミスをサポートする必要はないはずです。特に、プロミスを使用するよりもメリットがないためです。 たとえそれがそれらを提供しなくても、mongooseでpromiseを簡単に使用することができます-内部でサポートを実装するmongooseは、維持するのがより多く、おそらくより遅く、よりエラーが発生しやすくなります。 @ vkarpov15はコールバックAPIの作業を継続でき、promiseユーザーはとにかく簡単なワンライナーでMongooseをpromiseでラップできます。

@benjamingrは、外部のPromiseライブラリを使用していると想定しています。 私はスペックの1つに固執しています、そしてあなたは非常によく知っています、それはラッパーを持っていません。

私はあなたの主張を理解しています。 両方をサポートすると、大きな混乱が生じます。 私は個人的にコールバックを軽蔑しているので、代わりにそれらを削除します。 たぶん、2つの別々のリポジトリが必要です。1つはジェネレータとpromiseを使用し、もう1つはコールバックを使用します。 どちらも同じ構造とAPIを可能な限り近く維持します。

@benjamingrは、外部のPromiseライブラリを使用していると想定しています。 私はスペックの1つに固執しています、そしてあなたは非常によく知っています、それはラッパーを持っていません。

遅く、デバッグが難しく、機能が豊富でない実装を使用することを選択した場合、もちろん：Pを作成することを選択しますが、それはラッパーとどのように関連していますか？ ネイティブpromise*を使用して、bluebirdのpromisifyAllに似たラッパーを作成するのは完全に簡単です。

約束が必要な場合-別のパッケージとして有効化されたバージョンのMongooseは、次の方法で実行できます。

• ステップ1、お気に入りのエディターを開くか、問題のないエディターを開きます。
• ステップ2、「 module.exports = require("bluebird").promisifyAll(require("mongoose")) 」と入力します
• ステップ3、適切なpackage.jsonファイルを作成し、npmに公開します
• ステップ4、何万ものダウンロード。

これで、「これはネイティブのPromiseを使用していない」と考えていることがわかりました。それでも、Promiseプロトタイプからthenとcatch 、およびallを除くすべてのメソッドを削除できます。 Promise raceは、同じAPIになります-または、ネイティブのPromiseをエクスポートしていることを人々に伝えることができます-それは2つのPromises / A +実装であるため、彼らは知りません、約束します ;）

（*現在promiseを作成する高速な方法がないため、ユーザーレベルで高速に記述するのは困難です。そのため、io.jsはpromisify関数自体を最終的にエクスポートする可能性があります。つまり、promiseコンストラクターを使用することで強制します。とにかく遅くなります）。

それは間違いなくまともな選択肢です。 ただし、社内で約束を守りたいと思います。それは、好むと好まざるとにかかわらず、ユーザーがマングースを使用する方法であるため、しっかりとしたテストカバレッジがあるとよいので、「これが方法です。マングースでpromiseライブラリXを使用します。」 物事を別々のモジュールに切り離すときの欠点は、「このバージョンのマングース-プロミスはマングース3.8でのみ機能し、これはマングース> = 4.1で機能する」と言うのが難しいことです。また、マングースが包括的なプロミスラッパーを壊さないようにするのは困難です。 。

それは間違いなくまともな選択肢です。 ただし、社内で約束を守りたいと思います。それは、好むと好まざるとにかかわらず、ユーザーがマングースを使用する方法であるため、しっかりとしたテストカバレッジがあるとよいでしょう。

なぜそれに対してテストカバレッジが必要/必要なのですか？ コードでpromise自体をテストする意味はありません-ライブラリにはすでにテストがあります-これは、 httpモジュールを使用しているときにテストするようなものです。

そして、マングースが包括的な約束のラッパーを破ることを避けるのは難しいです。

Bluebirdは本当にシンプルなことをします-プロトタイプを見つけて、それにAsyncサフィックスが付いたメソッドを追加します-これは本当にシンプルで実際にうまく機能します-これはMongooseを含むほとんどのライブラリで1つのライナーであり、損益分岐点ではありませんでした過去1年間に一度でも。

エラーが発生しやすい可能性のある多くのスティッチングコードを維持する必要がある理由がわかりません。エッジケースで2つのAPIを手動でサポートする必要があるのは大変な作業のように思えますが、作業を進めるにつれて問題が発生する可能性があります。 bluebirdのpromisifyAllコードを「借用」し、他のPromiseライブラリ（結局はオープンソース）で動作するように適合させますが、手動では行いません。

「このバージョンのmongoose-promisesはmongoose3.8でのみ機能し、これはmongoose> = 4.1で機能します」と、mongooseが包括的なpromiseラッパーを壊さないようにすることは困難です。

さて、約束が自動的に行われる場合に発生しなければならない重大な変更の例を教えていただけませんか。

1）ユーザーの使い方をテストしたい。

2）怠惰になり、それを避けたいのですが、私が理解しているように、他のほとんどのpromiseライブラリにはpromisifyAllに相当するものがありません。 これが、「Xpromise機能Yのサポート」が最も人気のあるマングース機能のリクエストである理由だと思います。 さらに、Bluebirdの約束を書き直すつもりはありません。マングース関数に、ネイティブに約束を返すようにするだけです。

3）約束の実装とそれをどのように使用しているかによって異なります:)

1）ユーザーの使い方をテストしたい。

ユーザーがそれらを使用する方法をテストするとはどういう意味ですか？ コールバックの例えを教えていただけますか？

2）怠惰になり、それを避けたいのですが、私が理解しているように、他のほとんどのpromiseライブラリにはpromisifyAllに相当するものがありません。 これが、「Xpromise機能Yのサポート」が最も人気のあるマングース機能のリクエストである理由だと思います。 さらに、Bluebirdの約束を書き直すつもりはありません。マングース関数に、ネイティブに約束を返すようにするだけです。

書き直す必要はありません。それを取ることができます。ジェネリック実装は遅くなるため、ジェネリックではありません。

また、広く利用可能な機能を手動で実装しないのも怠惰ではありません。 NodeJSは、エクスプレスをコアに入れていないことに怠惰ですか？ TC39は、アンダースコアをコアに入れないのが面倒ですか？ 規則（コールバック）に固執することにより、ユーザーが必要な同時実行プリミティブを使用できるようになります。

3）約束の実装とそれをどのように使用しているかによって異なります:)

ええと、bluebirdの約束、またはQまたはWhenは、実装は異なりますが、すべて同じことを行うので、好きなものを選んでください。 私はそれがどのように壊れるか疑問に思っています。

ここでのこの議論で私が見逃していることが1つあります。

mongooseが標準のpromiseを返す場合（標準ではES6ネイティブ実装を意味します）、ES6互換のPromiseライブラリと互換性があるべきではありませんか？ Promise.all([model.query().exec(), ...])は問題なく実行でき、同等の場合はブルーバードqも実行できます。

では、コールバックと標準のPromiseを返して（現在のように、mpromiseを「取り除く」）、ユーザーにお気に入りのPromiseライブラリを使用させてみませんか？ それとも私はここで何かが欠けていますか？

@albertorestifoええと、ネイティブの約束は現時点では遅く、ユーザーランドライブラリと同等になるまでに数年かかる可能性があります。

@benjamingrあなたはいくつかの良い点を作ります。 mongooseの約束の主なポイントは、他のライブラリなしでmongoose非同期操作でyieldを使用できるようにすることです。そのため、当面の間、約束を守っています。 IMOは、今後マングースコアの一部となるはずです。

Qまたはいつ約束能力がありますか？

Qまたはいつ約束能力がありますか？

はい、事実上、私が知っている広く普及しているすべてのPromiseライブラリは、ある種の約束を提供しています。

いつです： https ：//github.com/cujojs/when/blob/master/docs/api.md#nodeliftall
Qは次のとおりです： https ：//github.com/kriskowal/q/wiki/API-Reference#qnfbindnodefunc -args

ネイティブpromiseにはまだありませんが、これは現在取り組んでいるものです-promiseを作成するための高速パス（つまり、promiseコンストラクターではない）が存在すると、NodeJSはネイティブpromiseのコアでそれをサポートする可能性があります（ユーザーランドで_高速_実行されます）。

マングースの約束の主なポイントは、他のライブラリなしでマングース非同期操作でyieldを使用できるようにすることです

とにかくpromiseで意味のある方法でyieldを使用するには、ライブラリが必要です。 ジェネレーターを非同期関数としてポンピングする9LoCを自分で作成できる場合は、間違いなくpromisifyを作成できます。ほとんどのユーザーと同じように、とにかくそのためのライブラリを使用します。

私は間違いなくマングースで約束を許可したい/必要があると思います、それらは前進の道であり、言語が現在どのように並行性を持っているかですが、私は正直にそれを方法ごとに手動で行うのは苦痛だと思います。 ドキュメントの「promiseでの使用」または「ジェネレータでの使用」セクションで、ライブラリを使用してどのように行われるかを示すだけでよい場合があります。

問題は、すでにメソッドごとに実行していることです。内部のPromiseラッパーを変更するだけです。 いずれにせよ、5.0までコアからpromiseを完全に削除することはできません。 私はそれを廃止するという考えに従順です。 @benjamingrは私が綿密に検討しなければならないいくつかの良い議論をしますが、次のステップはとにかく＃2688になると思います。

あなたは明らかにマングースの経験が豊富で、さらに重要なのはユーザーです。その選択を理解しています。 ご連絡いただきありがとうございます。

「its」と入力しましたが、iPhoneは「it's」に自動修正する必要があると感じ、GitHubはコメントの編集は興味深いユースケースではないと感じました。 ダブルコメントスパムでごめんなさい:)

コメントの右上にある鉛筆をクリックすると、Webサイトのコメントを編集できます。

私は常に心のこもった議論、特に何か新しいことを教えてくれる議論を開いています：ビール：後でもっと議論するためにあなたにpingするかもしれません:)

未来はネイティブの約束に属しているようです。 Mongooseは、mongo opsの業界標準であり、標準の約束に依存することも完全に有効です。

標準的な約束は必然的に成熟し、どのフレームワークよりも広く普及するでしょう。 それらを使用するための謙虚な+1。

@iliakanおそらく将来。 ネイティブプロミスが早くても2016年半ばまで「標準」になるとは思いません。断片化されたさまざまなプロミスライブラリは深く根付いており、微妙な癖が多すぎます。 いずれにせよ、マングースは、大幅な後方破壊の変更なしに「デフォルトでネイティブ」に切り替えることはできません。

@vkarpov15確かに理解しています。

その間に有益である可能性があるもの-妥協とネイティブの約束の間の主要な非互換性を概説する簡単なページ。 少なくともmpromiseで試してはいけないこと;）

現在catchがないことは正しいと思いますか？それがすべての制限です。

わからない、ES6APIを実際に掘り下げていない。 mpromiseはPromises/A +のみを実装します。つまり、.catch（）や長いスタックトレースなどは実装されません。基本的に、 .then()でないものは、mpromiseの実装の範囲外です。

一般的に、Promises / A +を実装するものは、高レベルの観点からES6 Promiseも実装しますが、その逆は当てはまりません。 Promises / A +は、低レベルの実装の詳細について非常に具体的です。たとえば、bluebirdはPromises / A +に正確に準拠していません。また、他の一般的なPromisesライブラリについてはわかりませんが、独自の方法。 それがこれを特にトリッキーにするものです。

つまり、ブルーバードとネイティブプロミスを使用できるようになりましたか？

うんrequire('mongoose').Promise = global.Promiseは、マングースにネイティブの約束を使用させます。 ES6のpromiseコンストラクターは使用できるはずですが、現時点では、ネイティブ、bluebird、およびQでのみテストしています。

@ vkarpov15これは素晴らしいです！ どうもありがとう！

@ vkarpov15どうもありがとう！ よくやった！

ええ、間違いなく。 かっこいい！ :)

@vkarpov15これは本当にいいです