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

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

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

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

Ruby

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

Q&A

解決済

1回答

307閲覧

userのalbumにpictureを紐付けたい

退会済みユーザー

退会済みユーザー

総合スコア0

Ruby on Rails 5

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

Ruby

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

0グッド

0クリップ

投稿2019/02/12 04:26

編集2019/02/12 04:35

userのalbumにpictureを紐付けたい

Ruby on Railsでアルバムアプリを作成しています。
userが多数のalbumを持っており、albumが多数のpictureを持っているという構図です。

albumのshowページに、そのalbumと紐付くpictureを表示したいのですが上手くいきません。
user1のalbum2に写真を表示したいのに、user1のalbum1に表示されてしまいます。

おそらくpictureの作成時にalbumとの紐付けができていないのだと思うのですが、解決策が分からずハマっています。
お手数ですがどなたかご教示ください。

ソースコード

class PicturesController < ApplicationController def create @album = current_user.albums.find_by(params[:id]) @picture = @album.pictures.build(picture_params) if @picture.save flash[:success] = "写真を追加しました!" redirect_to @album else render 'albums/show' end end private def picture_params params.fetch(:picture, {}).permit(:picture) end end

補足情報

同じcreateアクションのredirect_to、render共にalbum1のshowページが表示されてしまいます。

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

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

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

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

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

guest

回答1

0

ベストアンサー

まずはparams[:id]で、きちんとalbum2のIDが来ているのかを確認しましょう。
説明を見る限りはparams[:id]がalbum1のIDになっているように思います。

投稿2019/02/12 09:49

--KT--

総合スコア90

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

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

退会済みユーザー

退会済みユーザー

2019/02/12 11:56 編集

ご回答頂きありがとうございます! ご指摘の通り、album2ではなくalbum1のidが来ていました。 createアクションの一行目で 下記の書き方も試してみたのですが、どちらもエラーとなり上手く行きませんでした。 1. @album = current_user.albums.find(params[:id]) → ActiveRecord::RecordNotFound in PicturesController#create   Couldn't find Picture without an ID 2. @album = current_user.albums.find_by(id: params[:id]) → NoMethodError in PicturesController#create   undefined method `pictures' for nil:NilClass ちなみに2の「@album = current_user.albums.find_by(id: params[:id])」という書き方は AlbumsControllerでも以下のように使っているのですが、こちらは問題なく動作しているようです。 def correct_user @album = current_user.albums.find_by(id: params[:id]) redirect_to root_url if @album.nil? end 何が問題なのか検討がつきますでしょうか? お手数をお掛けしますが、お教え頂けると幸いです。
--KT--

2019/02/13 01:30

記載箇所とエラー箇所が違うのでなんとも言えませんが そもそも書き方の問題ではなく、params[:id]が間違っているので createアクションを呼び出しているview側の記述に問題があると思います。
退会済みユーザー

退会済みユーザー

2019/02/13 06:13

おっしゃる通りviewに問題がありました。 訂正したところ正しく動くようになりました。 一週間ハマっていたのですが、おかげさまで先に進むことができます。 本当にありがとうございました!
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.50%

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

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

質問する

関連した質問