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

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

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

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

Q&A

解決済

1回答

2125閲覧

NoMethodError (undefined method `[]' for nil:NilClass)

majikadezou

総合スコア34

Ruby on Rails

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

0グッド

0クリップ

投稿2019/12/23 09:05

編集2019/12/23 09:08

ユーザーのフォロー機能を作成しています。

フォローボタンをクリックするとエラーが出てしまいます。

エラーコード NoMethodError (undefined method `[]' for nil:NilClass): app/controllers/relationships_controller.rb:31:in `set_user' Started POST "/relationships" for ::1 at 2019-12-23 17:59:16 +0900 Processing by RelationshipsController#create as HTML Parameters: {"authenticity_token"=>"dHHJPTPNPibarpwoozu6+rEKQiiEjMVIyudoSuLIaA/Aix9J2iLNCM3DmPjvWmNgabq/lcTYAFjJCgios8mAVw==", "follow_id"=>"3", "commit"=>"Follow"}
relationships_controller class RelationshipsController < ApplicationController before_action :set_user def create user = User.find(params[:relationship][:follow_id]) following = current_user.follow(user) if following.save flash[:success] = 'ユーザーをフォローしました' redirect_to user else flash.now[:alert] = 'ユーザーのフォローに失敗しました' redirect_to user end end def destroy user = User.find(params[:relationship][:follow_id]) following = current_user.unfollow(user) if following.destroy flash[:success] = 'ユーザーのフォローを解除しました' redirect_to user else flash.now[:alert] = 'ユーザーのフォロー解除に失敗しました' redirect_to user end end private def set_user user = User.find(params[:relationship][:follow_id]) end end
フォローフォームの部分テンプレート <% unless current_user == user %> <% if current_user.following?(user) %> <%= form_for(current_user.relationships.find_by(follow_id: user.id), html: { method: :delete }) do |f| %> <%= hidden_field_tag :follow_id, user.id %> <%= f.submit 'Unfollow', class: 'btn btn-danger btn-block' %> <% end %> <% else %> <%= form_for(current_user.relationships.build) do |f| %> <%= hidden_field_tag :follow_id, user.id %> <%= f.submit 'Follow', class: 'btn btn-primary btn-block' %> <% end %> <% end %> <% end %>

部分テンプレートのform_forの書き方が悪いのかと思ったのですが、変えてみてもうまくいきません。[:relationship]が読み取れていないようなのですが。。。
分かる人いませんか?

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

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

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

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

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

guest

回答1

0

ベストアンサー

params[:relationship][:follow_id] の params[:relationship] がパラメータで送られてきていないためのエラーです
その原因は
hidden_field_tag :follow_id,
だから
f.hidden_field :follow_id,

とするか
params[:follow_id]
とするか。

なお、二つ問題があります。
before_action :set_user なので、indexなどparamsがこないactionでも呼ばれてしまうので、そのときエラーになります。only: [...] をつけると良いです

set_user では user = となっています。これでは 実際のactionでは使えません
@user = として、createなどでは @user で参照するようにしてください

投稿2019/12/23 09:21

編集2019/12/23 09:26
winterboum

総合スコア23284

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

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

majikadezou

2019/12/23 10:05

解決しました。ありがとうございます
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.50%

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

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

質問する

関連した質問