質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.48%
MongoDB

MongoDBはオープンソースのドキュメント指向データベースの1つです。高性能で、多くのリトルエンディアンシステムを利用することができます。

Node.js

Node.jsとはGoogleのV8 JavaScriptエンジンを使用しているサーバーサイドのイベント駆動型プログラムです。

JavaScript

JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。

Q&A

1回答

2154閲覧

mongooseのfindの中でfor文を使い、さらにfindをすることができません。

退会済みユーザー

退会済みユーザー

総合スコア0

MongoDB

MongoDBはオープンソースのドキュメント指向データベースの1つです。高性能で、多くのリトルエンディアンシステムを利用することができます。

Node.js

Node.jsとはGoogleのV8 JavaScriptエンジンを使用しているサーバーサイドのイベント駆動型プログラムです。

JavaScript

JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。

0グッド

0クリップ

投稿2015/11/19 10:30

###このエラーをどのようにしたらよいでしょうか?

javascript

1 2collection1.find({id: data.id }, function (err, docs1){ 3 for(var i = 0; i < docs1.length; i++){ // docs1.length = 3 4 // i = 0 , 1 , 2 5 console.log("aaa"); 6 collection2.find({ itemId: docs1[i].itemId }, function (err, docs2) { 7 // i= 3 , 3 , 3 8 console.log("bbb"); 9 }) 10 // i = 0 , 1 , 2 11 console.log("ccc"); 12 } 13 14}) 15

aaa
bbb
ccc
aaa
bbb
ccc
aaa
bbb
ccc
としたいですが、

上記のようなコードを書くと、

aaa (i = 0)
ccc (i = 0)
aaa (i = 1)
ccc (i = 1)
aaa (i = 2)
ccc (i = 2)
bbb (i = 3)
bbb (i = 3)
bbb (i = 3)

となってしまいます。

collection2の中でもiを0,1,2としたいのですが、おそらく非同期のせい?でうまくいきません。
誰かご存知の方がいらっしゃいましたら、ご教授お願いします。

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

think49

2015/11/19 13:05

回答サイドの環境で再現できるように collection1 の内部コード、HTMLを開示して実行可能な形式にしてください。 http://jsfiddle.net/ に最小コードをUPして頂くと大変助かります。
guest

回答1

0

仰るとおり非同期のため起きています。
jsはライブラリを使わないかぎり、同期的に処理を行いたい場合callbackを続けなければなりません。
なのでcccの処理はbbbの後に行えば意図通りの順番になるかと思います。
iが3になってしまっているのは、collection2のcallback内でconsole.logするタイミングよりも早くfor loopが終わってしまっているためかと思います。
こちらはスコープの問題とも言えそうです。
なので、iを利用したいのであれば、下記のように一つの手続きに切り出すとうまくいくのでは無いかと思います。

javascript

1var b = function(i) { 2 collection2.find({ itemId: docs1[i].itemId }, function (err, docs2) { 3 console.log("bbb (i = " + i + ")"); 4 }) 5}

javascript

1collection1.find({id: data.id }, function (err, docs1){ 2 for(var i = 0; i < docs1.length; i++){ // docs1.length = 3 3 // i = 0 , 1 , 2 4 console.log("aaa"); 5 collection2.find({ itemId: docs1[i].itemId }, function (err, docs2) { 6 // i= 3 , 3 , 3 7 // ここのiはfor loopが先に終わってしまっているため3になっていますが、docs2の値はiが1,2,3のものがそれぞれ返ってきているのでは・・・? 8 // 検証していないので合ってなかったらごめんなさい。 9 console.log("bbb"); 10 console.log("ccc"); 11 // bbbの処理の後に続けてcccの処理を行いたいのであれば、callback内に書く必要があります。 12 // cccの処理でもiの値を使いたい場合は上に書いたようにbbbの処理を一つの手続きに書きだすのが良いかと思います。 13 // callback地獄を避けたい場合、回避するためのライブラリもあったりします。(後述) 14 }) 15 } 16 17}) 18

同期的に処理を行うライブラリもあったりします。
asyncライブラリ

投稿2015/12/10 13:24

ara_ta3

総合スコア82

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

まだベストアンサーが選ばれていません

会員登録して回答してみよう

アカウントをお持ちの方は

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.48%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問