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

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

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

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

Q&A

解決済

1回答

1038閲覧

default_scopeを設定したらNoMethodErrorが発生した

404notfound

総合スコア14

Ruby on Rails 5

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

0グッド

0クリップ

投稿2019/07/13 15:19

編集2019/07/13 16:04

前提・実現したいこと

default_scopeを入れたことで今まで普通にみれていた画面でエラー(NoMethodError)が出るようになりました。
このエラーを解消し、元のように画面表示できるようにしたいです。

現在、railsでSNSアプリを作っています。
各ユーザーの詳細画面でユーザーが今まで投稿したツイートを新しい順で表示させようと思い、default_scopeを使ったところ、ユーザーが今までいいねをしたツイートを表示する画面でエラーが出るようになってしまいました。お恥ずかしながらdefault_scopeがあまり推奨されていないことを挿入後に知りました...
ちなみに、今まで一つもいいねをしていないユーザーのいいね一覧は今まで通り表示することができ、今まで一つでもいいねをしているユーザーのいいね一覧でエラーが出てしまいます。
-追記-
いいねしたことがあるユーザーでもいいね一覧が表示されるユーザーもいました。
今ローカル環境でユーザーが7人いるのですが、そのうち最初に作った1人のいいね一覧のみでNoMethodErrorが発生しています。

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

NoMethodError in Users#likes Showing Desktop/sample_app/app/views/users/likes.html.erb where line #27 raised: undefined method `user' for nil:NilClass Did you mean? super Extracted source (around line #27): 25 <div class="posts-index-item"> 26 <div class="post-left"> 27 <% if post.user.image.attached? %> 28 <%= image_tag post.user.image %> 29 <% else %> 30 <%= image_tag ("/default_user.jpg") %>

該当のソースコード

app/views/users/likes.html.erb

<div class="main user-show"> <div class="container"> <div class="user"> <% if @user.image.attached? %> <%= image_tag @user.image %> <% else %> <%= image_tag ("/default_user.jpg") %> <% end %> <h2><%= @user.name %></h2> <p><%= @user.profile %></p> <% if @user.id == @current_user.id %> <%= link_to("編集", "/users/#{@user.id}/edit") %> <% end %> </div> <ul class="user-tabs"> <li><%= link_to("投稿", "/users/#{@user.id}") %></li> <li class="active"><%= link_to("いいね!", "/users/#{@user.id}/likes") %></li> </ul> <% @likes.each do |like| %> <% post = Post.find_by(id: like.post_id) %> <div class="posts-index-item"> <div class="post-left"> <% if post.user.image.attached? %> <%= image_tag post.user.image %> <% else %> <%= image_tag ("/default_user.jpg") %> <% end %> </div> <div class="post-right"> <div class="post-user-name"> <%= link_to(post.user.name, "/users/#{post.user.id}") %> </div> <%= link_to(post.content, "/posts/#{post.id}") %> </div> </div> <% end %> </div> </div>

試したこと

エラー文を読む限りuserが認識されなくなったことは分かったのですが、良い対処法が思いつきませんでした。

default_scopeを入れたことでのエラー発生だったので、以下をそれぞれ試してみたのですが、エラー内容は変わりませんでした。
・app/models/post.rb内のdefault_scopeを記述した行の削除
・users.controller内のいいね一覧に関係する箇所を下記のように変更。

#変更前 def likes @user = User.find_by(id: params[:id]) @likes = Like.where(user_id: @user.id) end #変更後 def likes @user = User.find_by(id: params[:id]) @likes = Like.unscoped.where(user_id: @user.id) #unscopedを追加 end

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

関係がありそうなpost.rbをはります。
また、他にも必要なものがあれば教えていただければ幸いです。
よろしくお願いいたします。

app/models/post.rb

class Post < ApplicationRecord validates :content, {presence: true, length: {maximum: 140}} validates :user_id, {presence: true} default_scope -> { order(created_at: :desc) } def user return User.find_by(id: self.user_id) end end

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

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

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

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

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

hatsu

2019/07/13 15:40

>app/models/post.rb内のdefault_scopeを記述した行の削除 default_scopeを追加してエラーならば、上記でエラーは解決されると思うので何か別の問題があったと思われます。 <% post = Post.find_by(id: like.post_id) %>で正しくpostが取れているかが気になっているので、この行の後に<%= post %> でpostの値を表示した結果を見てみたいです。
404notfound

2019/07/13 15:52

ご回答ありがとうございます! そうなんですね、default_scopeを一度入れたら外すのは難しいといった記事をみたので該当行を消しただけでは消せないようなものなのだと思ってました... <% post = Post.find_by(id: like.post_id) %>の後に<%= post %>を入れてみたところ、質問と同様に<% if @user.image.attached? %>でNoMethodErrorが起きているというエラーがでます!
hatsu

2019/07/13 16:04

<% if post.user.image.attached? %>でエラーではなく<% if @user.image.attached? %>でエラーです? 質問時とエラー内容が変わらなければ、一時的にpost.userが書かれている箇所をコメントアウトし、<%= post %>の値を確認していただきたいです。。! <div class="posts-index-item"> <div class="post-left"> <% if post.user.image.attached? %> <%= image_tag post.user.image %> <% else %> <%= image_tag ("/default_user.jpg") %> <% end %> </div> <div class="post-right"> <div class="post-user-name"> <%= link_to(post.user.name, "/users/#{post.user.id}") %> </div> <%= link_to(post.content, "/posts/#{post.id}") %> </div> </div>
404notfound

2019/07/13 16:25

すみません、先ほどの@はコピーしてくるところを間違えていました。@はなしで<% if user.image.attached? %>でエラーが起きています。 コメントアウトして再度<%= post %>を表示させたところ、あるユーザーのpostの出力は「#<Post:0x00007f9cb58d8728> #<Post:0x00007f9cb5903978>」と表示されました。 また、質問に追記したのですが、質問した時のコードでは、いいねしたことがあるユーザー全員でエラーが出るのではなく、ユーザー7人のうち最初に登録したユーザーだけでエラーが出ることがわかりました。特定のユーザーのみでエラーが発生するということは何か他のところに原因があるのでしょうか.... コメントアウトすると、最初に登録したユーザーもエラーが起きずpostを出力できました。
404notfound

2019/07/13 19:36

投稿した方法で解決することができました! この質問に書いたコードからだけで解決できない問題であり、お手数をお掛けしてしまい大変申し訳ないです。 一緒に考えてくださり誠にありがとうございましたm(_ _)m
guest

回答1

0

自己解決

ユーザーによりエラーが発生するユーザーとしないユーザーがいたのですが、エラー発生の条件が過去にいいねした投稿が削除されていた場合、userが取ってこれなくなっていることがわかりました。

以下の方法でエラーは発生しなくなり、無事にアプリが動くようになりました。

・posts_controller.rbで投稿を削除した場合はその投稿をいいねしていたデータも消えるように変更
・今までローカル環境でのLikeテーブルに関するデータを全消去

投稿2019/07/13 19:32

404notfound

総合スコア14

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

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

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.50%

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

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

質問する

関連した質問