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

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

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

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

Devise

Deviseとは、Ruby-on-Railsの認証機能を追加するプラグインです。

Active Record

Active Recordは、一つのオブジェクトに対しドメインのロジックとストレージの抽象性を結合するデザインパターンです。

Q&A

解決済

2回答

2989閲覧

いいね機能実装中にエラー(NoMethodError)が発生しました

ikdnaht

総合スコア13

Ruby on Rails 5

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

Devise

Deviseとは、Ruby-on-Railsの認証機能を追加するプラグインです。

Active Record

Active Recordは、一つのオブジェクトに対しドメインのロジックとストレージの抽象性を結合するデザインパターンです。

0グッド

0クリップ

投稿2020/01/05 14:05

前提・実現したいこと

ruby on rails 初学者です。
写真投稿アプリを制作中、
いいね機能の実装の際にエラーメッセージが発生しました。

現状、deviseを導入した上で、
likeテーブルを作り、
自分がいいねをしていない場合は「いいね」、
いいねをしている場合は「いいねを取り消す」という処理にしたいです。

こちらの記事を参考にしました。
https://eiji-hb.hatenablog.com/entry/2019/10/23/151224

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

NoMethodError in Posts#index undefined method `id' for nil:NilClass

該当のソースコード

app/view/posts/index.html.haml

haml

1.post_lists 2 - @posts.each do |post| 3 %div.post_content 4 %div.post_content__upper_box 5 %div.post_content__upper_box__name 6 = link_to post.user.name, user_path(post.user), class: "post_content__upper_box__username" 7 - if user_signed_in? && current_user.id == post.user_id 8 = link_to "投稿を削除", post_path(post.id), method: :delete, class: "post_content__upper_box__delete" 9 = image_tag post.image.url, class: "post_content__image" 10 %div.post_content__rower_box 11 %div.post_content__rower_box__caption 12 = post.text 13 %div.post_content__rower_box__like 14 -if current_user.likes.find_by(post_id: @post.id) 15 = link_to 'いいね!を取り消す', post_like_path(@post), method: :delete 16 -else 17 = link_to 'いいね!', post_likes_path(@post), method: :post

app/config/routes.rb

ruby

1Rails.application.routes.draw do 2 devise_for :users 3 root to: 'posts#index' 4 5 resources :users, only: [:index, :show] 6 resources :posts, only: [:index, :new, :create, :destroy] do 7 resources :likes, only: [:create, :destroy] 8 end 9end

app/controllers/posts_controller.rb

ruby

1class PostsController < ApplicationController 2 before_action :authenticate_user!, only: [:show, :new] 3 4 def index 5 @posts = Post.includes(:user).order(created_at: :DESC) 6 @like = Like.new 7 end 8 9 def show 10 @post = Post.find(params[:id]) 11 end 12 13 def new 14 @posts = Post.new 15 end 16 17 def create 18 @post = Post.new(post_params) 19 if @post.save 20 redirect_to root_path 21 else 22 render :new 23 end 24 end 25 26 def destroy 27 post = Post.find(params[:id]) 28 post.destroy 29 redirect_to root_path 30 end 31 32 private 33 def post_params 34 params.require(:post).permit(:image, :text).merge(user_id: current_user.id) 35 end 36 37end

app/controllers/likes_controller.rb

ruby

1class LikesController < ApplicationController 2 before_action :set_post, only: [:create, :destroy] 3 def create 4 @like = current_user.likes.create(like_params) 5 redirect_to root_path(@post) 6 end 7 8 def destroy 9 @like = Like.find_by(like_params, user_id: current_user.id) 10 @like.destroy 11 redirect_to root_path(@post) 12 end 13 14 private 15 def set_post 16 @post = Post.find(params[:post_id]) 17 end 18 19 def like_params 20 params.permit(:post_id) 21 end 22end 23

app/controllers/users_controller.rb

ruby

1class UsersController < ApplicationController 2 3 def index 4 @users = User.all 5 end 6 7 def show 8 @user = User.find(params[:id]) 9 @posts = @user.posts 10 end 11 12end 13

試したこと

app/view/posts/index.html.haml内の条件分岐、「-if current_user.likes.find_by(post_id: @post.id)」以下において、post_idが取れていないことが原因かと考え、routes.rbのネストができているか確認をしてみました。

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

Rails 5.2.4.1
ruby 2.5.1

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

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

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

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

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

guest

回答2

0

自己解決

app/view/posts/index.html.hamlを下記のようにすることで、暫定的にですが、エラーの解消ができました。

