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

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

新規登録して質問してみよう
ただいま回答率
85.50%
Ruby on Rails 5

Ruby on Rails 5は、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。

JavaScript

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

Ajax

Ajaxとは、Webブラウザ内で搭載されているJavaScriptのHTTP通信機能を使って非同期通信を利用し、インターフェイスの構築などを行う技術の総称です。XMLドキュメントを指定したURLから読み込み、画面描画やユーザの操作などと並行してサーバと非同期に通信するWebアプリケーションを実現することができます。

Q&A

解決済

2回答

601閲覧

自動更新機能実装中に、500 (Internal Server Error)

ikdnaht

総合スコア13

Ruby on Rails 5

Ruby on Rails 5は、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。

JavaScript

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

Ajax

Ajaxとは、Webブラウザ内で搭載されているJavaScriptのHTTP通信機能を使って非同期通信を利用し、インターフェイスの構築などを行う技術の総称です。XMLドキュメントを指定したURLから読み込み、画面描画やユーザの操作などと並行してサーバと非同期に通信するWebアプリケーションを実現することができます。

0グッド

1クリップ

投稿2020/01/18 08:31

前提・実現したいこと

現在マッチングしたユーザー同士が、一対一でチャットができる機能を実装しています。
送信時の非同期化には成功したのですが、自動更新機能を実装している最中に、ターミナル上で500エラーが発生しておりうまくいきません。

原因と解決法をご教授いただけると幸いです。

※messageテーブルにaddressカラム(messageの宛先ユーザーid)を付与、current_user.idとaddressによりどのユーザー同士のチャットかをビュー側で判別するようにしています。

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

GET http://localhost:3000/api/messages/24?id=133 500 (Internal Server Error)

該当のソースコード

js

1$(function() { 2 3~~新規メッセージ非同期化の記述~~ 4 5var reloadMessages = function() { 6 last_message_id = $('.message:last').data("message-id"); 7 $.ajax({ 8 url: "/api/messages/24", // 24は動作確認のため一旦仮で記述したユーザーidです 9 type: 'get', 10 dataType: 'json', 11 data: {id: last_message_id} 12 }) 13 .done(function(message) { 14 if (message.length !== 0) { 15 var insertHTML = ''; 16 $.each(messages, function(i, message) { 17 insertHTML += buildHTML(message) 18 }); 19 $('.chat_main').append(insertHTML); 20 $('.chat_main').animate({ scrollTop: $('.chat_main')[0].scrollHeight}); 21 } 22 }) 23 .fail(function() { 24 console.log("ERROR") 25 }); 26 }; 27 setInterval(reloadMessages, 5000); 28});

app/controllers/api/messages_controller

class Api::MessagesController < ApplicationController def show @user = User.find(params[:id]) last_message_id = params[:id].to_i @my_messages = current_user.messages.where(address: @user.id) @your_messages = @user.messages.where(address: current_user.id) message_groups = @my_messages + @your_messages // binding.pry messages = message_groups.where("id > #{last_message_id}") end end

app/config/routes.rb

Rails.application.routes.draw do root "events#index" devise_for :users resources :users do member do get :following, :followers end end resources :events do member do post :join end end resources :relationships, only: [:create, :destroy] resources :matching, only: [:index] resources :rooms, only: [:show] resources :messages, only: [:show, :create] namespace :api do resources :messages, only: :show, defaults: { format: 'json' } end get '/user/like', to: 'matching#like', as: :like get '/user/liked', to:'matching#liked', as: :liked end

試したこと

binding.pryを使って、データがどこまで持っていけているか確認しました。
app/controllers/api/messages_controller

class Api::MessagesController < ApplicationController def show @user = User.find(params[:id]) last_message_id = params[:id].to_i @my_messages = current_user.messages.where(address: @user.id) @your_messages = @user.messages.where(address: current_user.id) message_groups = @my_messages + @your_messages // ①binding.pryをここに記載した場合 messages = message_groups.where("id > #{last_message_id}") end end

上記の①の位置でbinding.pryした場合
message_groupsは意図した数値が取れていました。

class Api::MessagesController < ApplicationController def show @user = User.find(params[:id]) last_message_id = params[:id].to_i @my_messages = current_user.messages.where(address: @user.id) @your_messages = @user.messages.where(address: current_user.id) message_groups = @my_messages + @your_messages messages = message_groups.where("id > #{last_message_id}") // ②binding.pryをここに記載した場合 end end

上記②の位置でbinding.pryした場合は、処理が止まらず500エラーとなってしまいました。ターミナル上には以下のエラー文が表示されました。

NoMethodError (undefined method `where' for #<Array:0x00007ff83ce537d8>

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

rails 5.2.4.1
ruby 2.5.1

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

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

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

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

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

guest

回答2

0

自己解決

初歩的なな誤りでした!

messages = message_groups.where("id > #{last_message_id}")

こちら、自分で定義した「message_groups」という変数に「.where」を使っていたので、うまくいきませんでした。「.where」の前はモデルでないといけないんですね。

投稿2020/01/23 01:53

ikdnaht

総合スコア13

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

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

0

message_groups = @my_messages + @your_messages
で、messages_groups には数値が入っているわけですね?
そうしますと
messages = message_groups.where("id > #{last_message_id}"
は 数値.where になっているのでエラーとなります。
messages = Message.where("id > #{last_message_id}"
の間違いでは?

投稿2020/01/18 13:32

winterboum

総合スコア23284

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

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

ikdnaht

2020/01/18 14:15

申し訳ありません、書き方を誤っておりました。 message_groupsには数値ではなく、意図したmessageテーブルのレコードが入っていることが確認できた、の誤りです。ですので、message_groupsには、以下のようなmessageのレコードが複数入っております。 #<Message:0x00007fd2d28ebdc0 id: 119, content: "自動更新して", created_at: Sat, 18 Jan 2020 11:52:51 JST +09:00, updated_at: Sat, 18 Jan 2020 11:52:51 JST +09:00, user_id: 24, address: 19>, last_message_idにも意図した数値(こちらはid、数値です)が入っていたので、ajaxからの情報とDBの情報の差分がmessagesに代入できると考えていたのですが、うまくいかないようで・・・。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.50%

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

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

質問する

関連した質問