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

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

新規登録して質問してみよう
ただいま回答率
85.50%
Vue.js

Vue.jsは、Webアプリケーションのインターフェースを構築するためのオープンソースJavaScriptフレームワークです。

Firebase

Firebaseは、Googleが提供するBasSサービスの一つ。リアルタイム通知可能、並びにアクセス制御ができるオブジェクトデータベース機能を備えます。さらに認証機能、アプリケーションのログ解析機能などの利用も可能です。

JavaScript

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

Q&A

解決済

2回答

1906閲覧

戻り値がプロミス型のものをループで複数回実行する方法

hellomark

総合スコア14

Vue.js

Vue.jsは、Webアプリケーションのインターフェースを構築するためのオープンソースJavaScriptフレームワークです。

Firebase

Firebaseは、Googleが提供するBasSサービスの一つ。リアルタイム通知可能、並びにアクセス制御ができるオブジェクトデータベース機能を備えます。さらに認証機能、アプリケーションのログ解析機能などの利用も可能です。

JavaScript

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

0グッド

0クリップ

投稿2017/04/17 02:48

編集2017/04/17 03:06

Firebase,Vue js,Vuexを使用しアプリケーションを作成しています。

Firebase realtime database からデータをループにて取得しようと試みています。
shopsDataにはデータが3つ保存されているのに、ループは一度しか実行されません。
Promise型で返ってくるものをループで回すことはできますでしょうか??
また、それは現実的ですか??

javascript

1for (var shop_uid in shopsData) { 2 //shop info 3 return db.ref('/shops/' + shop_uid).once('value').then(snapshot => { 4 if (snapshot.val()) { 5 commit(types.SET_SHOP, { 6 shopData: snapshot.val(), 7 uid: shop_uid 8 }) 9 } 10 }) 11}

↓javascritpそと側

javascript

1 return db.ref('/orgs/' + snapshot.val().org).once('value').then(snapshot => { 2 commit(types.SET_ORG, snapshot.val()) 3 let shopsData = snapshot.val().shops 4 5 return db.ref('/shops/' + currentShop).once('value').then(snapshot => { 6 commit(types.SET_CURRENTSHOP, snapshot.val()) 7 8 //assignStepsMaster all shop info 9 return db.ref('/paymentMethodMaster/' + currentShop).once('value').then(snapshot => { 10 commit(types.SET_PAYMENT_METHOD_MASTER, snapshot.val()) 11 12 return db.ref('/deliveryMethodMaster/' + currentShop).once('value').then(snapshot => { 13 commit(types.SET_DELIVERY_METHOD_MASTER, snapshot.val()) 14 console.log(shopsData); 15 16 for (var shop_uid in shopsData) { 17 //shop info 18 return db.ref('/shops/' + shop_uid).once('value').then(snapshot => { 19 if (snapshot.val()) { 20 commit(types.SET_SHOP, { 21 shopData: snapshot.val(), 22 uid: shop_uid 23 }) 24 } 25 }) 26 } 27 }) 28 }) 29 }) 30 })

shopsデータの中身
イメージ説明

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

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

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

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

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

akabee

2017/04/17 02:55

まず、Promiseでもループはできます。shopsDataがfor文に入った時点で3つ確実にあれば、ループは3度実行されると思います。shopsDataがどのように格納されるのか(要するにこのfor文の外側)もご記載いただけませんか?
hellomark

2017/04/17 03:01

プロミスの外にもプロミスがあり、入れ子になっております。
hellomark

2017/04/17 03:03

追記させていただきました。
akabee

2017/04/17 03:04 編集

であれば、外側のプロミスでshopsDataが1つしかないうちに質問文のfor文が実行されてしまっているためにループが1つしか実行されないという事象に陥っていると思います。やり方としてはshopsDataが全て揃った段階で次を実行するという、ある意味「非同期処理の同期処理」を行わなければならないのですが、これもコードを工夫すれば可能です。外側のコードが提示できれば、回答者側で意図通り動くようにしてみることも可能かと思います。
akabee

2017/04/17 03:05

追記ありがとうございます。確認してみます
hellomark

2017/04/17 03:07

一部更に追記したしました。ありがとうございます。
akabee

2017/04/17 03:31 編集

他の回答者さんからの回答が正しそうですね。すみません、間違ったご指摘をしていたと思いますので他の回答者さんのご指摘を確認してみて下さい。
guest

回答2

0

ベストアンサー

たとえば、

javascript

1function test(){ 2 for (var i=0; i<5; i++) return i; 3} 4console.log(test());

↑これは、0が返ってきて終わりです。
関数の中で、return文が評価されると、それ以降の式や分は評価されずに関数を出ます。
なので、forループで何度もreturnしても思うような動作にならないです。

ご提示のライブラリは使ったことがないですが、Promise.all()を使うのがいいのではないでしょうか。

Promise.all(iterable)メソッドは、反復引数のすべてのプロミスが完了したとき完了したプロミスを返すか、与えられたプロミスの中で最初の失敗理由とともに失敗したプロミスを返します。

Promise.all() - JavaScript | MDN

投稿2017/04/17 03:05

Lhankor_Mhy

総合スコア35865

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

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

hellomark

2017/04/17 14:33

Promise.all使用したらできました。 いや〜非同期処理はむずかしい。。。
guest

0

やりたいことをよく理解していないかもしれませんが、こうしてはどうでしょうか。

JavaScript

1var ret = []; // 空の配列を用意 2for (var shop_uid in shopsData) { 3 //shop info 4 var data = db.ref('/shops/' + shop_uid).once('value').then(snapshot => { 5 if (snapshot.val()) { 6 commit(types.SET_SHOP, { 7 shopData: snapshot.val(), 8 uid: shop_uid 9 }) 10 } 11 }); 12 ret.push(data); // 配列にプロミス型の戻り値を追加 13} 14return ret; // forで得られたプロミス型を全部まとめて配列で返す

投稿2017/04/17 02:54

masaya_ohashi

総合スコア9206

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

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

hellomark

2017/04/17 14:33

ヒントをいただきました!ありがとうございます!
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.50%

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

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

質問する

関連した質問