Ajaxのデータの取得について

Question

#実現したいこと
一定のタイミングでAjaxを使い、サーバーにアクセスしてview側に表示しているデータの差分を比較し、不足分だけを取得し、表示するようにしている。

#困っていること
var data_count = $('.chat_count').length;
上記のコードで要素数をカウントしているが、Ajaxで挿入した要素数がカウントされなくて困っている。
どのようにしたら、ajaxで出力した要素数がカウントされるかご教示いただけると幸いです。

$(function() {
  var POLLLING_INVERVAL_TIME_IN_MILLIS = 10000;//10s
  (function polling() {
    if(!document.hidden) {
        getCountUp();
    }
    window.setTimeout(polling, POLLLING_INVERVAL_TIME_IN_MILLIS);
  }());


  function getCountUp() {
    var room_id = document.getElementById("room_id").value;
    //ここで、要素の数を取得している。要素の数とサーバーのレコード数を比較して不足分だけ取得するようにしている。
    var data_count = $('.chat_count').length;
    console.log(data_count);
    if(data_count > 0){
      $.ajax({
          url: `/rooms/${room_id}/get_data_ajax/?count=${data_count}`,
          type: "GET",
          data: {memos: 'text'},
          dataType: "json",
          success: function (data) {
            for(var i=0;i<data.length;i++){
              console.log(data[i].text);
              $('#chat_areat').append('<div class="row data_count"><div class="col py-2 border-bottom"><div class="user-name">' + `${data[i].text}` + '</div><div class="content">' + `${data[i].text}` + '</div></div></div>');
            }
          },
          error: function (data) {
              console.log("error");
          }
      });
    }
  }
});

Comments

[yambejp]

DOMの処理が通常のjsとjQueryを雑多につかっていてわかりにくくないですか？
どちらかに寄せたほうが管理しやすいと思います

[nanase21]

ご指摘いただきありがとうございます。
仰る通り、jsかjqueryのどちらかに統一すべきなんですが、jsですべて書くのが難しく分からなくなるとついついjqueryを使ってしまうので可読性の悪いコードだと私自身も思います。

[yambejp]

ブラウザを限定できるなら$.ajaxではなくfetchでの処理が楽です
とくにfetchはpromiseを返してくれるので非同期処理を最適化できます

[nanase21]

ご教示いただきありがとうございます。
fetchについて調べさせていただきました。
jsにすべて書き直す際に、fetchを使用してみたいと思います。
それとお聞きしたいことがあるのですが、リアルタイムのオープンチャットをwebsoketを使用せずに実装しようとしているのですが、非同期通信(ロングポーリング)で実装するのはどのように思いますでしょうか？

Answer 1

こんにちは

ぱっと見で、「ここが怪しいのでは？」と思った点を回答します。

ご質問に

var data_count = $('.chat_count').length;
上記のコードで要素数をカウントしているが、

とあり、上記ではクラスにchat_count を持つ要素の数を数えていますが、ajax で取得してきた内容を append する行は、以下

$('#chat_areat').append('<div class="row data_count"><div class="col py-2 border-bottom"><div class="user-name">' + `${data[i].text}` + '</div><div class="content">' + `${data[i].text}` + '</div></div></div>');

となっており、 append される対象の一番外側の <div> のクラスは row data_count となっているほか、その中の子孫の要素のどれにも、var data_count = $('.chat_count').length;で数え上げの対象となっているchat_countというクラスを持つ要素は出てきていません。

もしこれが単純なミスでしたら、以下のように修正すれば意図通りになるかもしれません。

修正前

var data_count = $('.chat_count').length;

修正後

var data_count = $('.data_count').length;

以上参考になれば幸いです。

Comments

[nanase21]

ご教示いただきありがとうございます。
ご指摘いただいた通り,ajaxで出力するクラス名がchat_countからdata_countになっていたため意図した処理になっていませんでした。

[jun68ykt]

どういたしまして。解決したようで、よかったです????

Answer 2

    var room_id = document.getElementById("room_id").value;
    //ここで、要素の数を取得している。要素の数とサーバーのレコード数を比較して不足分だけ取得するようにしている。
    var data_count = $('.chat_count').length;
    console.log(data_count);
    if(data_count > 0){
      $.ajax({
          url: `/rooms/${room_id}/get_data_ajax/?count=${data_count}`,

この辺りのコードを見る限りで推測するに、

`/rooms/${room_id}/get_data_ajax/?count=${data_count}`

こことか、" か ' で囲まなくてはならないところを ` になっているし、

変数 room_id なのに ${room_id} とか、JavaScript と PHP を混同しているのかとも思える。

いずれにしても、デベロッパーツールを見ればエラーメッセージが表示されているだろうと思います。

Comments

[yasutomi]

テンプレート文字列くらいは知らないとダメです。
https://developer.mozilla.org/ja/docs/Web/JavaScript/Reference/template_strings

[退会済みユーザー]

あらま、ありがとうございます！
素直に知らんかった・・・