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

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

ただいまの
回答率

88.80%

ruby on rails で画像が2枚以上投稿されてしまうバグ

受付中

回答 1

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 666

makao1987

score 4

前提・実現したいこと chat-spaceというアプリの自動更新で画像が二枚投稿されてしまうバグを直したい

ここに質問の内容を詳しく書いてください。

初めての投稿です。よろしくお願いします。
chat-spaceというテキストと画像を投稿できるラインのようなアプリをエンジニアスクールの
課題で作っております。
使用言語はruby,ruby on rails,JavaScript(JQuery),
haml,scss,mysql、AWSです。

発生している問題・エラーメッセージ

JQweyで自動更新機能、インクリメンタルサーチ、非同期通信を実装してます。
今回のエラーは画像を投稿すると同じ写真が2枚もしくは3枚投稿されてしまうことです。
一枚だけちゃんと投稿される時もあり、少ない頻度で2枚以上投稿されてしまいます。

エラーメッセージ ありません。

該当のソースコード

app/assets/javascripts/message.js
$(function(){ 
  function buildHTML(message){
    var content = message.content ? `${ message.content }` : "";
    var imge = message.image ? `<img src= "${ message.image }">` : "";
      var html = `<div class="message" data-message-id="${message.id}">
                  <div class="upper-message">
                    <div class="upper-message__user-name">
                      ${message.user_name}
                    </div>
                    <div class="upper-message__date">
                      ${message.date}
                    </div>
                  </div>
                  <div class = "lower-message">
                    <p class="lower-message__content">
                      ${content}
                    </p>
                  </div>
                    ${imge}
                </div>`
      return html;
  }

  $('#new_message').on('submit', function(e){
  e.preventDefault();
  var formData = new FormData(this);
  var url = $(this).attr('action')
  $.ajax({
    url: url,
    type: "POST",
    data: formData,
    dataType: 'json',
    processData: false,
    contentType: false
  })
    .done(function(data){
      var html = buildHTML(data);
      $('.messages').append(html);
      $('.messages').animate({scrollTop: $('.messages')[0].scrollHeight}, 'fast');
      $('form')[0].reset();
    })
    .fail(function(){
      alert('error');
      });
    return false;
    });


var reloadMessages = function () {
  if (window.location.href.match(/\/groups\/\d+\/messages/)){
    var last_message_id = $('.message:last').data("message-id"); 
            $.ajax({ 
              url: "api/messages", 
              type: 'get', 
              dataType: 'json', 
              data: {id: last_message_id} 

            })
            .done(function(messages){ 
              messages.forEach(function (message) {
                var insertHTML = buildHTML(message); 
                $('.messages').append(insertHTML);
                $('.messages').animate({scrollTop: $('.messages')[0].scrollHeight}, 'fast');
              })
            })
            .fail(function () {
              alert('自動更新に失敗しました');
            });
          }
        };
        setInterval(reloadMessages, 10000);
});


app/views/messages/create.json.jbuilder

json.id @message.id
json.user_name @message.user.name
json.date @message.created_at.strftime("%Y年%m月%d日 %H時%M分")
json.content @message.content
json.image @message.image.url


app/views/api/messages/index.json.jbuilder

json.array! @messages do |message|
  json.content      message.content
  json.image        message.image.url
  json.date         message.created_at.strftime("%Y/%m/%d %H:%M")
  json.user_name    message.user.name
  json.id           message.id
end 

試したこと

画像を一枚投稿したいのに2枚以上投稿されてしまう。バグです。
エラーメッセージや、検証でコンソールを確認してもエラーはでません。
同じ画像が投稿されてしまうのですが、日付の表記が違うことから、
app/views/messages/create.json.jbuilderから投稿されている画像と
app/views/api/messages/index.json.jbuilderから投稿されている画像と判断できます。

自動更新機能と投稿した瞬間が被ってしまい、複数投稿されてしまうバグが発生しているのだと考えています。
対策としてsetInterval(reloadMessages, 10000);の数値を増やして、自動更新の頻度をへらしてみたりしたのですが、完全になくすができません。
協力お願いします。

補足情報(FW/ツールのバージョンなど)

ここにより詳細な情報を記載してください。

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 1

+1

turbolinksの影響が考えられるので、$(function(){ を$(document).on('turbolinks:load', function() {に変更してはどうでしょうか?

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2019/10/24 13:51

    回答ありがとうございます。記事をよくよんで試してみます。JSのことをよくわかっていないので時間がかかると思いますが、やってみます!!ありがとうございます

    キャンセル

  • 2019/10/24 13:58

    やり方だけだと下記となるようです。※やったことがないので情報提供となります。
    https://stackoverflow.com/questions/54806006/turbolinks-5-and-setinterval-issue/54829449#54829449

    キャンセル

  • 2019/10/24 17:32

    度々ありがとうございます。参考にさせていただきます。

    キャンセル

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

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

関連した質問

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