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

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

ただいまの
回答率

89.20%

ActiveRecord::RecordNotFound in MessagesController#index

解決済

回答 3

投稿 編集

  • 評価
  • クリップ 1
  • VIEW 624

Rick_darts

score 10

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

前提・実現したいこと

エラー解除

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

イメージ説明

該当のソースコード

コントローラーの記載

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とは別にルートを記載しないといけないのか、
コントロール部分のミスなのか
どう変えればいいかがわからず困っています。

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

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 3

check解決した方法

0

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

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

0

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

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

0

設計のミスです。

root 'messages#index'

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

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

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

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