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

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

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

Rubyはプログラミング言語のひとつで、オープンソース、オブジェクト指向のプログラミング開発に対応しています。

Ruby on Rails

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

Q&A

解決済

3回答

3578閲覧

ActiveRecord::RecordNotFound in MessagesController#index

Rick_darts

総合スコア10

Ruby

Rubyはプログラミング言語のひとつで、オープンソース、オブジェクト指向のプログラミング開発に対応しています。

Ruby on Rails

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

0グッド

1クリップ

投稿2019/11/06 07:54

編集2019/11/06 07:57

初学者の為、必要な情報が漏れている場合がございます。
漏れていた場合ご指摘いただけると幸いです。

前提・実現したいこと

エラー解除

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

イメージ説明

該当のソースコード

コントローラーの記載

class MessagesController < ApplicationController before_action :set_group def index @message = Message.new @messages = @group.messages.includes(:user) end def create @message = @group.messages.new(message_params) if @message.save redirect_to group_messages_path(@group), notice: 'メッセージが送信されました' else @messages = @group.messages.includes(:user) flash.now[:alert] = 'メッセージを入力してください。' render :index end end private def message_params params.require(:message).permit(:content, :image).merge(user_id: current_user.id) end def set_group @group = Group.find(params[:group_id]) @groups = current_user.groups end end

ルーティング

Rails.application.routes.draw do devise_for :users root 'messages#index' resources :users, only: [:edit, :update] resources :groups, only: [:new, :create, :edit, :update] do resources :messages, only: [:create, :index] end end

rake routes

Prefix Verb URI Pattern Controller#Action new_user_session GET /users/sign_in(.:format) devise/sessions#new user_session POST /users/sign_in(.:format) devise/sessions#create destroy_user_session DELETE /users/sign_out(.:format) devise/sessions#destroy new_user_password GET /users/password/new(.:format) devise/passwords#new edit_user_password GET /users/password/edit(.:format) devise/passwords#edit user_password PATCH /users/password(.:format) devise/passwords#update PUT /users/password(.:format) devise/passwords#update POST /users/password(.:format) devise/passwords#create cancel_user_registration GET /users/cancel(.:format) devise/registrations#cancel new_user_registration GET /users/sign_up(.:format) devise/registrations#new edit_user_registration GET /users/edit(.:format) devise/registrations#edit user_registration PATCH /users(.:format) devise/registrations#update PUT /users(.:format) devise/registrations#update DELETE /users(.:format) devise/registrations#destroy POST /users(.:format) devise/registrations#create root GET / messages#index edit_user GET /users/:id/edit(.:format) users#edit user PATCH /users/:id(.:format) users#update PUT /users/:id(.:format) users#update group_messages GET /groups/:group_id/messages(.:format) messages#index POST /groups/:group_id/messages(.:format) messages#create groups POST /groups(.:format) groups#create new_group GET /groups/new(.:format) groups#new edit_group GET /groups/:id/edit(.:format) groups#edit group PATCH /groups/:id(.:format) groups#update PUT /groups/:id(.:format) groups#update

試したこと

https://teratail.com/questions/50715
https://teratail.com/questions/30646
https://programming-beginner-zeroichi.jp/articles/45

等の記事を読ませていただきました。

ルーティングに問題があるということは理解ができましたが、
ルーティングのresourcesとは別にルートを記載しないといけないのか、
コントロール部分のミスなのか
どう変えればいいかがわからず困っています。

ご指摘ご指導お願いいたします。

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

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

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

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

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

guest

回答3

0

自己解決

設計ミスのため作り直しました。
改善点としては、routeの作り直しました。
理由としては、メッセージのままだとデータが引っ張ってこれず(やり方はあるみたいですが)
変にコードが増えるため、グループ/indexに書き換えました。

投稿2019/11/10 15:53

Rick_darts

総合スコア10

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

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

0

設計のミスです。

root 'messages#index'

MessagesControllerを使いまわしていますが、
ルートにアクセスした場合の期待している挙動(どのグループを表示するのか)が不明です。

投稿2019/11/06 08:48

asm

総合スコア15147

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

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

0

ActiveRecord::RecordNotFound ですのでルーティングは(直接は)関係ありません。
Groupにあるレコードが見つからないと出ています。
Groupの中に対象のレコードが存在するのか、存在するとして指定する方法は誤っていないか、などを調べる必要があります。

投稿2019/11/06 08:15

退会済みユーザー

退会済みユーザー

総合スコア0

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

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

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.49%

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

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

質問する

関連した質問