.post_lists - @posts.each do |post| %div.post_content %div.post_content__upper_box %div.post_content__upper_box__name = link_to post.user.name, user_path(post.user), class: "post_content__upper_box__username" - if user_signed_in? && current_user.id == post.user_id = link_to "投稿を削除", post_path(post.id), method: :delete, class: "post_content__upper_box__delete" = image_tag post.image.url, class: "post_content__image" %div.post_content__rower_box %div.post_content__rower_box__caption = post.text %div.post_content__rower_box__like - like = current_user.likes.find_by(post_id: post.id) - if like != nil = link_to 'いいね!を取り消す', post_like_path(post,like), method: :delete - else = link_to 'いいね!', post_likes_path(post), method: :post

投稿2020/01/08 06:19

ikdnaht

総合スコア13

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

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

0

controllerのエラーの様な表示ですが多分viewの方ですね。
viewに @post というのがあります。複数形の@postsは定義されていますが単数形は定義されていません。ので @post.id で掲示のエラーになります。
@のない、post.id の間違いだと思います

投稿2020/01/06 01:16

winterboum

総合スコア23284

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

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

ikdnaht

2020/01/06 01:57

ご回答いただきありがとうございます! おっしゃっていただいた通り、post.idに変更したところ、indexのビューが表示されるようにはなったのですが、いいねを押すと今度は ActionController::UrlGenerationError in Posts#index と表示されてしまいます。 ```haml -if current_user.likes.find_by(post_id: post.id) = link_to 'いいね!を取り消す', post_like_path(post), method: :delete - else = link_to 'いいね!', post_likes_path(post), method: :post ``` post_likes_path(post)の記述では、post_idが取ってこれないのでしょうか?
winterboum

2020/01/06 02:11 編集

routes.rb の記述漏れですね。 通常は index は GET のものなので、そこにPOSTとかDELETEとか入れるならそれなりの宣言が必要です。 あ、 違うかも。routes.rb の問題ではありそうですが
winterboum

2020/01/06 02:12

post_likes_path(post) →  post_like_path(post) かな
ikdnaht

2020/01/06 02:23

rails routesでは、下記のようになっていたので、createアクションにはpost_likes_path(post)をあてていたのですが違いますでしょうか? post_likes POST /posts/:post_id/likes(.:format) likes#create post_like DELETE /posts/:post_id/likes/:id(.:format) likes#destroy routes.rbは、下記のような状態です Rails.application.routes.draw do devise_for :users root to: 'posts#index' resources :users, only: [:index, :show] resources :posts, only: [:index, :new, :create, :destroy] do resources :likes, only: [:create, :destroy] end end
winterboum

2020/01/06 02:28

rails routes で post_likes POST /posts/:post_id/likes(.:format) likes#create となってます??
ikdnaht

2020/01/06 02:42

なっています!
winterboum

2020/01/06 03:08

それも、うぅむ〜〜 ですが。。。 それで likes#createに飛んでいないということは /posts/:post_id/likes になっていないのかな その部分のhtmlに変換されたところどうなってます?
ikdnaht

2020/01/06 03:13

= link_to 'いいね!', post_likes_path(post), method: :post としています!
winterboum

2020/01/06 03:14

いえ、それがどうhtmlに変換されたか、 です。
ikdnaht

2020/01/06 03:28

申し訳ありません。検証ツールでは、該当箇所が下記のように表示されています。 どうhtmlに変換されているかとは、こういった意味でしょうか? <a rel="nofollow" data-method="post" href="/posts/36/likes">いいね!</a>
winterboum

2020/01/06 03:52

それです。href="/posts/36/likes" になってますね。 この件私の理解を超えているのが2点 1) post_likes_path と post なのに複数形になっている 2) そのとおりのURLなのに Post#indexに飛ぶ あと一点確認させていただいて、それで理解不能でしたらごめんなさいです。 rails routes の先頭からpost_likesまでを見せてください
ikdnaht

2020/01/06 04:30

2)に関しては、likes_controllerのcreateアクションにredirect_to root_pathを入れているからでしょうか?いいねをした後に、root_pathのPost#indexページに戻したいという意図で、記述してしまっています。 rails 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 / posts#index users GET /users(.:format) users#index user GET /users/:id(.:format) users#show post_likes POST /posts/:post_id/likes(.:format) likes#create post_like DELETE /posts/:post_id/likes/:id(.:format) likes#destroy
winterboum

2020/01/06 06:23

すみません、お手上げです。 前の方に似たような宣言があって、そちらに引っ張られてるってのを想定したのですがありませんでした。
ikdnaht

2020/01/06 07:56

そうでうか。 ありがとうございます!エラーに対するアプローチの仕方など、勉強になりました!もう少しコードを見て、調べてみます!
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.50%

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

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

質問する

関連した質問