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

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

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

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

Ruby on Rails

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

Q&A

解決済

1回答

1054閲覧

ネスト時のupdateアクションのエラーについて

Masahiro85

総合スコア2

Ruby

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

Ruby on Rails

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

0グッド

0クリップ

投稿2020/11/03 02:15

編集2020/11/03 02:38

前提・実現したいこと

あるテーマに仮説を立てるアプリを作成しています。
あるテーマに紐づく仮説をネストの関係でルーティングしていますが、
仮説のupdateアクションでエラーが発生してしまいます。
コントローラの記述に誤りがあると推測していますが、そこでつまづいてしまっています。
すみませんが、ご教授いただきたくよろしくお願いします。

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

ActiveRecord::AssociationNotFoundError in GuessesController#update Association named '2' was not found on Guess; perhaps you misspelled it?
  • エラー発生時のParameter
{"_method"=>"patch", "authenticity_token"=>"9gsX0/ZOLwpR+/LBLrpDQ6alhQMwyzsefjy6+ZgtwtALEVBFVzlhyzbRKZJMCT5srUmCS1kFRPzC+cJq0VR7nw==", "guess"=>{"story"=>"gess1-2 new"}, "commit"=>"保存する", "moyatto_id"=>"1", "id"=>"2"}

該当のソースコード

  • guesses_contorollerの記述

ruby

1 def update 2 @moyatto = Moyatto.find(params[:moyatto_id]) 3 @guesses = @moyatto.guesses.includes(params[:id]) 4 binding.pry 5 @guess = @guesses.update(guess_params) 6 end 7 8 private 9 def guess_params 10 params.require(:guess).permit(:story).merge(user_id: current_user.id,moyatto_id: params[:moyatto_id]) 11 end
  • ルーティング
resources :moyattos, only:[:new,:create,:show,:edit,:update,:destroy] do resources :guesses, only:[:new,:create,:edit,:update] end
  • 各Modelとのアソシエーション
# Guessモデル belongs_to :user belongs_to :moyatto # Moyattoモデル belongs_to :user has_one_attached :image has_many :guesses # Userモデル devise :database_authenticatable, :registerable, :recoverable, :rememberable, :validatable has_many :moyattos has_many :guesses
  • VIew edit.html.erb
<%= form_with model:[@moyatto,@guess], local: true do |f| %> <p>仮説を登録</p> <%= f.text_area :story %> <%= f.submit "保存する" %> <% end %>

試したこと

  • guess_paramsにid: params[:id]をmergeしてみましたが、エラーは変わりませんでした。

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

  • binding_pryの時のguess_paramsの内容
=> <ActionController::Parameters {"story"=>"gess1-2 new", "user_id"=>1, "moyatto_id"=>"1", "id"=>"2"} permitted: true>

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

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

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

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

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

winterboum

2020/11/03 02:24

updateに @guesses = @moyatto.guesses.、、、 とありますが、ある一つのguessをupdateするのだとおもいますが違いますか? とすると @moyatto.guesseからどうやって選ぶのかその条件をおしえてください
Masahiro85

2020/11/03 02:30

ありがとうございます。おっしゃる通りでmoyattoに紐づく、guessの中から1つをupdateしたいです。 選ぶ記述は、'.includes(params[:id])' で今持っているidから取得して @guessesへ入れ、これにupdateの情報を渡したいと思っています。 paramsにidが入っており、取得可能と思いました。
winterboum

2020/11/03 02:33

viewがないので判断できませんが、この id はguessのidなのですか?
Masahiro85

2020/11/03 02:39

ソースコードにviewを追加いたしました。 この内容からguessのidだと思っています。 (違うでしょうか?)
guest

回答1

0

ベストアンサー

@guesses = @moyatto.guesses.includes(params[:id])
が凄まじいです
@guess = Guess.find(params[:id]) で。
その後も
@guess.update(guess_params)

投稿2020/11/03 02:41

winterboum

総合スコア23567

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

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

Masahiro85

2020/11/03 02:49

無事updateできました。 @moyatto.guesses.includes(params[:id])の記述は凄まじいのですね、勉強になりました。 どうもありがとうございます!
winterboum

2020/11/03 02:51

はい、意味不明です @moyatto.guesses は良いです @moyatto.guesses.includes()とすると、guessesを取り込むときにN+1問題回避のために他の関連も取り込んでね、 ということなのでありえます。 が、、、、そこに関連のモデルではなく IDが、、、
winterboum

2020/11/03 02:53

あと、 変数名の単数形複数形には注意を払ってください。 大抵は人間が誤動作するだけですが、ときにRailsも誤動作します
Masahiro85

2020/11/03 03:02

.includesのスマートな使い所はもう一度学習いたします! 変数名のご指摘も承知しました。 どうもありがとうございました。
winterboum

2020/11/03 03:58

includesのスマートな使い所 ではなく、includes 自体の使い方をまず。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.35%

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

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

質問する

関連した質問