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

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

ただいまの
回答率

91.00%

  • JavaScript

    13835questions

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

  • Monaca

    862questions

    「Monaca」はiOS、Android、Windows向けのアプリ開発に対応した、Cordovaベースのモバイルアプリ開発プラットフォームです。HTML5、JavaScriptといったWeb標準技術を用いてモバイルアプリ開発を行うことができます。

値がundefindになる時がある

受付中

回答 1

投稿 編集

  • 評価
  • クリップ 1
  • VIEW 150

matsuo

score 70

いつもお世話になっております。
monacaとニフクラ mobile backendを使ってアプリを作成しているのですが、取得した値(place_id)がundefindになる時があります。
私のソースコードがいけないのか、monacaがいけないのか判断がつかず…
以下が当該のソースコードです。
事前にDBにplace_idが1~5の場所が登録してあり、ランダムで場所を取得し、取得済み(visited_place_id)として別のテーブルに(配列で)保存するという挙動になります。

<!DOCTYPE HTML>
<html>
<head>
  <meta charset="utf-8">
  <meta name="viewport" content="width=device-width, initial-scale=1, maximum-scale=1, user-scalable=no">
  <meta http-equiv="Content-Security-Policy" content="default-src * data:; style-src * 'unsafe-inline'; script-src * 'unsafe-inline' 'unsafe-eval'">
  <link rel="stylesheet" href="components/loader.css">
  <link rel="stylesheet" href="css/style.css">
  <script src="components/loader.js"></script>
  <script src="js/ncmb-2.1.3.min.js"></script>
  <script src="js/custom.js"></script>
  <script>
    $(function()
    {
    //mBaaSの初期化
      var application_key = "XXXXXXX";
      var client_key = "XXXXXXX";
      var ncmb = new NCMB(application_key, client_key);
      var place_id;
      var user_obj_id;
      var visited_place_id;

        $('#save_id').on('click', function()
        {
          //userのobjectIdを取得する
          var user = ncmb.User.getCurrentUser();
          user_obj_id = user["objectId"];

          //場所のidを取得する
          var Coordinate = ncmb.DataStore("Coordinate");
          Coordinate.fetchAll()
            .then(function(results){
              place_id = results[Math.floor(Math.random() * 4)].place_id;
              console.log("place_id_1 : " + place_id);
              return place_id;
            })
            .catch(function(err){
              alert("place_idとれなかったよ");
            });

          //visited_place_idにplace_idを追加する
          var visited = ncmb.DataStore("Visited");
          visited.equalTo("user_objectId", user_obj_id)
            .fetchAll()
            .then(function(results){
              var visited_place_id = results[0].visited_place_id;
              visited_place_id.push(place_id);
              results[0].set("visited_place_id", visited_place_id)
              .update()
              .then(function()
              {
                alert("更新完了!");
                location.href = "./visited_list.html";
              })
              .catch(function(err)
              {
                alert("送信できんかった\n" + err);
              })
              return false;
            })
            .catch(function(err){
              console.log(err);
            });
          return false;
        });


    });
    </script>
</head>
<body>
    <p>
      ここはhome.htmlです
    </p>
    <form>
      <input type="submit" id="save_id" value="idを保存する!" />
      <!-- <input type="submit" id="logout" value="ログアウト" /> -->
    </form>
</body>
</html>

初歩的な質問で申し訳ないですが、ご教授いただければと思います。
どうぞよろしくお願い致します。

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

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

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

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 1

+2

console.log("place_id_1 : " + place_id); でundefindならresultsが自分の予想している内容と違うか確認してみてください。

visited_place_id.push(place_id); の部分であれば、.fetchAll() は非同期で動くため、Coordinate.fetchAll() の then の中で visited.equalTo を実行してみては。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/01/18 04:39

    いつもありがとうございますm(._.)m
    place_id_1は毎回問題なくとれるので、Coordinate.fetchAll() の then の中に visited.equalTo以下の処理を移したところ、ページ遷移しなくなってしまいました><

    キャンセル

  • 2018/01/18 04:44

    var visited = ncmb.DataStore("Visited");から入れていますか?
    同じ環境を再現できないので状況はわかりませんが、非同期処理部分の問題であることは確実ですね。

    キャンセル

  • 2018/01/18 04:47

    そうです!
    エラーも特に出ていないです。非同期処理の中に非同期処理を入れたらダメってことなんでしょうか…

    キャンセル

  • 2018/01/18 08:57

    修正したコードを質問文に「追記」してください。(決して元のコードを消さないでください)

    キャンセル

  • 2018/01/18 10:32

    console.log("place_id_1 : " + place_id);
    プラスでStringに変換かけずに、第二引数にした方がわかりやすいのでよく利用しています。
    console.log("place_id_1 :", place_id);

    キャンセル

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

  • ただいまの回答率 91.00%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る

  • JavaScript

    13835questions

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

  • Monaca

    862questions

    「Monaca」はiOS、Android、Windows向けのアプリ開発に対応した、Cordovaベースのモバイルアプリ開発プラットフォームです。HTML5、JavaScriptといったWeb標準技術を用いてモバイルアプリ開発を行うことができます。