jQuery(ajax)にて新規メッセージを取得するコードを書いていますが、うまく動きません。誰か助けてください。

Question

RailsにてjQueryを用いたアプリを作っています。
setIntervalを使いajax通信をし、
別の人が新規メッセージを投稿していた場合、現在表示されているメッセージとの差分を比較し、
新規メッセージがある場合はアラーム（**件の新着メッセージを表示）を出し、それをクリックするとメッセージが実際に追加される仕組みです。

ajax通信、for文などはうまく動いているのですが、肝心の新着メッセージを追加する、という機能がうまくいっていません。
二重に投稿されるなど、ajax通信をすればするほど同じ投稿が繰り返される、といった感じです。

一度クリックして新着メッセージを表示しても次にまた新着があり、クリックした時に一つ前のメッセージが追加されるなど、ajax通信を行ってからif文、for文を抜けるまでに何か問題があると思います。

どなたか力を貸していただけるとありがたいです。

$(function(){

    setInterval(function(){
    reload();
  },5000);

  function reload(){

    $.ajax({
    type: 'GET',
    url: '/messages.json',
    dataType: 'json'
  }).done(function(data) {
    var all_message_num = $('.message-list').length;

    if (all_message_num !== data.length){
      var data_num = data.length;
      var suuzi = data.length - all_message_num;

      var htm =
        '<div class="message-alarm">' +
          suuzi +
        '件の新着メッセージを表示' +
        '</div>';
      $('.message-alarm-div').html(htm);
    };
    click(suuzi,data);
    });
    function click(suuzi,data){
        $('.message-alarm-wrapper').on('click', function(){
        for (var i = suuzi; i >0; i--){
          var num = i;
          var numm = num -1;
        var html_tag = buildNewMessage(data[numm]);
        $('.message-lists').prepend(html_tag);
        $('.message-alarm-div').empty(html_tag);
        }
      });
      };
  };

Answer 1

とりあえず、コンパイルエラーを指摘しますと

        $('.message-alarm-div').empty(html_tag);
        }
      });
      }
  }
});

と、部分が変ですね。
あとは、jsonの内容と。HTML部分がないと動作の確認は出来ないと思いますよ。

Comments

[dongw]

回答ありがとうございます！
問題がどうやらif文の括弧の位置の問題だったりして根本から問題ありだったので一度検討しなおしてみます。